jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jukka Zitting (JIRA)" <j...@apache.org>
Subject [jira] Updated: (JCR-1574) [PATCH] PropertyWrapper has a typo that causes an infinite loop
Date Wed, 07 May 2008 12:36:55 GMT

     [ https://issues.apache.org/jira/browse/JCR-1574?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Jukka Zitting updated JCR-1574:
-------------------------------

    Fix Version/s:     (was: core 1.4.4)

AFAIK that method is never executed (PropertyWrapper is internal to the SAX event generator),
so the issue is a bit theoretical.

Instead of fixing this as proposed, I'd rather get rid of the entire PropertyWrapper class
(and the loads of unused code it contains) by refactoring the SAX event generation code.

> [PATCH] PropertyWrapper has a typo that causes an infinite loop
> ---------------------------------------------------------------
>
>                 Key: JCR-1574
>                 URL: https://issues.apache.org/jira/browse/JCR-1574
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-core
>    Affects Versions: core 1.4.2
>            Reporter: Dave Brosius
>            Priority: Minor
>         Attachments: PropertyWrapper_typo_infinite_loop.patch
>
>
> PropertyWrapper.refresh is missing the delegation to 'property' for refresh, and has
an infinite loop instead. Patch fixes this.
> [CODE] 
>    /** {@inheritDoc} */
>     public void refresh(boolean keepChanges) throws InvalidItemStateException,
>             RepositoryException {
>         refresh(keepChanges);
>     }
> [/CODE]

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message