<div dir="ltr">Hi Patrick, Hi Levente, Hi ALl,<div><br></div><div>    I just did a simple fix for two test failures in UTF16TextConverterTest and committed to inbox only to discover that there are several other versions waiting review.  In particular Monty has submitted a substantial change.  Let me ask that those people who understand the txt converter framework have a look at these proposals.</div><div><br></div><div>Another observation is that there are a couple of methods (UTF16TextConverter UTF32TextConverter) that do this:</div><div><br></div><div><div>swapLatin1EncodingByteOrder</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">    </span>latin1Encodings := latin1Encodings collect: [:each | </div><div><span class="gmail-Apple-tab-span" style="white-space:pre">         </span>each ifNotNil: [each reverse]]</div><div><br></div><div>but reverse (quite rightly) isn't implemented in Squeak.  It's meaningless.  One needs to specify a byte size and reverse for that byte size, e.g.  1 reversed in UTF16TextConverter would be 256, but in UTF32TextConverter would be 16777216, right?</div><div><br></div><div>But since the latin1Encodings in both  UTF16TextConverter UTF32TextConverter are (rightly) all nil these could just as well be implemented as ^self.</div><div><br></div><div class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</div></div>