I know why!

The old class builder did not have an API to set the comment when building or 
changing a class.
So Monticello actually created the class and *then* it explicitly set a comment.

When I did the change to support Slot definitions, I moved MC over to the new 
API and set the comment
directly when creating the class. (Interesting how simple the code got…).

So this means that in the past MC never used the class builder to set the 
comment… or maybe in some cases,
the comment change was even announced double?

I think we an just remove the #suspendAllWhile: guard.

> On 21 Mar 2015, at 15:01, Nicolai Hess <[email protected]> wrote:
> 
> Someone knows why ClassCommented-announcements aren't 
> announced during a mc package load ?
> 
> see: 
> 15190 <https://pharo.fogbugz.com/default.asp?15190>
> Nautilus incorrectly flags Error classes as "missing class comments"
> 
> and
> SlotClassBuilder>>#build
> 
> ....
> SystemAnnouncer uniqueInstance suspendAllWhile: [
>             comment ifNotNil: [result classComment: comment stamp: 
> commentStamp]].



Reply via email to