Re: [Simple-support] Array deserialization issue, with test and patch
Brought to you by:
niallg
|
From: Shevek <sh...@an...> - 2014-02-02 22:47:17
|
The patch isn't to fix the JDK bug. It's to fix the bug exhibited by the provided test case, which is to do with Converter getting called for array types, not just element types, but apparently only when CycleStrategy is used. The JDK bug is just an aside, as I had to change the test runner. By the way, an update to org.simpleframework.xml.strategy.CycleStrategyTest.testArray also fails since I wrote this code and test. Thank you very much! S. On 02/02/2014 01:38 PM, Niall Gallagher - Yieldbroker wrote: > I will have a look at the patch and add it in, however the bug below is with the JDK not with the library. I would imagine this will break many libraries relying on StAX, just don't use that version of the JDK. > > -----Original Message----- > From: Shevek [mailto:sh...@an...] > Sent: 01 February 2014 13:27 > To: sim...@li... > Subject: [Simple-support] Array deserialization issue, with test and patch > > I last posted about this in 2012, and here it is again, only this time with a standalone test case, as well as a patch. The issue hasn't changed, and neither has the patch, but hopefully having the test will help. > > Test fails without patch, passes with patch. All other tests pass with patch. > > Please, please, please fix this. It's been bugging us for two years now, and we've been carrying custom binaries of simple-xml in all our repositories. > > Previous thread (2012) was: "RegistryStrategy causes NPE when Converter is used" > > http://sourceforge.net/mailarchive/forum.php?thread_name=1336375189.50334.YahooMailClassic%40web161005.mail.bf1.yahoo.com&forum_name=simple-support > > I mirrored simple-xml to github to make it easier to write and run tests. I also discovered that the simple-xml test suite fails on modern JDK due to https://bugs.openjdk.java.net/browse/JDK-8028111 > > S. > |