Commit646dc16

authored

Merge pull request#2047 from EliahKagan/quoted-empty

Remove explicit empty `""` handling in ConfigParser

2 parents63160de +2f22524 commit646dc16Copy full SHA for 646dc16

File tree

-3

lines changed

-3

lines changed

Lines changed: 0 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -501,9 +501,6 @@ def string_decode(v: str) -> str:`
`501`	`501`	`ifpos!=-1andoptval[pos-1].isspace():`
`502`	`502`	`optval=optval[:pos]`
`503`	`503`	`optval=optval.strip()`
`504`		`-ifoptval=='""':`
`505`		`-optval=""`
`506`		`-# END handle empty string`
`507`	`504`	`optname=self.optionxform(optname.rstrip())`
`508`	`505`	`iflen(optval)>1andoptval[0]=='"'andoptval[-1]!='"':`
`509`	`506`	`is_multi_line=True`

Lines changed: 2 additions & 0 deletions

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -416,6 +416,10 @@ def test_config_with_quotes(self):`
`416`	`416`	`self.assertEqual(cr.get("user","name"),"Cody Veal")`
`417`	`417`	`self.assertEqual(cr.get("user","email"),"cveal05@gmail.com")`
`418`	`418`
	`419`	`+deftest_config_with_empty_quotes(self):`
	`420`	`+cr=GitConfigParser(fixture_path("git_config_with_empty_quotes"),read_only=True)`
	`421`	`+self.assertEqual(cr.get("core","filemode"),"","quotes can form a literal empty string as value")`
	`422`	`+`
`419`	`423`	`deftest_config_with_quotes_with_literal_whitespace(self):`
`420`	`424`	`cr=GitConfigParser(fixture_path("git_config_with_quotes_whitespace_inside"),read_only=True)`
`421`	`425`	`self.assertEqual(cr.get("core","commentString"),"# ")`

Comments

(0)