Opened 16 years ago

Closed 16 years ago

Last modified 13 months ago

#8287 closed patch

Hack for config manager glitch

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

Description

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 16 years ago.
Patch against an October 9 CVS snapshot

Download all attachments as: .zip

Change History (5)

by eriktorbjorn, 16 years ago

Attachment: config_fix.diff added

Patch against an October 9 CVS snapshot

comment:1 by eriktorbjorn, 16 years ago

Owner: set to fingolfin

comment:2 by fingolfin, 16 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, 16 years ago

Status: newclosed

comment:4 by digitall, 13 months ago

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