Rendering of README.md inline with inner tree view dirs

John Keeping john at keeping.me.uk
Tue Jun 12 14:07:07 CEST 2018


On Tue, Jun 12, 2018 at 05:27:09PM +0800, Andy Green wrote:
> 
> 
> On June 12, 2018 5:24:37 PM GMT+08:00, John Keeping <john at keeping.me.uk> wrote:
> >On Tue, Jun 12, 2018 at 10:35:25AM +0200, Christian Hesse wrote:
> >> Andy Green <andy at warmcat.com> on Tue, 2018/06/12 13:53:
> >> > 5) I get some gcc 8.1 warnings, I made a couple of patches to get
> >around 
> >> > them.  In a couple of places, the code seemed legit really.
> >> 
> >> I do see these on Arch as well. :)
> >> Did not find the time to address them.
> >> 
> >> > [inline patches]
> >> > 
> >> > I should include these or the slightly dubious warnings are just 
> >> > regarded as an annoyance?
> >> 
> >> Definitely to be addressed. I pushed your changes here:
> >> 
> >> https://git.zx2c4.com/cgit/commit/?h=ch/gcc-8&id=e7ac824f
> >> 
> >> Builds without warnings and tests are happy, but I did not yet take a
> >> detailed look.
> >
> >We'll need those posted as proper patches with a sign-off though.  It
> >will also be easier to review them that way.
> 
> No worries I'll send a series with everything including your rebased
> patches when the remaining things are sorted.  I was checking if
> anyone cared about the warnings.

If the GCC-8 fixes are independent of the other changes (from a quick
skim I think they are), then please send them as a separate series so
that we can get the warnings fixes in without waiting for other changes.


John


More information about the CGit mailing list