> On 18 בפבר׳ 2016, at 21:58, Andrea Faulds <a...@ajf.me> wrote: > > > Hmm. Well, if we're doing mass deprecations, maybe we should finally get rid > of hebrev() and hebrevc()? It feels out-of-place to have a poorly-documented > standard library function specifically for converting between two different > legacy 8-bit encodings used for Hebrew, especially when one of them is just > the other with nl2br() included! Users who need to convert between different > variants of ISO-8859-8 can use iconv() instead, or perhaps use a userland > polyfill of hebrev/c().
I'm not aware that you can use iconv to convert visual to logical Hebrew or vice versa. Not saying you can't, just that I'm not aware it does - I don't think it support visual encodings at all. Do you know how to convert between them using iconv? If you do, I'm fine reimplementing these functions using iconv, again, if we can eliminate the pointless upgrade hassle. The other alternative of a userland version isn't that simple to write, would be slower, and most importantly buys us nothing. I'm guessing that hebrevc() predates nl2br() (could be wrong), but regardless, I wouldn't touch it as there's zero gain to be had. Last - we're not or at least shouldn't be doing mass deprecations. There should be an extremely strong reason to break perfectly good code. Zeev -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
