[ 
https://issues.apache.org/jira/browse/CXF-7520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16326098#comment-16326098
 ] 

ASF GitHub Bot commented on CXF-7520:
-------------------------------------

ffang commented on issue #365: [CXF-7520]:Ensure CXF can build with JDK9 
GA(build 9+181)
URL: https://github.com/apache/cxf/pull/365#issuecomment-357645837
 
 
   @jimma about we need extend EscapeHandler to support the empty string, I 
just found same and got 
   JSONProviderTest.testWriteNullValueAsString
   JSONProviderTest.testWriteNullValueAsNull
   passed with jdk9 + jaxb-core 2.3,  ensure empty string can call into 
out.write is the key part to reverse the behavior changed in both jdk9 and 
jaxb-core 2.3.
   And I agree that we should add Extended EscapeHandler in cxf-core module, as 
it has jaxb-core dependency and even a jaxb package there already.
   
   Append a patch what I got so far, this is to fix JSONProviderTest failure 
with jdk9 oriented so it didn't include your fix in jaxb databinding module
   
[CXF-7520-20180115.txt](https://github.com/apache/cxf/files/1631328/CXF-7520-20180115.txt)
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Ensure CXF can build with JDK9 GA(build 9+181)
> ----------------------------------------------
>
>                 Key: CXF-7520
>                 URL: https://issues.apache.org/jira/browse/CXF-7520
>             Project: CXF
>          Issue Type: Bug
>            Reporter: Freeman Fang
>            Assignee: Freeman Fang
>            Priority: Major
>             Fix For: 3.2.2
>
>
> a couple of cxf test failied with JDK9 GA due to recent change
> Mainly in JAXB and SAAJ



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to