<br><div class="gmail_quote">On Wed, Mar 5, 2008 at 1:56 PM, Don Stewart <<a href="mailto:dons@galois.com">dons@galois.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
byorgey:<br>
<div class="Ih2E3d">> What's the current status on this? Don? Spencer?<br>
<br>
</div>Currently Spencer's out of action, and expressed some doubts, which is<br>
making it hard to reach a consensus here.<br>
<div class="Ih2E3d"><br>
> Summary so far:<br>
><br>
> * Andrea's patch moves runLayout into the LayoutClass, changing the<br>
> signature slightly so that layouts receive all possible information.<br>
> * This would necessitate updating all *users* of LayoutClass to use<br>
> runLayout instead of doLayout/emptyLayout. The patches for these changes<br>
> are already available.<br>
> * New instances of LayoutClass can choose whether to implement<br>
> runLayout, doLayout, or pureLayout, and so on. Existing instances of<br>
> LayoutClass will not be affected (except those that are also users of it,<br>
> e.g. LayoutModifier).<br>
> * This provides a nice solution (IMO the best) for PerWorkspace.<br>
<br>
> Also:<br>
><br>
> * If this patch is applied to the core, I am willing to take<br>
> responsibility for making sure the contrib library is updated<br>
> appropriately, and that thorough documentation is made available to guide<br>
> those wishing to understand the architecture of the LayoutClass.<br>
> * David and I think this is elegant and should be applied. (See David's<br>
> prior e-mail for an excellent discussion of some of the issues involved.)<br>
> * Spencer has voiced discomfort with the style of this change to the<br>
> API.<br>
> * Don has not expressed an opinion one way or the other.<br>
<br>
</div>I'm happy if Brent and David are on this one.<br>
<div class="Ih2E3d"><br>
> It would be nice if a decision could be reached on this one way or the<br>
> other, so we can move ahead with PerWorkspace and other things. I'm not<br>
> trying to be pushy, just trying to spur things forward in a friendly way.<br>
<br>
</div>As Spencer only expressed doubt here, and I'm persuaded by Brent's<br>
arguments (and his offering to champion them), let's proceed.<br>
<font color="#888888"></font></blockquote><br>OK, I am currently preparing a bundle of patches to update the core and contrib all at once. I'll send them out once I'm done (probably Monday).<br><br>-Brent<br></div>