Re: [Nautilus-list] [PATCH] Make EelTextCaption behave properly inpreferences
- From: Darin Adler <darin bentspoon com>
- To: migras atlas uvigo es
- Cc: Lista de nautilus <nautilus-list lists eazel com>
- Subject: Re: [Nautilus-list] [PATCH] Make EelTextCaption behave properly inpreferences
- Date: Sat, 19 May 2001 17:29:17 -0700
On Saturday, May 19, 2001, at 04:47 PM, Miguel Rodríguez Pérez wrote:
Please ignore my previous patch, it works fine, but it can be done much
simpler...
The concept of this patch is good.
To match Nautilus style guidelines, it needs to do "strcmp (xxx, yyy) == 0"
rather than "!strcmp(xxx, yyy)". Note both the space after the name of
the function before the parenthesis, and use of == 0 with strcmp, not !.
Also, the single statement inside the if needs to be surrounded by braces
as described in the style guide document (nautilus/doc/style-guide.html).
If you fix these minor style issues, it would be fine to commit this
change.
-- Darin
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]