From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Feustel Subject: Re: USB virt 2.6 split driver patch series Date: Mon, 21 Nov 2005 11:30:38 -0500 Message-ID: <200511211130.39217.dfeustel@verizon.net> References: <1132579104.31295.110.camel@localhost.localdomain> <200511211024.11770.dfeustel@verizon.net> <20051121160316.GA25718@snarc.org> Reply-To: dfeustel@mindspring.com Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <20051121160316.GA25718@snarc.org> Content-disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Vincent Hanquez Cc: harry , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Monday 21 November 2005 11:03, Vincent Hanquez wrote: > On Mon, Nov 21, 2005 at 10:24:11AM -0500, Dave Feustel wrote: > > Thanks for posting the example with explanation. > > What about this example bothers you and makes > > you think this use of brackets should not be permitted > > (if in fact this is your position)? > > [ oleg's answer is pretty much what I think ] > > but yeah, my main reason would be that it deepens the code nesting > unnecessarily; that's really bad that the left half (or more) of the 80 > columns is blank. > > you end up doing code in the [50:80] range, > > like: > big_name_of_function(do_something, > 1 + 1542, > something_else); > instead of: > > big_name_of_function(do_something, 1 + 1542, something_else); I hadn't thought of that, but I have run into exactly the kind of code that you describe. It's definitely a PITA to work with. -- Switch to Secure OpenBSD with a KDE desktop!!! NOW with Virtual PC OS support via QEMU and Beowulf clustering using PETSc and MPICH2!