Hi, I'd really like (at least) a second pair of eyes to review the patch to UnmarshallHandler before I commit the complete patch.
Basically, as the comments in the issue itself show, the user attached an XML for unmarshalling that was wrong in the sense that the names of the XML elements did not match the expected names (as mandated by the default naming scheme used by introspection). After fixing the element names, the attached JUnit test case completed fine. But with wrong names being used, a NPE was thrown. I had a look at UnmarshallHandler, and it seems to me that one additional (simple) if (tempDesc != null && ....) should do the trick. Problem is that I am not that literate on the XML side of things, and whilst both the master and regression test suite finish without any problems (well, no more problems than before for the regression test suite ... ;-)), I just want to get a second opinion. Thanks Werner ------------------------------------------------- If you wish to unsubscribe from this list, please send an empty message to the following address: [EMAIL PROTECTED] -------------------------------------------------
