mkgroup usage/version patch

Joshua Daniel Franklin joshuadfranklin@yahoo.com
Thu Mar 14 15:54:00 GMT 2002


--- Corinna Vinschen <cygwin-patches@cygwin.com> wrote:
> While I'm going with the decision to add a version information
> I don't understand why you changed 
> 
> - the usage output method.  I don't like to have a big multiline format
>   string in fprintf.
Well, I could just as easily changed all the lines to 
fprintf(stream, "...
I just used :s/foo/bar/12 anyway. I did it with the multiline since 
1. That's how cygcheck, cygpath, etc. do it now
2. I find it easier to read, personally. (It looks a lot like the 
   CLI output that way)
 
> - the function usage() to exit instead of returning the exitcode.
Again, no reason. That's just the way some of the other utils work.
I was going for consitency in all the code in /winsup/utils

Looking at the patch on the website at
http://cygwin.com/ml/cygwin-patches/2002-q1/msg00342.html
I'm seeing something else weird:
+	        print_version ();
+                exit (0);
                ^
What's this extra space? That's not in my code. Do other people see it
in their attachments, or is it just in the HTML? I already deleted the
message from myself.

__________________________________________________
Do You Yahoo!?
Yahoo! Sports - live college hoops coverage
http://sports.yahoo.com/



More information about the Cygwin-patches mailing list