On Thu, Oct 25, 2018 at 2:09 PM Ramsay Jones
<ramsay@xxxxxxxxxxxxxxxxxxxx> wrote:
> Yes, this will 'fix' the 'commit-reach.h' header (not surprising),
> but I prefer my patch. ;-)
I apologize, I joined the list recently and so might had missed a
reroll; the merged series in pu doesn't seem to include it and the
error was around the code I changed, so wanted to make sure it would
be addressed sooner rather than later.
eitherway, I agree with you my patch (or something better) would fit
better in your topic branch than on mine and while I haven't seen your
patch I am sure is most likely better.
> Still puzzled.
this are the last lines of a `make hdr-check` in Fedora Rawhide, it
should behave the same regardless of OS or compiler used IMHO
HDR commit-reach.h
commit-reach.h:45:28: warning: ‘struct object_id’ declared inside
parameter list will not be visible outside of this definition or
declaration
int ref_newer(const struct object_id *new_oid, const struct object_id
*old_oid);
^~~~~~~~~
In file included from commit-slab.h:5,
from commit-reach.h:4:
commit-reach.h: In function ‘contains_cache_at_peek’:
commit-slab-impl.h:47:14: error: dereferencing pointer to incomplete
type ‘const struct commit’
nth_slab = c->index / s->slab_size; \
^~
commit-slab-impl.h:7:2: note: in expansion of macro ‘implement_commit_slab’
implement_commit_slab(slabname, elemtype, static MAYBE_UNUSED)
^~~~~~~~~~~~~~~~~~~~~
commit-slab.h:49:2: note: in expansion of macro ‘implement_static_commit_slab’
implement_static_commit_slab(slabname, elemtype)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
commit-reach.h:57:1: note: in expansion of macro ‘define_commit_slab’
define_commit_slab(contains_cache, enum contains_result);
^~~~~~~~~~~~~~~~~~
commit-reach.h: At top level:
commit-reach.h:69:41: warning: ‘struct object_array’ declared inside
parameter list will not be visible outside of this definition or
declaration
int can_all_from_reach_with_flag(struct object_array *from,
^~~~~~~~~~~~
make: *** [Makefile:2685: commit-reach.hco] Error 1
Carlo
