[PATCH]fs/jffs2/wbuf.c: add compatibility support for OOB data block

Artem B. Bityuckiy dedekind at yandex.ru
Fri Jul 22 08:12:56 EDT 2005


Jörn Engel wrote:
> 
> Sevenfold indentation is not what I call good coding style.  Try to
> make this stuff simpler.  Use "if (!...) continue" and similar
> constructs.
> 
Agree, although it is how JFFS2 is written. Let's not worsen it, but 
instead, make it tidier. Please, if you don't mind, reformat the whole 
function, you may split it on shorter sub-functions, etc.

BTW, IMO the Summary patch of Ferenc Havasi is not yet in CVS (even 
though it greatly helps with mount time) because he didn't bothered 
making it cute. He just took messy JFFS2 code, and made it even messier. 
I'd be glad to see Ferenc's patch nice and in CVS.

-- 
Best Regards,
Artem B. Bityuckiy,
St.-Petersburg, Russia.





More information about the linux-mtd mailing list