Branch: refs/heads/master Home: https://github.com/Starlink/starjava Commit: 255fb30c84b8817949d091e26c46f6a55af5a76f https://github.com/Starlink/starjava/commit/255fb30c84b8817949d091e26c46f6a55af5a76f Author: Mark Taylor <[log in to unmask]> Date: 2014-08-12 (Tue, 12 Aug 2014) Changed paths: M ttools/src/testcases/uk/ac/starlink/ttools/task/MultiCopyTest.java M ttools/src/testcases/uk/ac/starlink/ttools/task/VotCopyTest.java M ttools/src/testcases/uk/ac/starlink/ttools/task/multi.vot Log Message: ----------- ttools: improve unit tests for unicode in VOTable Adjust the unit tests for the votcopy task to check that columns with datatype="unicodeChar" are encoded properly in the various VOTable output serializations. In the existing code they are not, leading to a complete read failure. Commit: d40fc77649a35a4a99d1a3e9af80997483f7c744 https://github.com/Starlink/starjava/commit/d40fc77649a35a4a99d1a3e9af80997483f7c744 Author: Mark Taylor <[log in to unmask]> Date: 2014-08-12 (Tue, 12 Aug 2014) Changed paths: M topcat/src/docs/sun253.xml M ttools/src/docs/sun256.xml M ttools/src/main/uk/ac/starlink/ttools/copy/VotCopyHandler.java M votable/src/main/uk/ac/starlink/votable/Encoder.java M votable/src/main/uk/ac/starlink/votable/VOSerializer.java Log Message: ----------- votable: fix/improve output of unicodeChar columns When using votcopy to write a BINARY or BINARY2 serialized VOTable in which some (non-empty) columns have datatype="unicodeChar", there was a serious bug which meant the output was inconsistent with the FIELD declaration (FIELD said datatype="unicodeChar", but the output stream used 1-byte characters). This rendered the output VOTable unreadable (reported by Peter Draper). Fixed. Some other slight improvement to handling of unicodeChar columns too: columns that originally had datatype="unicodeChar" (originating from a VOTable) are now written out with the same datatype, rather than being squashed to type char. However, it's still a bit of a mess, partly because unicode handling is sloppy in STIL in general, and partly because VOTable's unicodeChar datatype value doesn't really make sense (VOTable 1.3 and earlier). The first issue may require more work somewhere down the line, e.g. STIL becoming aware of whether columns are Unicode or ASCII-alike, at least for output. The second has been acknowledged in recent VOTable discussions, and may be resolved or improved in a future version of the VOTable standard. Compare: https://github.com/Starlink/starjava/compare/a3fb3f770ca7...d40fc77649a3