----- Forwarded message from Dan Carpenter <dan.carpenter at oracle.com> -----
> Date: Thu, 9 May 2013 21:55:26 +0300
> From: Dan Carpenter <dan.carpenter at oracle.com>
> To: William Hubbs <w.d.hubbs at gmail.com>
> Cc: gregkh at suse.de, devel at driverdev.osuosl.org
> Subject: Re: speakup issues
>
> It's like every function in speakup has something that's just a
> little bit ugly about it.
>
> For example
> 1) No spaces around operations. IE "if (x&4) {" vs "if (x & 4) {".
> 2) No blank lines between the variable declarations and the code.
> 3) The 'E' in E_DEFAULT stands for "enum" which is bogus.
> 4) Custom macros like spk_lock()
> 5) Using u_char instead of u8. (Yes, there are thousands of places
> in the kernel which do this but it's mostly horrid legacy code
> like ISDN).
> 6) In synth_direct_store() the last 5 bytes of the tmp[] buf are not
> used.
> 7) Unnecessary casts.
> 8) Things like this:
>
> char *cp;
> u_char *cp1;
>
> "cp1" is a poorly chosen variable name.
>
> Anyway, there is a lot of stuff. It just needs someone to go
> through it bit by bit and re-write it cleaner.
>
> regards,
> dan carpenter
>
----- End forwarded message -----
