Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3050 closed defect (fixed)

local.cfg doesn't like tabs

Reported by: elexis Owned by: leper
Priority: If Time Permits Milestone: Alpha 18
Component: Core engine Keywords:
Cc: Patch:

Description

The parsing of the local.cfg file fails if one adds a tab.

Consider this line:

hotkey.togglefullscreen = "Alt+Return"      ; Toggle fullscreen/windowed mode

If I add a tab after the last quotation mark and before the semicolon, the Alt-button (instead of Alt+Return) will be assigned to the fullscreen-toggle functionality.

If I add the tab somewhere else, the line will have no effect at all.

Additional spaces don't cause any problems.

This problem only happens with A18! A17 does not have this problem (no matter if compiled or installed by ubuntu repository).

Change History (4)

comment:1 by leper, 9 years ago

Owner: set to leper
Status: newassigned

comment:2 by leper, 9 years ago

Resolution: fixed
Status: assignedclosed

Fixed in r16326.

comment:3 by fabio, 9 years ago

Since now tabs are supported, it would be nice to replace spaces with tabs where there are comments to the rigth of the setting, so that they can be easily aligned.

comment:4 by leper, 9 years ago

That would require us using some explicit tab length, else it might not be properly aligned, and for alignment spaces are and should be used.

Note: See TracTickets for help on using tickets.