lkml.org 
[lkml]   [2007]   [Dec]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] SH/Dreamcast - add support for GD-Rom device
Date
On Thursday 27 December 2007, Joe Perches wrote:
> On Thu, 2007-12-27 at 16:52 +0000, Adrian McMenamin wrote:
> > This patch adds support for the CD-Rom drive on the SEGA Dreamcast.
>
> Because it was already so close, might as well make it checkpatch clean.

for the 80 col limit, that's up to the author. if Adrian is happy with it as
is, that's fine.

> I also added a function gdrom_is_busy() to make a couple of tests
> fit on a single line and perhaps easier to read.

i'd tend to agree it does make it easier to read, however you didnt make it
inline which means gcc may not inline it which means there's pointless
overhead of doing a function call.

otherwise, arbitrary breaking on 80 cols actually makes the code harder to
read for no gain.
-mike
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2007-12-28 01:53    [W:0.462 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site