Re: 2.6.13-rc2-mm1: some speakup nitpicks

[Alexey Dobriyan]

On Saturday 09 July 2005 06:07, Adrian Bunk wrote:
> - SPEAKUP_DEFAULT shouldn't be asked if SPEAKUP=n
> - "make namespacecheck" shows tons of needlessly global code
> - the static variable special_handler is EXPORT_SYMBOL'ed
> - #define MIN should be removed
> - the file cvsversion.h only for keeping a CVS date is a bit
>   overkill
> - spk_con_module.h is not exactly how we use header files in the kernel
> - many of the #ifdef MODULE's point to things that could be done better
>   (especially the #include "mod_code.c"'s)
> - the things in synthlist.h could be done less ugly
> - speakupconf is a userspace script that belongs under Documentation/
> - dtload.c is not kernel code, and should therefore not be in that
>   directory
> - the code should follow Documentation/CodingStyle better
>   (no spaces between the braces and function arguments)
> - building speakup_keyhelp.c modular even in a kernel that doesn't
>   support modules is silly
> - #include <linux/...> belongs before #include <asm/...>

- Plenty, plenty of sparse warnings.
-	#define PROC_READ_PROTOTYPE char *page, char **start, off_t off, \
				int count, int *eof, void *data
	#define PROC_WRITE_PROTOTYPE struct file *file, const char *buffer, \
				u_long count, void *data
-	#define CAP_A 'A'
	#define CAP_Z 'Z'

  and other fun in drivers/char/speakup/spk_priv.h

- LINUX_VERSION_CODE ifdeffery
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/