Opened 20 years ago

Closed 20 years ago

Last modified 5 years ago

#8287 closed patch

Hack for config manager glitch

Reported by: eriktorbjorn Owned by: fingolfin
Priority: normal Component: --Other--
Version: Keywords:
Cc: Game:


The new config manager tries to parse blank lines as key/value pairs. Since my .scummvmrc begins like this

# Global settings

[scummvm] cdrom=-1 aspect_ratio=true

the parser chokes because "we have a key without a domain first". I don't want to change this part of the code (unless I absolutely have to) until I know it has settled down, but this patch should make it ignore empty lines completely.

Ticket imported from: #821018. Ticket imported from: patches/392.

Attachments (1)

config_fix.diff (795 bytes ) - added by eriktorbjorn 20 years ago.
Patch against an October 9 CVS snapshot

Download all attachments as: .zip

Change History (5)

by eriktorbjorn, 20 years ago

Attachment: config_fix.diff added

Patch against an October 9 CVS snapshot

comment:1 by eriktorbjorn, 20 years ago

Owner: set to fingolfin

comment:2 by fingolfin, 20 years ago

I put an alternative fix into CVS which ignores whitespaces at the line start for key/value pairs. This way one could use indention, if one wants to

[scummvm] cdrom=-1 aspect_ratio=true

comment:3 by fingolfin, 20 years ago

Status: newclosed

comment:4 by digitall, 5 years ago

Component: --Other--
Note: See TracTickets for help on using tickets.