login
Header Space

 
 

Coding Style question - alignment of variables

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <kernel-mentors@...>
Date: Thursday, January 26, 2006 - 5:14 pm

Hello all,

I'm reviewing a patch and have the following CodingStyle question.

The question is, which of these two is the correct method for indentation:

	result = foo(some_long_variable_name_foo_bar_1, foo2, foo3,
		     foo4, foo5);

or

	result = foo(some_long_variable_name_foo_bar_1,
		     foo2, foo3, foo4, foo5)

?

I believe the first example is correct -- but I'm not 100% sure.

Thanks,

P.
_______________________________________________
Kernel-mentors mailing list
Kernel-mentors@selenic.com
http://selenic.com/mailman/listinfo/kernel-mentors
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Hi - Need a mentor, Dheeraj Kandula, (Wed Jul 11, 5:22 pm)
Coding Style question - alignment of variables, Prarit Bhargava, (Thu Jan 26, 5:14 pm)
RFC: SPI patch, Gary Jennejohn, (Thu Jan 10, 9:46 am)
driver for review, Bahadir Balban, (Thu Apr 13, 12:45 pm)
Diffenrence Completion and Semaphore, minchan Kim, (Thu May 18, 9:47 pm)
I need a mentor!, Joe moo, (Sun Aug 24, 5:18 pm)
HELP : taking up a kernel project, vibi sreenivasan, (Mon Jul 14, 9:16 am)
enthusiast looking for a mentor, Shobhit Jindal, (Thu Aug 24, 10:53 am)
jim.cromie has invited you to Spokeo, , (Tue Mar 25, 8:07 am)
Dynsched - dynamic cpu scheduler selection, Felix Obenhuber, (Sat Nov 18, 11:29 am)
Filesystem compression, kishore, (Sun Dec 10, 10:26 am)
swap: which is the maximum size allowed?, Eriberto, (Sun Jan 28, 4:07 pm)
(no subject), chandrashekhar, (Wed Feb 21, 6:28 am)
How can I join an active project?, Jiancong Xie, (Wed Apr 18, 4:40 am)
Build and test system for multiple kernels, François-Frédéric Ozog..., (Thu Nov 22, 4:06 am)
Re: [RFC]__sock_create() change, pradeep singh, (Mon May 14, 4:12 am)
[PATCH] char: Nano 7240 watchdog driver, Gilles GIGAN, (Fri Oct 5, 11:10 pm)
Re: I need a mentor!, Ahmed S. Darwish, (Mon Aug 25, 9:09 am)
Re: HELP : taking up a kernel project, Ahmed S. Darwish, (Tue Jul 15, 8:32 am)
Re: Build and test system for multiple kernels, Stjepan Gros, (Thu Nov 22, 9:41 am)
Re: [PATCH] char: Nano 7240 watchdog driver, Dave Jones, (Sat Oct 6, 12:02 am)
Re: [PATCH] char: Nano 7240 watchdog driver, Randy Dunlap, (Sat Oct 6, 1:27 am)
Re: [PATCH] char: Nano 7240 watchdog driver, Arjan van de Ven, (Sat Oct 6, 4:24 pm)
Re: Hi - Need a mentor, Randy Dunlap, (Wed Jul 11, 9:55 pm)
Re: [RFC]__sock_create() change, Arjan van de Ven, (Mon May 14, 11:10 am)
Re: [RFC]__sock_create() change, pradeep singh, (Tue May 15, 1:09 am)
Re: [RFC]__sock_create() change, Chris Wedgwood, (Mon May 14, 6:51 pm)
Re: How can I join an active project?, Arjan van de Ven, (Fri Apr 20, 2:20 pm)
Re: How can I join an active project?, Felix Obenhuber, (Wed Apr 18, 12:41 pm)
Re: (no subject), Randy Dunlap, (Wed Feb 21, 4:56 pm)
Re: (no subject), Greg KH, (Wed Feb 21, 4:45 pm)
Re: swap: which is the maximum size allowed?, Tony Clayton, (Mon Jan 29, 3:24 am)
Re: swap: which is the maximum size allowed?, Arjan van de Ven, (Wed Jan 31, 6:38 am)
Re: Filesystem compression, Phillip Lougher, (Mon Dec 11, 12:09 am)
Re: enthusiast looking for a mentor, Jim Cromie, (Thu Aug 24, 1:05 pm)
Re: enthusiast looking for a mentor, Vivek Joshi, (Fri Aug 25, 2:13 am)
Re: Expertise required on building code for SMP, Arjan van de Ven, (Thu Jul 13, 3:45 am)
Re: Diffenrence Completion and Semaphore, Zach Brown, (Thu May 18, 10:32 pm)
Re: Diffenrence Completion and Semaphore, Arjan van de Ven, (Thu May 18, 10:02 pm)
Re: Diffenrence Completion and Semaphore, minchan Kim, (Thu May 18, 11:41 pm)
Re: Diffenrence Completion and Semaphore, Arjan van de Ven, (Fri May 19, 8:52 am)
Re: Diffenrence Completion and Semaphore, minchan Kim, (Fri May 19, 9:52 am)
Re: driver for review, Peter Korsgaard, (Fri Apr 14, 2:07 pm)
RE: driver for review, Dustin McIntire, (Mon Apr 17, 9:31 pm)
Re: driver for review, Peter Korsgaard, (Sun Apr 23, 2:54 pm)
RE: driver for review, Dustin McIntire, (Sun Apr 23, 3:35 pm)
Re: driver for review, Bahadir Balban, (Tue Apr 18, 11:17 am)
RE: driver for review, Dustin McIntire, (Wed Apr 19, 1:12 am)
Re: driver for review, Matt Mackall, (Thu Apr 13, 4:20 pm)
Re: driver for review, Bahadir Balban, (Tue Apr 18, 11:01 am)
Re: Coding Style question - alignment of variables, Prarit Bhargava, (Thu Jan 26, 5:28 pm)
Re: Coding Style question - alignment of variables, Jim Cromie, (Thu Jan 26, 7:06 pm)
Re: Coding Style question - alignment of variables, Muli Ben-Yehuda, (Thu Jan 26, 7:11 pm)
speck-geostationary