You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2010/08/17 23:28:04 UTC
svn commit: r986492 -
/subversion/branches/performance/subversion/libsvn_subr/stream.c
Author: stefan2
Date: Tue Aug 17 21:28:04 2010
New Revision: 986492
URL: http://svn.apache.org/viewvc?rev=986492&view=rev
Log:
Fix signedness and integer size mismatches.
* subversion/libsvn_subr/stream.c
(stream_readline_chunky, skip_handler_apr): fix integer usage
Modified:
subversion/branches/performance/subversion/libsvn_subr/stream.c
Modified: subversion/branches/performance/subversion/libsvn_subr/stream.c
URL: http://svn.apache.org/viewvc/subversion/branches/performance/subversion/libsvn_subr/stream.c?rev=986492&r1=986491&r2=986492&view=diff
==============================================================================
--- subversion/branches/performance/subversion/libsvn_subr/stream.c (original)
+++ subversion/branches/performance/subversion/libsvn_subr/stream.c Tue Aug 17 21:28:04 2010
@@ -376,7 +376,7 @@ stream_readline_chunky(svn_stringbuf_t *
svn_stream_mark_t *mark;
apr_size_t numbytes;
const char *eol_pos;
- apr_off_t total_parsed = 0;
+ apr_size_t total_parsed = 0;
/* invariant for this call */
size_t eol_len = strlen(eol);
@@ -880,7 +880,12 @@ static svn_error_t *
skip_handler_apr(void *baton, apr_size_t *count)
{
struct baton_apr *btn = baton;
- return svn_io_file_seek(btn->file, SEEK_CUR, count, btn->pool);
+ apr_off_t offset = *count;
+
+ SVN_ERR(svn_io_file_seek(btn->file, SEEK_CUR, &offset, btn->pool));
+ *count = offset;
+
+ return SVN_NO_ERROR;
}
static svn_error_t *