On Wed, 4 Jul 2012 at 21:05:18 +0200, Carlos R. Mafra wrote: > On Tue, 3 Jul 2012 at 19:34:38 +0200, Rodolfo García Peñas wrote: > > Subject: [PATCH 04/13] wDefaultGetImage splitted > > > > -static void wApplicationExtractDirPackIcon(WScreen * scr, char *path, char > > *wm_instance, char *wm_class) > > +void wApplicationExtractDirPackIcon(WScreen * scr, char *path, char > > *wm_instance, char *wm_class) > > You could take the opportunity to write WScreen *scr instead of having > the space... > > > > +/* Get the file name of the image, using instance and class */ > > +char *get_default_icon_filename(WScreen *scr, char *winstance, char > > *wclass, char *command, > > + Bool noDefault) > > The comment should include a description of the noDefault parameter. > Btw, it's a bad idea to have a boolean parameter which is a negation > of something. I find it more natural to have > > Default = True > > instead of > > noDefault = False > > when you want the Default case.
I will apply it anyway though. Those thoughts just passed on my mind, but they are not fundamental. -- To unsubscribe, send mail to [email protected].
