You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by iv...@apache.org on 2011/02/19 20:05:48 UTC

svn commit: r1072419 - /subversion/trunk/subversion/libsvn_diff/deprecated.c

Author: ivan
Date: Sat Feb 19 19:05:47 2011
New Revision: 1072419

URL: http://svn.apache.org/viewvc?rev=1072419&view=rev
Log:
Remove several compiler warnings.

* subversion/libsvn_diff/deprecated.c
  (datasources_open): Use apr_size_t to remove warning about 
   signed/unsigned mismatch.
  (token_discard, token_discard_all): Do not return void. It seems Visual 
   Studio compiler doesn't like such syntax.

Modified:
    subversion/trunk/subversion/libsvn_diff/deprecated.c

Modified: subversion/trunk/subversion/libsvn_diff/deprecated.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_diff/deprecated.c?rev=1072419&r1=1072418&r2=1072419&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_diff/deprecated.c (original)
+++ subversion/trunk/subversion/libsvn_diff/deprecated.c Sat Feb 19 19:05:47 2011
@@ -56,7 +56,7 @@ datasources_open(void *baton, apr_off_t 
                  apr_size_t datasource_len)
 {
   struct fns_wrapper_baton *fwb = baton;
-  int i;
+  apr_size_t i;
 
   /* Just iterate over the datasources, using the old singular version. */
   for (i = 0; i < datasource_len; i++)
@@ -104,14 +104,14 @@ token_discard(void *baton,
               void *token)
 {
   struct fns_wrapper_baton *fwb = baton;
-  return fwb->vtable->token_discard(fwb->old_baton, token);
+  fwb->vtable->token_discard(fwb->old_baton, token);
 }
 
 static void
 token_discard_all(void *baton)
 {
   struct fns_wrapper_baton *fwb = baton;
-  return fwb->vtable->token_discard_all(fwb->old_baton);
+  fwb->vtable->token_discard_all(fwb->old_baton);
 }