groovy
  1. groovy
  2. GROOVY-3109

StackOverflowError with DOMCategory and missing property

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.5.7, 1.6-beta-2
    • Fix Version/s: 1.6-rc-1, 1.7-beta-1
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      1

      Description

      Run the attached script to see the bug in action. It seems getProperty() and DOMCategory.get() end up in an infinite loop. This is a serious problem for Gant since it means that you basically can't use categories in Gant scripts.

        Issue Links

          Activity

          Hide
          blackdrag blackdrag added a comment -

          I close this bug as the fix is now in 1.5.8 too

          Show
          blackdrag blackdrag added a comment - I close this bug as the fix is now in 1.5.8 too
          Hide
          Jim White added a comment -

          Applying this fix breaks the 1.5 build on JDK 1.4.

          http://bamboo.ci.codehaus.org/browse/GROOVY-GROOVY-611

          Show
          Jim White added a comment - Applying this fix breaks the 1.5 build on JDK 1.4. http://bamboo.ci.codehaus.org/browse/GROOVY-GROOVY-611
          Hide
          blackdrag blackdrag added a comment -

          I created GROOVY-3220 as a clone for 1.5.x of this bug. Since it works on 1.7/1.6 it might very well be not really related to the patch, but to a problem uncovered by the patch. Anyway, it may be needed to divert from the path taken for 1.7/1.6 so we should protocol thatin the other issue

          Show
          blackdrag blackdrag added a comment - I created GROOVY-3220 as a clone for 1.5.x of this bug. Since it works on 1.7/1.6 it might very well be not really related to the patch, but to a problem uncovered by the patch. Anyway, it may be needed to divert from the path taken for 1.7/1.6 so we should protocol thatin the other issue
          Hide
          Jim White added a comment -

          Merged the changes for GROOVY-3220 into the trunk. Will do for 1.6 if no complaints.

          Show
          Jim White added a comment - Merged the changes for GROOVY-3220 into the trunk. Will do for 1.6 if no complaints.
          Hide
          Jim White added a comment -

          Your comment in 3220 didn't sound like a complaint about my change, so I've merged it into 1.6.

          Show
          Jim White added a comment - Your comment in 3220 didn't sound like a complaint about my change, so I've merged it into 1.6.

            People

            • Assignee:
              Guillaume Laforge
              Reporter:
              Peter Ledbrook
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: