Re: [uml-devel] [PATCH 6/6] UML - Fix hostfs style

2007-08-24 Thread Blaisorblade
On venerdì 24 agosto 2007, Jeff Dike wrote: > On Thu, Aug 23, 2007 at 04:54:59PM +0200, Blaisorblade wrote: > > > actually. Personally I'd prefer: > > > > > > else > > > type = OS_TYPE_DIR; > > > > I strongly agree with this style; beyond style itself, one strong reason > > is that join

Re: [uml-devel] [PATCH 6/6] UML - Fix hostfs style

2007-08-24 Thread Jeff Dike
On Thu, Aug 23, 2007 at 04:54:59PM +0200, Blaisorblade wrote: > > actually. Personally I'd prefer: > > > > else > > type = OS_TYPE_DIR; > > I strongly agree with this style; beyond style itself, one strong reason is > that joining statements hinder singlestepping through function

Re: [uml-devel] [PATCH 6/6] UML - Fix hostfs style

2007-08-23 Thread Blaisorblade
On sabato 18 agosto 2007, Satyam Sharma wrote: > On Fri, 17 Aug 2007, Jeff Dike wrote: > > Style fixes in hostfs. > > @@ -328,17 +326,17 @@ int hostfs_readdir(struct file *file, vo > > [...] > > - if(error) break; > > + if (error) break; > > if (error) >