On 9/3/2011 20:55, Nicolas Cellier wrote:
It would not hurt me to see isBinary integrated in trunk and replacing the isInfix sends... Maybe the purpose was to avoid polymorphism with Stream>>isBinary ?
Probably. I could easily see something like:
ByteArray>>isBinary ^true
String>>isBinary ^false
in which case Symbol>>isBinary would be a major surprise. My preference would be a more explicit name along the lines of #isBinarySelector or #isSelectorBinary or something. #isBinary has a number of potential conflicts and the proposed implementation is used way to rarely to occupy such a generic name.
Cheers, - Andreas