Huh, sorry, that last mail went only to you (Samuel) :) Hi Samuel, btw.
Some weeks ago i did an option-parsing stuff myself, but since I am really no expert on glib, and the code was kind of ugly I did not submit. What I did was, for all those search commands, i added an option to limit the number of results, and an option to specify the offset, I added two files (.h, .c) which held some macros for the generic options... How does that sound to you? By the way, 'not expert on glib' I really mean _no_ expert on glib, your code is really smooth about all those things I did not knew :) That much for now, rgds, Marcus On 9/17/06, Samuel Cormier-Iijima <[EMAIL PROTECTED]> wrote: > On 9/17/06, Marcus Fritzsch <[EMAIL PROTECTED]> wrote: > > Hi There, > > > > On 9/17/06, Samuel Cormier-Iijima <[EMAIL PROTECTED]> wrote: > > > On 9/17/06, Jamie McCracken <[EMAIL PROTECTED]> wrote: > > > > Samuel Cormier-Iijima wrote: > > > > > Here's another small patch for GOption support, this time in > > > > > tracker-search. It also adds the option of limiting the number of > > > > > results, makes use of g_strjoinv, and other small changes. > > > > > > > > > > > > > great thanks - good work! > > > > > > > > Have now committed > > > > > > > > It would be nice to also add service option as well to tracker-search? > > > > > > > > EG tracker-search -s Music rock > > > > > > ok, i've added this. also, i made it do a synchronous call instead of > > > async > > > > > > Samuel Cormier-Iijima > > > > Nice work indeed ;) - but one thing, I think it would be better to > > print a warning if the service is not recognised, that warning should > > also mention that the service searched is Other Files - just a > > usability thing I fellt over some time ago :) > > > > > > Best regards, Marcus > > > > Yea, actually I thought the same thing myself :-). Thanks though, and > update patch attached. > > Samuel Cormier-Iijima > > > _______________________________________________ tracker-list mailing list tracker-list@gnome.org http://mail.gnome.org/mailman/listinfo/tracker-list
