Menu

#86 Compilation of 1.4 branch broken since r1368

For_1.4
closed-fixed
nobody
None
5
2009-07-23
2009-07-22
Eric Petit
No

See log below, r1368 introduced some use of ev_int32_t or ev_int64_t (through EV_INTPTR_T), but those are not defined anywhere - evutil.h only defines the unsigned versions (ev_uint32_t, etc).

gcc -DHAVE_CONFIG_H -I. -I.. -I.. -I../compat -g -O2 -Wall -fno-strict-aliasing -c bench.c
bench.c: In function ‘read_cb’:
bench.c:75: error: ‘ev_int32_t’ undeclared (first use in this function)
bench.c:75: error: (Each undeclared identifier is reported only once
bench.c:75: error: for each function it appears in.)
bench.c:75: error: expected ‘;’ before ‘idx’
bench.c:80: error: ‘widx’ undeclared (first use in this function)
bench.c: In function ‘run_once’:
bench.c:92: error: ‘ev_int32_t’ undeclared (first use in this function)
bench.c:92: error: expected ‘;’ before ‘i’
bench.c:95: error: ‘i’ undeclared (first use in this function)
bench.c:95: warning: left-hand operand of comma expression has no effect
bench.c:106: warning: left-hand operand of comma expression has no effect
make[5]: *** [bench.o] Error 1

Discussion

  • Nick Mathewson

    Nick Mathewson - 2009-07-23

    Oops; I had a commit that I forgot to push. How is r1374 for you?

     
  • Eric Petit

    Eric Petit - 2009-07-23
    • status: open --> closed-fixed
     
  • Eric Petit

    Eric Petit - 2009-07-23

    It is fixed, thank you!

     
  • Nick Mathewson

    Nick Mathewson - 2009-07-23

    You're welcome, and thank you for the bug report.

    Closing.

     

Log in to post a comment.