search

sandyre / libopencad

OpenSource library under X/MIT license to work with CAD files (DWG/DXF). Please use repository linked below, this one is abandoned.
https://github.com/nextgis-borsch/lib_opencad
MIT License
197 stars 63 forks source link

gibberish in cadinfo output #29

Open drnextgis opened 8 years ago

drnextgis commented 8 years ago

Is it ok?

screenshot from 2016-08-27 05 36 20

BishopGIS commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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)