I see both sides. Perhaps what really should be done is to split out a separate class for the actual implementation of Sema? Or alternately, there may be parts of Sema which can be refactored into independent helper classes. I don't have a good view of Sema as a whole, I have only touched pieces of it in an ad hoc manner.<div>
<br></div><div>I don't think you should get saddled with the work Sebastian, so I would say go ahead and follow Chris' suggestions and we can push improving Sema's organization to another topic or a rainy day. :)</div>
<div><br></div><div> - Daniel</div><div><div><br><div class="gmail_quote">On Thu, Nov 6, 2008 at 12:43 PM, Sebastian Redl <span dir="ltr"><<a href="mailto:sebastian.redl@getdesigned.at">sebastian.redl@getdesigned.at</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="Ih2E3d">Chris Lattner wrote:<br>
><br>
> On Nov 5, 2008, at 11:02 AM, Sebastian Redl wrote:<br>
><br>
>> Hi,<br>
>><br>
>> Attached patch moves the C++ named cast helper functions out of Sema<br>
>> and makes them static functions local to their own file., as<br>
>> requested by Chris. (The patch also moves all named cast stuff to its<br>
>> own file.)<br>
><br>
> This looks great to me, thanks Sebastian!<br>
</div>So, one voice for, one against. Do we have a tie-breaker? Or is Chris a<br>
tie-breaker by himself? ;-)<br>
<font color="#888888"><br>
Sebastian<br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>