Bug #2233
C99-ism in the code of mod_accesslog
100%
Description
The mod_accesslog code has a C99-ism in it. The attached patch fixes the code.
Associated revisions
mod_accesslog: Fix var declarations mixed in source (fixes #2233)
History
Updated by brad@comstyle.com almost 8 years ago
Olaf-van-der-Spek wrote:
What's wrong with using C99 features?
It would be nice if the code actually compiled and there is no C99 code anywhere else in the lighttpd codebase and similar issues have been fixed in the past.
Updated by brad@comstyle.com almost 8 years ago
Olaf-van-der-Spek wrote:
What's wrong with using C99 features?
It isn't much of a "feature" to make the code uglier.
Anyway, the same issue was fixed in src/response.c...
http://redmine.lighttpd.net/projects/lighttpd/repository/revisions/2721
Updated by Olaf-van-der-Spek almost 8 years ago
brad@comstyle.com wrote:
It would be nice if the code actually compiled and there is no C99 code anywhere else in the lighttpd codebase and similar issues have been fixed in the past.
It does compile, AFAIK.
Why do C99 features result in ugly code?
Updated by stbuehler almost 8 years ago
- Status changed from New to Fixed
- % Done changed from 0 to 100
Applied in changeset r2743.
Also available in: Atom
mod_accesslog: Fix var declarations mixed in source (fixes #2233)
git-svn-id: svn://svn.lighttpd.net/lighttpd/branches/lighttpd-1.4.x@2743 152afb58-edef-0310-8abb-c4023f1b3aa9