Quoting Dylan Baker (2017-03-20 09:51:58) > Quoting Emil Velikov (2017-03-20 08:11:58) > > On 8 March 2017 at 18:11, Emil Velikov <emil.l.veli...@gmail.com> wrote: > > > On 7 March 2017 at 21:14, Dylan Baker <dy...@pnwbakers.com> wrote: > > >> There are a number of small style cleanups and simplifications in this > > >> series, > > >> but the main changes are: > > >> - use a mako template to generate the header and code rather than prints > > >> - be python 3.x ready (the goal isn't to write python 3 code, but to > > >> write code > > >> that is easy to port or hybridize) > > >> - generate the header and the code in one go > > >> > > >> I've put emphasis on the readability of the template rather than the > > >> readability > > >> of the output code, it's relatively easy to pipe the code through > > >> 'indent' to > > >> make it more readable. > > >> > > >> This is available at my github: > > >> https://github.com/dcbaker/temp submit/anv_entrypoints-cleanup > > >> > > >> Notable changes in Version 2: > > >> - Pass XML file via an argument > > >> - add flag to control output directory > > >> - Attempt to update android makefiles > > >> > > >> Notable changes in Version 3: > > >> - Fix "do not edit" message to have proper file name > > >> - More Android.mk changes > > >> - Don't write both files at the same time > > >> - Provide the file name to be written to as an argument > > >> > > >> Notable changes in Version 4: > > >> - Do write both files at the same time, in a separate patch > > >> - Hopefully fix the build systems for real > > >> > > > Seems to work on my end. Since I'm not a python expert I'll only got for > > > Tested-by: Emil Velikov <emil.veli...@collabora.com> > > > > > > Note that this will clash with the compile move series and I've > > > perfectly fine with fixing either one - depending on which lands > > > first. > > > > > Dylan, is there anything blocking these patches ? As mentioned above > > there will be some trivial fixes needed, but I can fix them + push if > > you/others are happy with the lot. > > > > Thanks > > Emil > > I don't think so at this point. Linkmauve had indicated that he had comments, > but at this point I think it's fair to tell him to send patches if he doesn't > like something. I had planned to push Friday but my son was sick so I was home > with him. If you wanted to fix up the build system and push you can do that, > or > I can push them and you can follow up with build system changes, whichever > works > better for you. > > Dylan
ping. I'll push this in 14-16 hours unless you tell me otherwise. Dylan
signature.asc
Description: signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev