Re: Supporting protected fields in genclass

2008-12-03 Thread Rich Hickey
On Tue, Dec 2, 2008 at 10:40 PM, Matt Revelle <[EMAIL PROTECTED]> wrote: > Updated patch attached, the extra let for readability looked ugly to > me after some sleep. > > > > > > > > > On Dec 1, 2008, at 3:44 PM, Rich Hickey wrote: > > > > > > > > > On Dec 1, 2:57 pm, Matt Revelle <[EMAIL PROTECTE

Re: Supporting protected fields in genclass

2008-12-02 Thread Matt Revelle
Updated patch attached, the extra let for readability looked ugly to me after some sleep. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.c

Re: Supporting protected fields in genclass

2008-12-01 Thread Matt Revelle
Patch with requested changes attached. Also fixed a bug, the error message now correctly refers to the class used to initiate the field search. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "Clojure" group. To

Re: Supporting protected fields in genclass

2008-12-01 Thread Matt Revelle
Rich, Will make the change and resubmit tonight or tomorrow night. Having computer issues this evening. -Matt On Dec 1, 2008, at 3:44 PM, Rich Hickey <[EMAIL PROTECTED]> wrote: > > > > On Dec 1, 2:57 pm, Matt Revelle <[EMAIL PROTECTED]> wrote: >> Yes, this again. Now that we have AOT, I wa

Re: Supporting protected fields in genclass

2008-12-01 Thread Rich Hickey
On Dec 1, 2:57 pm, Matt Revelle <[EMAIL PROTECTED]> wrote: > Yes, this again. Now that we have AOT, I wanted to revisit supporting > the exposing of inherited protected fields that originate from farther > up the hierarchy chain than the super class. > > Attached is a patch against SVN revision