Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gibberish in cadinfo output #29

Open
drnextgis opened this issue Aug 26, 2016 · 3 comments
Open

gibberish in cadinfo output #29

drnextgis opened this issue Aug 26, 2016 · 3 comments

Comments

@drnextgis
Copy link

drnextgis commented Aug 26, 2016

Is it ok?

screenshot from 2016-08-27 05 36 20

@BishopGIS
Copy link
Contributor

BishopGIS commented Aug 26, 2016

cadinfo just returns strings as they stored in DWG file. No encoding released. But it worth to add this in cadinfo. It seems to me that this issue should be marked as enhancement.

@sandyre
Copy link
Owner

sandyre commented Aug 27, 2016

Library has a way to get what encoding is presented, but adding additional recoding to UTF8 (as an example) should be a good enhancement, since library calling application wont need to use its own recoding algorithms (as GDAL CAD Driver does now)

@BishopGIS
Copy link
Contributor

BishopGIS commented Aug 27, 2016

But I meant to recode in application - not in library. Most software already have some implementations of strings recoding. So using library the software need to use not own but library recording, which duplicates functionality and limits developer.
Adding iconv is enough here. It can be added the same way as GTest did - using NextGIS Borsh scripts (see https://github.com/nextgis-borsch/lib_iconv and usage example https://github.com/nextgis-borsch/lib_xml2/blob/master/CMakeLists.txt#L106)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants