Hi Hrvoje,

yeah.. i figured that out 5 mins after i sent the email!!! been looking at
this for 4-5 hours!!!

btw, i just saw the comment you menttioned in the init.c file.... is it
possible to add this same comment to main.c.

when i started to add the new option, i was looking at/in main.c, and never
saw any comment there...!!!!

it would potentially help others who are loking to modify the app..

thanks

-bruce


-----Original Message-----
From: Hrvoje Niksic [mailto:[EMAIL PROTECTED]
Sent: Wednesday, June 28, 2006 10:01 AM
To: [EMAIL PROTECTED]
Cc: 'Mauro Tortonesi'; 'Tony Lewis'; [email protected]
Subject: Re: wget - Adding Cmdline Option


"bruce" <[EMAIL PROTECTED]> writes:

>  add to 'struct cmdline_option option_data[]'
>       .
>       .
>       { "version", 'V', OPT_FUNCALL, (void *) print_version,
no_argument },
>       { "wait", 'w', OPT_VALUE, "wait", -1 },
>>>>   { "college-file", 'C', OPT_VALUE, "collegefile", -1 },
>       { "waitretry", 0, OPT_VALUE, "waitretry", -1 },
>   };

That array needs to be ordered alphabetically, by command name.  (Wget
uses binary search for searching the list, so the ordering really is
necessary.)  That's why you're triggering the assert.  The comment
above the array definitions mentions that.

Reply via email to