Hi Henrik,<div><br></div><div>�� �twice you write a slip that the compiler should not allow:</div><div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre">	</span>(map := characterToGlyphMap)</div><div>
<span class="Apple-tab-span" style="white-space:pre">		</span>ifNotNil: [:map | code := map at: code + 1].</div><div><br></div><div><div><span class="Apple-tab-span" style="white-space: pre; ">	</span>code := (map := characterToGlyphMap)</div>
<div><span class="Apple-tab-span" style="white-space:pre">		</span><span class="Apple-tab-span" style="white-space: pre; ">	</span><span class="Apple-tab-span" style="white-space:pre">		</span>ifNil: [aCharacter charCode]</div>
<div><span class="Apple-tab-span" style="white-space:pre">					</span>ifNotNil: [:map | map at: aCharacter charCode + 1]</div></div><div><br></div>these should read</div><div><br></div><div><span class="Apple-tab-span" style="white-space: pre; ">	</span>characterToGlyphMap�ifNotNil: [:map | code := map at: code + 1].</div>
<div><br></div><div><div><span class="Apple-tab-span" style="white-space: pre; ">	</span>code := characterToGlyphMap</div><div><span class="Apple-tab-span" style="white-space: pre; ">		</span><span class="Apple-tab-span" style="white-space: pre; ">	</span>ifNil: [aCharacter charCode]</div>
<div><span class="Apple-tab-span" style="white-space: pre; ">			</span>ifNotNil: [:map | map at: aCharacter charCode + 1]</div></div><div><br></div><div>HTH,</div><div>Eliot</div><div><br></div><div>also in�BitBlt&gt;&gt;#primDisplayString:from:to:map:xTable:kern: you write</div>
<div><span class="Apple-tab-span" style="white-space:pre"><br></span></div><div><span class="Apple-tab-span" style="white-space:pre">			</span>ascii := glyphMap at: (aString at: charIndex) asciiValue + 1.</div><div><br></div>
<div>whereas the current Squeak code reads</div><div><br></div><div><div><span class="Apple-tab-span" style="white-space:pre">		</span>ascii := (aString at: charIndex) asciiValue.</div><div><span class="Apple-tab-span" style="white-space:pre">		</span>glyphMap ifNotNil:[ascii := glyphMap at: ascii+1].</div>
</div><div><br></div><div>Should Squeak keep its original code (which seems equivalent) or is there a specific reason to eliminate the ifNotNil: test?</div><div><br></div><div>TIA</div><div>Eliot</div><div><br></div><div>
<div class="gmail_quote">2010/8/30 Henrik Sperre Johansen <span dir="ltr">&lt;<a href="mailto:henrik.s.johansen@veloxit.no">henrik.s.johansen@veloxit.no</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
�On 30.08.2010 22:05, Nicolas Cellier wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2010/8/30 St�phane Ducasse&lt;<a href="mailto:stephane.ducasse@inria.fr" target="_blank">stephane.ducasse@inria.fr</a>&gt;:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Aug 30, 2010, at 8:33 PM, Johan Brichau wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
so what did you do?<br>
<br>
I see that the file got removed... �I will submit a bug issue for that with the file attached.<br>
<br>
The *real* problem is that you cannot remove it either! It does _not_ want to go away!<br>
</blockquote>
Oops! A bad spirit in pharo :)<br>
<br>
</blockquote>
Any person able to understand MultiDisplayScanner arcanes wouldn&#39;t see<br>
any magic in this. But I really wonder whether a single person can<br>
manage such complexity... Henrik ?<br>
<br>
Nicolas<br>
</blockquote>
Hurr durr, I hate a challenge, can&#39;t really stop...<br>
In Pharo, the display primitive fails, and the fallback code does not deal correctly with the characterToGlyphMap used by StrikeFonts to store arrows at the normal ^_ spots, and substitute with characters at slots 128/129.<br>

In Squeak, the glyph-by-glyph rendering is encountered, and neither hasGlyphOf: nor glyphInfoInto: takes the map into account<br>
Attached is the fixed code for all three methods, should work in both Pharo and Squeak with these.<br>
(Issue 2886 on the Pharo tracker)<br>
<br>
Cheers,<br>
Henry<br>
<br>
<br>
<br>_______________________________________________<br>
Pharo-project mailing list<br>
<a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
<a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br></blockquote></div><br></div>