Hi Johannes,
Johannes Stezenbach wrote:
>Hi,
>
>Doug Scoular wrote:
>
>
>> Just one more question... does he create the patch against
>> the latest stable linux-2.6.12.1 source or does he get
>> the latest dvb-kernel from CVS and create the patch
>> against some tag there (if so, what tag... er... HEAD) ?
>>
>>
>
>Preferably the patch should be against dvb-kernel CVS (no tag, i.e.
>HEAD), however as long as the patch still applies it doesn't
>matter that much what version you created it against. So if
>you happen to have an old patch lying around you can just
>send that, and if it doesn't apply we'll tell you to recreate
>it, thus you can avoid doing unnecessary work.
>
>
>
Great! that makes sense... thanks.
>Please make sure the patch does not introduce trailing
>whitespace and uses proper indentation (one tab, no spaces).
>
>
I'll try to ensure no trailing whitespace and use the following elisp:
(defun linux-c-mode ()
"C mode with adjusted defaults for use with the Linux kernel."
(interactive)
(c-mode)
(setq c-indent-level 8)
(setq c-brace-imaginary-offset 0)
(setq c-brace-offset -8)
(setq c-argdecl-indent 8)
(setq c-label-offset -8)
(setq c-continued-statement-offset 8)
(setq indent-tabs-mode nil)
(setq tab-width 8))
As per the Linux coding style guide so the code I add has no
actual tabs but sets of 8 spaces for "tabbing".
http://pantransit.reptiles.org/prog/CodingStyle.html
As per the now famous http://www.jwz.org/doc/tabs-vs-spaces.html.
Is this okay ?
Cheers,
Doug
