Implement blueprint 'consistent-header-guards'.
Implement https://blueprints.launchpad.net/tarantool/+spec/consistent-header-guards Audit all header files in tarantool code base for consistent use of header guards: - consistent naming scheme, following doc/coding-style.txt - header guard goes before the copyright - each header is self-sufficient. - add copyright headers where they were missing. Remove __packed__, __unused__, __noinline__ and __regparm2__ from util.h, since the name __unused__ conflicted with a parameter name in third_party/valgrind/valrgind.h, which led to a dependency in inclusion order between util.h and this header. The define names were not standard, I couldn't come up with some meaningful and standard names, and they were only saving some typing, while not providing any extra functionality.
Showing
- core/admin.c 1 addition, 1 deletioncore/admin.c
- core/admin.rl 1 addition, 1 deletioncore/admin.rl
- core/coro.c 2 additions, 2 deletionscore/coro.c
- core/diagnostics.c 1 addition, 0 deletionscore/diagnostics.c
- core/fiber.c 6 additions, 6 deletionscore/fiber.c
- core/iproto.c 1 addition, 1 deletioncore/iproto.c
- core/log_io.c 4 additions, 7 deletionscore/log_io.c
- core/log_io_remote.c 2 additions, 1 deletioncore/log_io_remote.c
- core/palloc.c 9 additions, 16 deletionscore/palloc.c
- core/pickle.c 2 additions, 5 deletionscore/pickle.c
- core/salloc.c 1 addition, 1 deletioncore/salloc.c
- core/say.c 2 additions, 1 deletioncore/say.c
- core/stat.c 2 additions, 2 deletionscore/stat.c
- core/tarantool.c 3 additions, 3 deletionscore/tarantool.c
- core/tbuf.c 2 additions, 1 deletioncore/tbuf.c
- core/util.c 2 additions, 2 deletionscore/util.c
- include/admin.h 3 additions, 3 deletionsinclude/admin.h
- include/coro.h 4 additions, 5 deletionsinclude/coro.h
- include/fiber.h 9 additions, 12 deletionsinclude/fiber.h
- include/iproto.h 27 additions, 6 deletionsinclude/iproto.h
Loading
Please register or sign in to comment