View Issue Details

IDProjectCategoryView StatusLast Update
0004395Spring engineGeneralpublic2014-05-16 18:29
ReporterMajorBoredom Assigned ToKloot  
PrioritylowSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version96.0 
Summary0004395: LogSections entries must end with a comma or the last entry is not recognized
DescriptionLogOutput.cpp:88-114

If the LogSections entry does not end with a comma, then the .find call on line 90 returns string::npos. So, the last entry is not found.
Steps To ReproduceJust try adding "VFS:20" to the LogSections entry in springsettings.cfg with no comma at the end.

This would ordinarily enable the VFS logger at the level of L_DEBUG, but it will not work.

Then, try adding a comma at the end. Magic!
TagsNo tags attached.
Checked infolog.txt for Errors

Activities

MajorBoredom

2014-05-16 07:20

reporter   ~0013103

Hm, strangely my fix makes the entry appear in the initialization messages but doesn't necessarily cause L_DEBUG level messages to appear. More testing required.

MajorBoredom

2014-05-16 08:52

reporter   ~0013104

Ok, I was compiling in release mode instead of debug.

The workaround of placing a comma at the end of the entry works fine.

jK

2014-05-16 10:33

developer   ~0013105

There is no workaround to make L_DEBUG appear in release builds.
They aren't compiled at all in release builds and so it is impossible to trigger them in non-debug builds.

MajorBoredom

2014-05-16 15:03

reporter   ~0013106

The workaround is for the LogSections entry. If you provide a comma-delimited list of logged sections in springsettings.cfg, the last entry in the list is not parsed unless there is a trailing comma after it.

There's nothing special about the L_DEBUG level itself in this bug -- you could just as easily set the log level for a given section to 40 (L_WARNING) in springsettings.cfg and if there was a trailing comma missing, the entry would be ignored.

Issue History

Date Modified Username Field Change
2014-05-16 07:16 MajorBoredom New Issue
2014-05-16 07:20 MajorBoredom Note Added: 0013103
2014-05-16 08:52 MajorBoredom Note Added: 0013104
2014-05-16 10:33 jK Note Added: 0013105
2014-05-16 15:03 MajorBoredom Note Added: 0013106
2014-05-16 18:29 Kloot Changeset attached => spring release bc2c795e
2014-05-16 18:29 Kloot Assigned To => Kloot
2014-05-16 18:29 Kloot Status new => resolved
2014-05-16 18:29 Kloot Resolution open => fixed