[cfe-dev] Moving named cast helpers out of Sema

[Daniel Dunbar]

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.
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. :)

 - Daniel

On Thu, Nov 6, 2008 at 12:43 PM, Sebastian Redl <
sebastian.redl at getdesigned.at> wrote:

> Chris Lattner wrote:
> >
> > On Nov 5, 2008, at 11:02 AM, Sebastian Redl wrote:
> >
> >> Hi,
> >>
> >> Attached patch moves the C++ named cast helper functions out of Sema
> >> and makes them static functions local to their own file., as
> >> requested by Chris. (The patch also moves all named cast stuff to its
> >> own file.)
> >
> > This looks great to me, thanks Sebastian!
> So, one voice for, one against. Do we have a tie-breaker? Or is Chris a
> tie-breaker by himself? ;-)
>
> Sebastian
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20081107/a3d5b436/attachment.html>