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

Re: [E-devel] [ENTRANCE][PATCH] entrance_edit2

On Fri, Jul 21, 2006 at 07:01:01PM +0100, Essien Ita Essien wrote:
> >>I know I said before that I'd take a look at them, and I know I did and
> >>they looked good for the most part. Somehow I got distracted and never
> >>got them committed. I remember thinking the two functions for getting
> >>the geometries could be combined into a single one (since they were
> >>almost duplicate, and were called in succession iirc).
> >>  
> >>    
> >Yeah, that was my original intent, but i thought it an overkill to 
> >return a geometry struct, or returning values via passed in pointers 
> >(doesn't feel too natural) or even a two int array (like pipe() does). 
> >  
> >>rephorm
> >>    
> okie dokie. this modified patch follows rephorms suggestion, so the 
> geometry parsing is now:


Looks good. See the commit logs for a small note (not your fault).
Good stuff :)