[Issue 7515] The new std.string.translate is slow for ASCII text

2012-06-10 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 Jonathan M Davis changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-06-10 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #13 from github-bugzi...@puremagic.com 2012-06-10 20:32:42 PDT --- Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/08acc16b830b35572e6ea004d936a7b2

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-05-28 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #12 from bearophile_h...@eml.cc 2012-05-28 17:13:48 PDT --- (In reply to comment #11) > It retains the ability to handle 256 code units rather than the 128 that ASCII > actually uses. Thank you :-) Please take also a look at Issue

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-05-28 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #11 from Jonathan M Davis 2012-05-28 15:41:22 PDT --- Okay. After some rethinking the issue, I've closed the old pull request and created a new one: https://github.com/D-Programming-Language/phobos/pull/609 It retains the ability

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-11 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #10 from Jonathan M Davis 2012-03-11 15:33:12 PDT --- The fact that the version that's going away deals with a 256 element array and therefore just so happens to work with Extended ASCII may be enough to justify just making the new

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-11 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #9 from Jonathan M Davis 2012-03-11 15:12:49 PDT --- Okay. It seems to me that this is getting a bit ridiculous. Phobos doesn't support Extendend ASCII _anywhere_. Its support of ASCII-specific stuff is already minimal. So, to set

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-11 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #8 from bearophile_h...@eml.cc 2012-03-11 14:40:45 PDT --- (In reply to comment #7) Thank you for your answers. > If you want a truly generic mapping function which operates on ubyte and > creates an array which holds all of the po

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-11 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #7 from Jonathan M Davis 2012-03-11 14:22:41 PDT --- If you want a truly generic mapping function which operates on ubyte and creates an array which holds all of the possible values of that type, then it makes no sense to be puttin

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-11 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #6 from bearophile_h...@eml.cc 2012-03-11 13:53:26 PDT --- I have thought some more about this and I have changed my mind: there is no point to artificially restrict the usefulness of this function. So I suggest: 1) To not use a opa

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-06 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #5 from Jonathan M Davis 2012-03-06 16:06:35 PST --- And _why_ would you ever need to build it manually? That's what makeTrans is for. The docs would have to explain exactly how the array is laid out for it to be at all reasonable

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-06 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #4 from bearophile_h...@eml.cc 2012-03-06 15:42:22 PST --- (In reply to comment #3) > It seems to me that the only reason that makeTrans exists in the first place > rather than just putting it in translate is so that you can reuse t

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-06 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #3 from Jonathan M Davis 2012-03-06 15:27:13 PST --- > 1) If they don't already say it, then I suggest to add in the ddoc of the > functions that the translation arrays have a length 128. This for both old D > programmers and Pytho

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-06 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 --- Comment #2 from bearophile_h...@eml.cc 2012-03-06 04:13:54 PST --- (In reply to comment #1) > https://github.com/D-Programming-Language/phobos/pull/478 Thank you. Your patch seems nice. Two suggestions: 1) If they don't already say it, th

[Issue 7515] The new std.string.translate is slow for ASCII text

2012-03-06 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=7515 Jonathan M Davis changed: What|Removed |Added CC||jmdavisp...@gmx.com --- Comment #1