Message ID | 20140709070106.GA18813@spoyarek.pnq.redhat.com |
---|---|
State | New |
Headers | show |
Siddhesh Poyarekar <siddhesh@redhat.com> writes: > diff --git a/locale/programs/simple-hash.c b/locale/programs/simple-hash.c > index ef371a0..8d087da 100644 > --- a/locale/programs/simple-hash.c > +++ b/locale/programs/simple-hash.c > @@ -27,11 +27,7 @@ > #include <stdint.h> > #include <sys/types.h> > > -#if HAVE_OBSTACK > # include <obstack.h> Please remove the indentation. Ok with that change. Andreas.
On 9 July 2014 08:01, Siddhesh Poyarekar <siddhesh@redhat.com> wrote: > Remove the HAVE_OBSTACK macro check and include obstack check in > include path order since we don't have a copy of obstack.h in the > current directory. > > Siddhesh > > * locale/programs/simple-hash.c [!HAVE_OBSTACK]: Remove code. > > --- > locale/programs/simple-hash.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/locale/programs/simple-hash.c b/locale/programs/simple-hash.c > index ef371a0..8d087da 100644 > --- a/locale/programs/simple-hash.c > +++ b/locale/programs/simple-hash.c > @@ -27,11 +27,7 @@ > #include <stdint.h> > #include <sys/types.h> > > -#if HAVE_OBSTACK > # include <obstack.h> > -#else > -# include "obstack.h" > -#endif > > #ifdef HAVE_VALUES_H > # include <values.h> I wonder if we could go further than this - it looks like we should be able to remove HAVE_CONFIG_H and HAVE_VALUES_H too. Also the define of bcopy seems redundant. And the definition of BITSPERBYTE. Maybe we can remove the include of values.h too? Your change looks ok in itself though (with fixed indentation).
Will Newton <will.newton@linaro.org> writes: > I wonder if we could go further than this - it looks like we should be > able to remove HAVE_CONFIG_H and HAVE_VALUES_H too. > > Also the define of bcopy seems redundant. > > And the definition of BITSPERBYTE. Maybe we can remove the include of > values.h too? I think the original intent was that the programs can also be built standalone. Andreas.
diff --git a/locale/programs/simple-hash.c b/locale/programs/simple-hash.c index ef371a0..8d087da 100644 --- a/locale/programs/simple-hash.c +++ b/locale/programs/simple-hash.c @@ -27,11 +27,7 @@ #include <stdint.h> #include <sys/types.h> -#if HAVE_OBSTACK # include <obstack.h> -#else -# include "obstack.h" -#endif #ifdef HAVE_VALUES_H # include <values.h>