<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
                                        Hi Lauren --<div><br></div><div>This will raise a debugger:</div><div>   PackageOrganizer default packageOfSystemCategory: nil<br></div><div><span style="font-size: 13.3333px"><br></span></div><div><span style="font-size: 13.3333px">This will not raise a debugger:</span></div><div><span style="font-size: 10pt">   PackageOrganizer default packageOfSystemCategory: nil ifNone: nil</span><br></div><div><br></div><div>> <span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">and noticed PackageInfo </span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">handles the nil system category as never yourself</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">I don't understand what you mean here. :-/</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">></span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">That discovered, I still think putting a short circuit return in</span></div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> PackageOrganizer makes sense. If nil means 'not me', why spend time</span><br style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> enumerating?</span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"> </span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Well, you suggest a typical "ifNil"-check. The contract in #packageOfSystemCategory:ifNone: is that only valid system categories are used as the "aSystemCategory" argument. Providing 'nil' is not a valid system category. Now, anything can happen. Usually, we rely on the fact that it is not possible to have package with a nil name.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">If we would not rely on this contract, we would start to sprinkle "ifNil" checks all over the place. That's not a good style. It produces code that is difficult to maintain because nil has no specific domain.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Best,</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Marcel</span></div><div class="mb_sig"></div>
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 14.03.2022 08:43:16 schrieb Lauren Pullen <drurowin@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi Marcel,<br><br>On 3/11/22 08:19, Marcel Taeumel wrote:<br>> Considering the patch you proposed via PackageInfo-Base-lrnp.76, I think that you package organizer contains an invalid package 'nil'.<br>> <br>> Would you check that via "PackageOrganizer default packages"?<br>> Can you remember how you got into that state?<br>> <br>> It's unusual that the package organizer has a 'nil' package in its registry. Therefore, I would rather not start to patch the system with such extra "ifNil"-checks.<br>I checked a fresh trunk image, no nils in the package organizer.<br><br>I was looking at the debugger for it again and noticed PackageInfo<br>handles the nil system category as never yourself... it's the subclass<br>MCMockPackageInfo that doesn't do the nil check.<br><br>That discovered, I still think putting a short circuit return in<br>PackageOrganizer makes sense.  If nil means 'not me', why spend time<br>enumerating?<br><br></div></blockquote></div>