Project

General

Profile

Bug #2233

C99-ism in the code of mod_accesslog

Added by brad@comstyle.com about 10 years ago. Updated about 10 years ago.

Status:
Fixed
Priority:
Normal
Category:
mod_accesslog
Target version:
ASK QUESTIONS IN Forums:

Description

The mod_accesslog code has a C99-ism in it. The attached patch fixes the code.


Files

patch-src_mod_accesslog_c.diff (713 Bytes) patch-src_mod_accesslog_c.diff brad@comstyle.com, 2010-07-06 15:31
#1

Updated by Olaf-van-der-Spek about 10 years ago

What's wrong with using C99 features?

#2

Updated by brad@comstyle.com about 10 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.

#3

Updated by brad@comstyle.com about 10 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

#4

Updated by Olaf-van-der-Spek about 10 years ago

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?

#5

Updated by stbuehler about 10 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset r2743.

Also available in: Atom