<div dir="ltr">Hi,<div>I made adjustRightX a no-op. </div><div>Fix in trunk.</div><div><a href="http://source.squeak.org/trunk/MorphicExtras-kfr.249.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/MorphicExtras-kfr.249.mcz</a>  <br></div><div><br></div><div>Cheers,</div><div>Karl</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 31, 2018 at 8:29 PM tim Rowledge <<a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
> On 2018-12-31, at 11:20 AM, Chris Cunningham <<a href="mailto:cunningham.cb@gmail.com" target="_blank">cunningham.cb@gmail.com</a>> wrote:<br>
> <br>
> <br>
> What the intent looks like is to avoid the #adjustRightX if the wrapFlag is not set - which is causing the error(s), at least when I tried it.<br>
<br>
Yup, that's the only visible difference - unless of course it turned out that the *order* of sending messages was important here. Occasionally that justifies special cases, but even then, an #isKindOf: is justification for a slapped wrist at the least.<br>
<br>
> <br>
> Another alternative might be to implement TextOnCurve>>adjustRightX to be a no-op (with a suitable comment) so that it didn't do anything in this case. Then the isKindOf: could go away.<br>
<br>
Yup.<br>
<br>
<br>
tim<br>
--<br>
tim Rowledge; <a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
Cap'n!  The spellchecker kinna take this abuse!<br>
<br>
<br>
<br>
</blockquote></div>