Author: stefan2
Date: Sat Feb 19 09:36:51 2011
New Revision: 1072283

URL: http://svn.apache.org/viewvc?rev=1072283&view=rev
Log:
Fix style issues reported in http://svn.haxx.se/dev/archive-2011-02/0391.shtml .

* subversion/svnserve/server.h
  (serve_params_t.compression_level): fix copy'n'pasto in docstring
* subversion/svnserve/main.c
  (main): use defines instead of magic numbers

Found by: danielsh

Modified:
    subversion/branches/integrate-compression-level/subversion/svnserve/main.c
    subversion/branches/integrate-compression-level/subversion/svnserve/server.h

Modified: 
subversion/branches/integrate-compression-level/subversion/svnserve/main.c
URL: 
http://svn.apache.org/viewvc/subversion/branches/integrate-compression-level/subversion/svnserve/main.c?rev=1072283&r1=1072282&r2=1072283&view=diff
==============================================================================
--- subversion/branches/integrate-compression-level/subversion/svnserve/main.c 
(original)
+++ subversion/branches/integrate-compression-level/subversion/svnserve/main.c 
Sat Feb 19 09:36:51 2011
@@ -569,10 +569,10 @@ int main(int argc, const char *argv[])
 
         case 'c':
           params.compression_level = atoi(arg);
-          if (params.compression_level < 0)
-            params.compression_level = 0;
-          if (params.compression_level > 9)
-            params.compression_level = 9;
+          if (params.compression_level < SVN_NO_COMPRESSION_LEVEL)
+            params.compression_level = SVN_NO_COMPRESSION_LEVEL;
+          if (params.compression_level > SVN_BEST_COMPRESSION_LEVEL)
+            params.compression_level = SVN_BEST_COMPRESSION_LEVEL;
           break;
 
         case 'M':

Modified: 
subversion/branches/integrate-compression-level/subversion/svnserve/server.h
URL: 
http://svn.apache.org/viewvc/subversion/branches/integrate-compression-level/subversion/svnserve/server.h?rev=1072283&r1=1072282&r2=1072283&view=diff
==============================================================================
--- 
subversion/branches/integrate-compression-level/subversion/svnserve/server.h 
(original)
+++ 
subversion/branches/integrate-compression-level/subversion/svnserve/server.h 
Sat Feb 19 09:36:51 2011
@@ -114,7 +114,7 @@ typedef struct serve_params_t {
 
   /* Data compression level to reduce for network traffic. If this
      is 0, no compression should be applied and the protocol may
-     fall back to txdelta "version 0" bypassing zlib entirely.
+     fall back to svndiff "version 0" bypassing zlib entirely.
      Defaults to SVN_DEFAULT_COMPRESSSION_LEVEL. */
   int compression_level;
 


Reply via email to