On Thu, Oct 01, 2015 at 10:23:42AM +0200, Thierry Reding wrote: > On Thu, Oct 01, 2015 at 10:10:43AM +0200, Thierry Reding wrote: > > On Wed, Sep 30, 2015 at 01:31:32PM +1000, David Gibson wrote: > > > On Tue, Sep 29, 2015 at 11:09:05AM +0200, Thierry Reding wrote: > > > > From: Thierry Reding <treding@xxxxxxxxxx> > > > > > > > > These three patches add a couple of string functions that have proven > > > > useful in U-Boot's copy of libfdt, so they are likely to be useful for > > > > other users as well. > > > > > > > > Patch 1 adds a function to count the number of strings in a property's > > > > value. This also adds a new DTS sample along with a small test program > > > > to validate the implemented functions. > > > > > > > > Patch 2 adds a function to retrieve the index of a given string in any > > > > given property's value. This adds code to the test program introduced in > > > > the previous patch to exercise the new functionality. > > > > > > > > Patch 3 adds a function to retrieve a string by index from a property's > > > > value. This extends the test program introduced in patch 1 to validate > > > > the new functionality. > > > > > > Thanks, I've applied this. > > > > > > I did end up making a couple of small changes. > > > > > > First, I forgot to say earlier that I also wanted to change the > > > testcase name to stringlist* for clarity. > > > > Okay, makes sense. > > > > > Second, Travis CI picked up some -Wshadow warnings: in a bunch of > > > places you were using 'index' as a variable name, which shadows the > > > libc index(3) function. I'm not sure why that didn't show up on a > > > local "make check", but in any case I've changed thos to 'idx' to fix > > > it. > > > > As far as I can tell that's because index(3) is declared in strings.h on > > my system (the file ships with the GNU libc 2.22) and that header file > > isn't included from anywhere. Perhaps your system differs from that? > > > > Actually there's also a prototype for index(3) in string.h on my system > > but it is guarded with an #ifdef __USE_MISC, and upon closer inspection > > that does get set, so the index(3) symbol should be available. No idea > > why I don't get a warning, though, -Wshadow is definitely getting passed > > to the compiler. > > Ah, some further digging shows that this behaviour was changed in GCC > 4.8 (search for -Wshadow): Oops, there should have been a link here: https://gcc.gnu.org/gcc-4.8/changes.html Thierry
Attachment:
signature.asc
Description: PGP signature
