<div dir="ltr">Sure. Sounds good.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 1:29 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">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>
<div class="HOEnZb"><div class="h5"><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 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 TargetLayout.<br>
>><br>
>> Thoughts?<br>
<br>
--<br>
Simon Atanasyan<br>
</div></div></blockquote></div><br></div>