[asterisk-commits] mmichelson: branch mmichelson/npm_fixes r181013 - /team/mmichelson/npm_fixes/...

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Tue Mar 10 18:03:12 2009
New Revision: 181013

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=181013
Log:
Remove a comment of mine since I now understand the logic.

I'm not bothering to modify this comment since it became massively
obvious to me why the code was written like it was once I actually
thought about it :)


Modified:
    team/mmichelson/npm_fixes/main/pbx.c

Modified: team/mmichelson/npm_fixes/main/pbx.c
URL: http://svn.digium.com/svn-view/asterisk/team/mmichelson/npm_fixes/main/pbx.c?view=diff&rev=181013&r1=181012&r2=181013
==============================================================================
--- team/mmichelson/npm_fixes/main/pbx.c (original)
+++ team/mmichelson/npm_fixes/main/pbx.c Tue Mar 10 18:03:12 2009
@@ -1917,8 +1917,6 @@
 				} else if (*s1 == '-') { /* remember to add some error checking to all this! */
 					char s3 = *(s1-1);
 					char s4 = *(s1+1);
-					/* XXX MCM - This seems wrong. Why increment s3 at the beginning of the for loop?
-					 */
 					for (s3++; s3 <= s4; s3++) {
 						*s2++ = s3;
 					}