<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">Le ven. 4 janv. 2019 à 06:57, karl ramberg <<a href="mailto:karlramberg@gmail.com">karlramberg@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div>This code is probably wrong.<br><div>I posted this fix to inbox because I was unsure how all these nested blocks worked.</div><div><br></div><div>It kind of works now, because the FontImporterTool opens.</div><div><br></div><div>Best,</div><div>Karl</div><div><br></div><div></div></div></div></blockquote></div></div><div dir="auto">No, it's good, just remove the extra ^ </div><div dir="auto">I'm not even sure that it does not work, but without it i'm sure that it works (i'm thinking of the ensure), so removal can avoid a brainstorm ;)</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 3, 2019 at 10:58 PM Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank" rel="noreferrer">nicolas.cellier.aka.nice@gmail.com</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"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr">Le jeu. 3 janv. 2019 à 22:34, <<a href="mailto:commits@source.squeak.org" target="_blank" rel="noreferrer">commits@source.squeak.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">A new version of TrueType was added to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/TrueType-kfr.52.mcz" rel="noreferrer noreferrer" target="_blank">http://source.squeak.org/inbox/TrueType-kfr.52.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: TrueType-kfr.52<br>
Author: kfr<br>
Time: 3 January 2019, 10:34:19.705286 pm<br>
UUID: 248937ad-7e63-4c27-b659-6165fdda9ffe<br>
Ancestors: TrueType-pre.51<br>
<br>
FontImporterTool would not open because 'Noto Color Emoji' error:  'File does not have a naming table'.  <br>
Delete the font from FontPaths and continue<br>
<br>
=============== Diff against TrueType-pre.51 ===============<br>
<br>
Item was changed:<br>
  ----- Method: TTFileDescription class>>openFontFile:do: (in category 'instance creation') -----<br>
  openFontFile: fontFileName do: aBlock<br>
        "Open the font with the given font file name"<br>
        | fontFilePath file |<br>
        fontFilePath := FontPaths at: fontFileName <br>
                ifAbsentPut:[self findFontFile: fontFileName].<br>
        fontFilePath ifNil:[^nil].<br>
        file := [FileStream readOnlyFileNamed: fontFilePath] on: Error do:[:ex|<br>
                "We lost the font; someone might have moved it away"<br>
+               FontPaths removeKey: fontFileName ifAbsent:[].<br>
+               ^nil].<br>
+       [^[aBlock value: file binary]  on: Error do:[:ex|<br>
+               "Error reading font. Skip it" <br>
+               FontPaths removeKey: fontFileName ifAbsent:[].<br>
+               ^nil] ] ensure:[file close].<br></blockquote><div>does a ^ works well inside exception handling?</div><div>I think that just nil would do the work...<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>
-               fontFilePath removeKey: fontFileName ifAbsent:[].<br>
-               ^nil<br>
-       ].<br>
-       ^[aBlock value: file binary] ensure:[file close].!<br>
<br>
<br>
</blockquote></div></div>
<br>
</blockquote></div>
<br>
</blockquote></div></div></div>