This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC 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: Improving gengtype (for plugin support notably) - how to get a rather big patch accepted?


2010/8/24 Basile Starynkevitch <basile@starynkevitch.net>:
> Hello All
>
> Jeremie Salvucci and me Basile are working on improving gengtype. Our
> patch is still buggy [curious people might retrieve it from
> http://starynkevitch.net/Basile/gengtype-r163335-24august-2010.diff

I was meaning to take a look to get an idea what the patch is doing,
but it arbitrarily changes indentation in otherwise-untouched places.
Not that it matters now, as you are not submitting it, but I believe
it is easier to always work within code formatting rules than to
properly format only before submission.

Other than that, I can only agree with what Ian advised for you.

-- 
Laurynas


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