Re: [PATCH] USB: SisUSB2VGA: Lindent drivers/usb/misc/sisusbvga/sisusb_struct.h
David Brownell <[email protected]>
| Newsgroups | gmane.linux.usb.devel |
|---|---|
| Message-ID | <[email protected]> |
On Friday 10 August 2007, Felipe Balbi wrote: >Better indentation I don't think so. This whole patch seems to make one type of change: > - unsigned char St_ModeID; > - unsigned short St_ModeFlag; > - unsigned char St_StTableIndex; > - unsigned char St_CRT2CRTC; > - unsigned char St_ResInfo; > - unsigned char VB_StTVFlickerIndex; > - unsigned char VB_StTVEdgeIndex; > - unsigned char VB_StTVYFilterIndex; > - unsigned char St_PDC; ... all neatly indented so the fields line up. TheProblemISeeWithThatCodeIsThatItUsesCamelCase. > + unsigned char St_ModeID; > + unsigned short St_ModeFlag; > + unsigned char St_StTableIndex; > + unsigned char St_CRT2CRTC; > + unsigned char St_ResInfo; > + unsigned char VB_StTVFlickerIndex; > + unsigned char VB_StTVEdgeIndex; > + unsigned char VB_StTVYFilterIndex; > + unsigned char St_PDC; ... the fields stopped lining up. In that particular stretch it's not all that much worse, although it does seem harder to find the member names. However, overall I couldn't agree this is an improvement. ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel