History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: WSTX-122
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Tatu Saloranta
Reporter: Tatu Saloranta
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Woodstox

Add support for EBCDIC

Created: 09/Jun/07 12:17 AM   Updated: 31/Oct/07 02:39 PM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified


 Description  « Hide
Currently Woodstox only detects that input appears to be EBCDIC (as per guidance from xml specifications encoding handling), but does not try to handle it but throws a specific exception. This is since it wasn't known whether (a) JDK supports EBCDIC, and/or (b) if anyone really needs support. (b) is no longer true (a user has requested support), and it sounds like JDK does have some level of support. If so, adding EBCDIC handling might be easy to do.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Tatu Saloranta - 15/Sep/07 01:48 AM
Ok, so the magic encoding name in JDK seems to be "cp037"... let's see if I can add support easily.

Tatu Saloranta - 15/Sep/07 02:00 AM
Looks like a relatively simple changes to use cp037, along with minimal auto-detection, works ok with simple sample EBCDIC files I have access to. It is hard to evaluate if that is a proper solution, though, without more extensive test document suite.

At any rate, I'll add support in trunk: could backport this to 3.2 too if necessary.


Tatu Saloranta - 31/Oct/07 02:38 PM
Added EBCDIC support to trunk, backported to 3.2 branch. Simple baseline unit tests added, seems to work. User that requested feature also did some testing, so I think this can be resolved.

Tatu Saloranta - 31/Oct/07 02:39 PM
Released in 3.2.2, will be part of 4.0. Closing.