<div dir="ltr">Thank you for doing this!</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 4:12 PM, Simon Atanasyan <span dir="ltr"><<a href="mailto:simon@atanasyan.com" target="_blank">simon@atanasyan.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Merging TargetLayout and Layout classes unexpectedly produced rather<br>
small diff. I committed the change at r234065.<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Apr 3, 2015 at 11:30 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> Sure. Sounds good.<br>
><br>
> On Fri, Apr 3, 2015 at 1:29 PM, Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br>
>><br>
>> Good point. But I suggest to do that by two steps. First, remove<br>
>> TargetLayout and rename DefaultLayout to TargetLayout. Second, merge<br>
>> TargetLayout and Layout. The first step is easy, the second step<br>
>> generates large diff, requires reviewing etc.<br>
>><br>
>> On Fri, Apr 3, 2015 at 10:40 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
>> > I'm wondering if we even need TargetLayout.<br>
>> ><br>
>> > DefaultLayout is the only derived class of Layout. TargetLayout is the<br>
>> > only<br>
>> > derived class of DefaultLayout. It looks like we can merge Layout,<br>
>> > DefaultLayout and TargetLayout into one.<br>
>> ><br>
>> > On Fri, Apr 3, 2015 at 12:36 PM, Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Hi Rui, Shankar,<br>
>> >><br>
>> >> Do we really need empty TargetLayout class? No we have the following<br>
>> >> hierarchy:<br>
>> >><br>
>> >> Layout <- DefaultLayout<> <- TargetLayout<> <- xxxTargetLayout<><br>
>> >><br>
>> >> I suggest to remove TargetLayout and rename DefaultLayout to<br>
>> >> TargetLayout.<br>
>> >><br>
>> >> Thoughts?<br>
>><br>
>> --<br>
>> Simon Atanasyan<br>
><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Simon Atanasyan<br>
</font></span></blockquote></div><br></div>