On Mon, Sep 7, 2009 at 5:37 AM, David Andrews<dammi...@gmail.com> wrote: > > > triddell wrote: >> This patch fixed the issue for me on the IBM JDK. > > Sadly, not for me! (IBM 1.5.0) c/c/pprint/ColumnWriter still fails > with: > [java] java.lang.ClassFormatError: (clojure/contrib/pprint/ > ColumnWriter) field name is invalid at offset=0 (ColumnWriter.clj:36) > > ... during contrib build with genclass.clj modified to use \$ in place > of \- >
I took another look at this. I believe the issue is that my original change was a quick-fix and doesn't really handle all cases. The issue that Dave is hitting seems to be occurring because of the default prefix character of '-' for the genclass methods. If I change the prefix character in clojure/contrib/pprint/ColumnWriter.clj to _ and change the corresponding defn-s, then it seems to accept the class. I went digging through genclass.clj to try and make this fix more robust but I haven't been able to fully understand where this needs to go yet. Still digging... /mike. --~--~---------~--~----~------------~-------~--~----~ 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 Note that posts from new members are moderated - please be patient with your first post. To unsubscribe from this group, send email to clojure+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/clojure?hl=en -~----------~----~----~----~------~----~------~--~---
