[squeak-dev] The Inbox: SMLoader-jr.95.mcz

Thiede, Christoph Christoph.Thiede at student.hpi.uni-potsdam.de
Wed Dec 15 21:29:20 UTC 2021


Hi Jakob,


by contract, I think #contents:notifying: should return true. Shall I add this for you and merge it? :-)


Best,

Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von commits at source.squeak.org <commits at source.squeak.org>
Gesendet: Mittwoch, 15. Dezember 2021 22:05:38
An: squeak-dev at lists.squeakfoundation.org
Betreff: [squeak-dev] The Inbox: SMLoader-jr.95.mcz

A new version of SMLoader was added to project The Inbox:
http://source.squeak.org/inbox/SMLoader-jr.95.mcz

==================== Summary ====================

Name: SMLoader-jr.95
Author: jr
Time: 15 December 2021, 10:05:37.777858 pm
UUID: 167a84f9-4d21-e946-b599-8c44add0fe06
Ancestors: SMLoader-mt.94

Fix error when attempting to save a release in the SqueakMap Package Loader.

The code pane is nowadays built to send contents:notifying:. The SMReleaseBrowser did not implement this, so the method in CodeHolder was executed, which attempts to compile the code. But compiling the code in the SMReleaseBrowser does not make sense; one must set the loadScript instead. The contents: method of SMReleaseBrowser already does this.

=============== Diff against SMLoader-mt.94 ===============

Item was added:
+ ----- Method: SMReleaseBrowser>>contents:notifying: (in category 'model access') -----
+ contents: aText notifying: aController
+        ^ self contents: aText!


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211215/57d6f8a6/attachment.html>


More information about the Squeak-dev mailing list