[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 3/5] xen: arm: head.S PT_DEV is unused, drop and rename PT_DEV_L3



On Wed, 2012-12-19 at 10:49 +0000, Tim Deegan wrote:
> At 10:20 +0000 on 19 Dec (1355912423), Ian Campbell wrote:
> > On Tue, 2012-12-18 at 18:33 +0000, Stefano Stabellini wrote:
> > > On Tue, 18 Dec 2012, Ian Campbell wrote:
> > > > This shortens an overly long line.
> > > > 
> > > > Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > > 
> > > honestly I would rather keep it because it has been quite useful for
> > > debugging in the past once all the bugs have been fixed (TM) then we can
> > > remove it ;-)
> > 
> > Can you not just re-add it for debug?
> > 
> > I mostly just want to get rid of the overlong line, I could nuke the
> > spaces from the comment (in all of them, not just this one) instead?
> 
> Could you just remove the 'lev3: ' from the comment, pulling it in to
> exactly 80 chars?  Your' added 'second level' and 'third level' make it
> redundant, and I'd rather not lose the spaces in the comments.

I think that makes it exactly 80 characters, which is probably ok.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.