[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug #44238] Add support for menu.c32 and vesamenu.c32 in lib/syslin
|
From: |
Andrei Borzenkov |
|
Subject: |
Re: [bug #44238] Add support for menu.c32 and vesamenu.c32 in lib/syslinux_parse |
|
Date: |
Sat, 14 Feb 2015 20:48:14 +0300 |
В Sat, 14 Feb 2015 09:22:19 +0100
Lunar <address@hidden> пишет:
> Andrei Borzenkov:
> > В Fri, 13 Feb 2015 22:09:04 +0100
> > Lunar <address@hidden> пишет:
> >
> > > Andrei Borzenkov:
> > > > [… review and comments …]
> > >
> > > Ok, here's another try.
> > >
> >
> > It looks OK but as someone reported that your patches seem to cause
> > problems elsewhere let's wait for confirmation.
> >
> > @gluglug - could you test both update versions? Thank you.
>
> fchmmr said that there was an issue with Trisquel 7.0 GNOME. But if
> there's one, it doesn't seem to be related to the patches.
>
Well, one of your patches had double free, it may lead to unexpected
results. It would still be helpful to confirm that problem went away.
BTW if you could write test for syslinux parser it would be great!
pgpEqRFmvd5Bc.pgp
Description: OpenPGP digital signature