summaryrefslogtreecommitdiff
path: root/doc/todo/File.containsConfPair___38___LightDM.autoLogin_properties/comment_1_c8240ba3abf5cf458eba8ed7e31eaccf._comment
blob: a5a2b80cf6ca8e69d92d433ec857dde6ddb0db22 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
[[!comment format=mdwn
 username="joey"
 subject="""comment 1"""
 date="2015-08-04T14:23:33Z"
 content="""
Thanks for submitting these patches!

Looking at `containsConfPair`, it assumes an ini-style file,
so is a little misplaced in Property.File. which is otherwise about generic
text files.

So, it would probably make sense to move it to a new Property.IniFile
module.

However, [[forum/parsing_a_config_file]] recently pointed out that
the tor config file has a similar need. It's not ini format, but
shares the same basic idea of a "section" line which is followed by 
lines setting things specific to that section.

So, it would be great if `containsConfPair` could be generalized to also
cover that tor config file use case. I think this would be pretty easy;
just make it take one string containing the whole section line (including
square brackets for ini file, or whatever for tor config file), and a
second string containing the whole setting line.
"""]]