<div dir="ltr">On Thu, Jul 17, 2008 at 6:40 PM, Hernán Ordiales <<a href="mailto:h@ordia.com.ar">h@ordia.com.ar</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Thu, Jul 17, 2008 at 6:34 PM, Pau Arumí <<a href="mailto:parumi@iua.upf.edu">parumi@iua.upf.edu</a>> wrote:<br>
> On dj, 2008-07-17 at 18:18 -0300, Hernán Ordiales wrote:<br>
>> just one note more, i think that the main issue that makes really big<br>
>> this patch is because now we are using BaseTypedIn|OutControl where we<br>
>> had In|OutControl (registry, etc)<br>
>><br>
>> with this class diagram:<br>
>> class BaseTypedIn|OutControl<br>
>> class TypedIn|OutControl: BaseTypedIn|OutControl<br>
>> class In|OutControl: TypedIn|OutControl<TControlData><br>
>><br>
>> so many of the code using In|OutControls types is replaced by<br>
>> BaseTypedIn|Out to allow a general treatment of the controls. Other<br>
>> issue is that in some parts where now is required a Base type, current<br>
>> code is using an interface not present in Base, only at old<br>
>> In|OutControl classes, and are methods not-convenient to add to Base,<br>
>> so a code change/refactor is required.<br>
><br>
> Yes, but many of these rafactorings can be done at trunk before the<br>
> actual switch. For example substitute<br>
> proc.GetInControl("MyIn").DoControl(1.); -> DoControlAsFloat(proc,<br>
> "MyIn", 1.)   (I'm using wrong function names, I know)<br>
<br>
</div>I like this idea, i think we can make the delta smaller with this kind<br>
of things.<br>
<div><div></div><div class="Wj3C7c"><br>
> And maybe we should be refactoring In|OutControl to make it more similar<br>
> to BaseTypedIn|OutControl interfaces also in trunk. However I'd like to<br>
> see code on that issue.<br>
><br>
> All this belongs to the "start making the delta smaller" approach.</div></div></blockquote><br>The first thing to make delta smaller is to make Get*Controls() and Get*Control() not to use the control directly ( ex. bla_proc.GetInControls().Get("control").DoControl() )<br>
With the last patch i've sent many of these are solved using external free functions.<br>So, please, check the last patch and comment about it.<br>About the branch, i think it's a good idea but if we have no other chance, i agree with david about the problems merging a branch into a developed trunk.<br>
Bye<br></div></div>