This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: D language support


Tom Tromey, el 23 de abril a las 12:09 me escribiste:
> >>>>> "Joel" == Joel Brobecker <brobecker@adacore.com> writes:
> 
> Joel> Although making the variable static limits the damage, I pretty strongly
> Joel> disagree that having this inter-routine global makes the code more
> Joel> readable. But, as I said, I don't see this as a blocking problem because
> Joel> it's localized to an area that I hope will require little attention from
> Joel> anyone but you; so I am happy to let it go provided that another Global
> Joel> Maintainer says it's OK - I'll try to remember to ask Tom for his take
> Joel> on it, if no one else provides an opinion...
> 
> I took another look at it.
> 
> I agree with Joel.  In this case, the variables in question could easily
> be replaced with function arguments.
> 
> I realize this means adding a bunch of "*"s all over the place, which is
> kind of ugly.  But the flip side is that globals make functions more
> difficult to understand in isolation.
> 
> I also realize that this sort of thing appears elsewhere.  I'd rather we
> not emulate bad decisions in new code, though.
> 
> I saw a couple more style nits too, more in the next note.

Is the indent program invoked with -gnu option enough to fix the style
nits or GDB has its own rules? It might save some time to everybody =)

Thanks to all the people involved in the patch review process BTW.

-- 
Leandro Lucarella (AKA luca)                     http://llucax.com.ar/
----------------------------------------------------------------------
GPG Key: 5F5A8D05 (F8CD F9A7 BF00 5431 4145  104C 949E BFB6 5F5A 8D05)
----------------------------------------------------------------------
El amor es como una reina ortopÃdica.
	-- Poroto


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]