[Issue 7674] regex replace requires escaped format

2012-03-22 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7674


Dmitry Olshansky  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---


[Issue 7674] regex replace requires escaped format

2012-03-17 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7674



--- Comment #2 from github-bugzi...@puremagic.com 2012-03-17 11:04:23 PDT ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/097faaca84b0d8f4031cc56721ec846ff4bd6a6e
fix Issue 7674 - regex replace requires escaped format

Kill unnecessary escaping in replace format string.

https://github.com/D-Programming-Language/phobos/commit/bdaa3260ffe62237d7360e84f8e6f01b30e4ab54
changelog entry for issue 7674

https://github.com/D-Programming-Language/phobos/commit/55debbccd020059aec0764cbcbd1773c3a85fe73
Merge pull request #492 from blackwhale/issue-7674

Issue 7674 - regex replace requires escaped format

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---


[Issue 7674] regex replace requires escaped format

2012-03-09 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7674



--- Comment #1 from Dmitry Olshansky  2012-03-09 
03:30:45 PST ---
(In reply to comment #0)
> When using std.regex.replace the format parameter is also requiring to be a
> valid regex.
Strange.
But speaking of format string it is expected to be a plain string there is no
need to escape a thing in there IRC.

 The below example is expected to pass
> 
> import std.regex;
> 
> void main() {
> auto str = "hello?";
> 
> assert(str.replace(regex(r"\?", "g"), r"\?") == r"hello\?");
> }
> 
> This should fail:
> 
> import std.regex;
> 
> void main() {
> auto str = "hello?";
> 
> assert(str.replace(regex(r"\?", "g"), r"\\?") != r"hello\?");
> }
> 
> Maybe I'm wrong on this, but the current requirement seems off.
Apparently it is, I'll take a look.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---