latchset / libverto

Offical repository for libverto, an async event loop abstraction library
MIT License
16 stars 6 forks source link

Non-prototype declarations #4

Closed frozencemetery closed 13 years ago

frozencemetery commented 13 years ago

Issue migrated from trac ticket # 3

component: component1 | priority: major | resolution: fixed

2011-09-07 13:52:39: @greghudson created the issue


libverto's headers contain a number of function declarations which aren't prototypes because they use () instead of (void). In definitions this is okay, but in prototypes this causes the compiler to treat the declaration as a K&R-style declaration and not a prototype.

verto-glib.h:verto_ctx *verto_new_glib();
verto-glib.h:verto_ctx *verto_default_glib();
verto-libevent.h:verto_ctx *verto_new_libevent();
verto-libevent.h:verto_ctx *verto_default_libevent();
verto-libev.h:verto_ctx *verto_new_libev();
verto-libev.h:verto_ctx *verto_default_libev();
verto-module.h:typedef verto_ctx *(*verto_ctx_constructor)();
verto-tevent.h:verto_ctx *verto_new_tevent();
verto-tevent.h:verto_ctx *verto_default_tevent();
frozencemetery commented 13 years ago

2011-09-26 10:10:10: @npmccallum changed status from new to closed

frozencemetery commented 13 years ago

2011-09-26 10:10:10: @npmccallum changed resolution from * to fixed*

frozencemetery commented 13 years ago

2011-09-26 10:10:10: @npmccallum commented


Fixed in f6dda6fd28994dc602423a3af4a9226c476782e7.