Hi, I was reading the coding conventions which says, "every root class that implements #initialize must have a #new on the class side" and I was wondering, if the following is implied on subclasses of WAComponent and WATask:
<div><div><br class="webkit-block-placeholder"></div><div>class methods:</div><div><br class="webkit-block-placeholder"></div><div>WRONG</div><div><br class="webkit-block-placeholder"></div><div>new</div><div> ^super new initialize
</div><div><br class="webkit-block-placeholder"></div><div>CORRECT</div><div><br class="webkit-block-placeholder"></div><div>new</div><div> ^self basicNew initialize</div><div><br class="webkit-block-placeholder"></div>
<div><br class="webkit-block-placeholder"></div><div>If the above is correct, is this the case for classes that are subclassed from Object?</div><div><br class="webkit-block-placeholder"></div><div>Thanks in advance,</div>
<div><br class="webkit-block-placeholder"></div><div>-Conrad<br><br><div><span class="gmail_quote">On 8/31/07, <b class="gmail_sendername">Philippe Marschall</b> <<a href="mailto:philippe.marschall@gmail.com">philippe.marschall@gmail.com
</a>> wrote:</span><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">2007/8/31, Michael Lucas-Smith <<a href="mailto:mlucas-smith@cincom.com">mlucas-smith@cincom.com
</a>>:<br>> Philippe Marschall wrote:<br>> > 2007/8/30, Michael Lucas-Smith <<a href="mailto:mlucas-smith@cincom.com">mlucas-smith@cincom.com</a>>:<br>> ><br>> >> Hello Seasiders,<br>> >>
<br>> >> Squeak has the #initialize pattern on Object - but VisualWorks doesn't.<br>> >> I'm not sure about Gemstone or Dolphin.<br>> >><br>> >> Aside from the argument that may be we should have that pattern too- in
<br>> >> the interest of code portability, what is the best approach here.<br>> >><br>> ><br>> > Acutally this issue like many others is covered in the Seaside coding<br>> > conventions:
<br>> > <a href="http://www.seaside.st/communit">http://www.seaside.st/communit</a>y/conventions<br>> ><br>> > I just simply forgot is in this case. Seaside2.8a1-pmm.463 should fix this.<br>> ><br>
> Thanks. I was a little surprised, as I hadn't seen this previously. I<br>> figured if it were a common problem, it would have been happening<br>> everywhere. Good to see it's covered in the conventions :)
<br><br>That's mostly thanks to Michel Bany. For a long time he helped<br>increase the portability by making us aware of our "sins" and fixing<br>unportable stuff.<br><br>Cheers<br>Philippe<br><br>> Michael
<br>> _______________________________________________<br>> Seaside mailing list<br>> <a href="mailto:Seaside@lists.squeakfoundation">Seaside@lists.squeakfoundation</a>.org<br>> <a href="http://lists.squeakfoundation">
http://lists.squeakfoundation</a>.org/cgi-bin/mailman/listinfo/seaside<br>><br>_______________________________________________<br>Seaside mailing list<br><a href="mailto:Seaside@lists.squeakfoundation">Seaside@lists.squeakfoundation
</a>.org<br><a href="http://lists.squeakfoundation">http://lists.squeakfoundation</a>.org/cgi-bin/mailman/listinfo/seaside<br></blockquote></div><br></div></div>