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 post to this group, send email to clojure@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/clojure?hl=en -~----------~----~----~----~------~----~------~--~---
genclass-protected-fields.diff
Description: Binary data
On Dec 1, 2008, at 3:44 PM, Rich Hickey wrote: > > > > 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 1133. >> >> I promise to never bring this up again. =) >> >> -Matt >> > > I'm glad you did. There were a couple of enhancements that got delayed > in the AOT process. > > As far as the patch, I'd prefer not to have any logic that uses > exceptions as part of normal flow control (i.e. calling > getDeclaredField and eating the exception), so could you please define > in terms of getDeclaredFields instead? > > Thanks, > > Rich > > --~--~---------~--~----~------------~-------~--~----~ > You received this message because you are subscribed to the Google > Groups "Clojure" group. > To post to this group, send email to clojure@googlegroups.com > To unsubscribe from this group, send email to [EMAIL PROTECTED] > For more options, visit this group at > http://groups.google.com/group/clojure?hl=en > -~----------~----~----~----~------~----~------~--~--- >
