Skip to content

Add support for multiple AutoCAD versions - #27

Merged
marevol merged 3 commits into
masterfrom
claude/add-autocad-version-support-011CUwVLNsqfRNQ7sg4nAH8R
Nov 9, 2025
Merged

marevol merged 3 commits into
masterfrom
claude/add-autocad-version-support-011CUwVLNsqfRNQ7sg4nAH8R

Conversation

@marevol

@marevol marevol commented Nov 9, 2025

Copy link
Copy Markdown
Contributor
  • Add AC1032 enum to DRW::Version in drw_base.h
  • Create dwgReader32 class following existing pattern (extends dwgReader18)
  • Add AC1032 string mapping in libdwgr.cpp
  • Update DXF header handling to support AC1032
  • Add version tests for R2013 (AC1027) and R2018 (AC1032)
  • All existing tests pass successfully

The implementation follows the established pattern where newer versions (AC1024, AC1027, AC1032) extend dwgReader18 with minimal version-specific overrides. Text encoding automatically uses UTF-8/UTF-16 for modern versions.

- Add AC1032 enum to DRW::Version in drw_base.h
- Create dwgReader32 class following existing pattern (extends dwgReader18)
- Add AC1032 string mapping in libdwgr.cpp
- Update DXF header handling to support AC1032
- Add version tests for R2013 (AC1027) and R2018 (AC1032)
- All existing tests pass successfully

The implementation follows the established pattern where newer versions
(AC1024, AC1027, AC1032) extend dwgReader18 with minimal version-specific
overrides. Text encoding automatically uses UTF-8/UTF-16 for modern versions.
- Add dwgreader32.h to dist_noinst_HEADERS
- Add dwgreader32.cpp to libdxfrw_la_SOURCES

This completes AC1032 (AutoCAD 2018) support for Autotools-based builds.
Regenerate configure and Makefile.in files to include dwgreader32
sources and headers in the Autotools build system. This ensures both
CMake and Autotools builds work correctly with AC1032 support.
@marevol
marevol merged commit 0675f06 into master Nov 9, 2025
4 checks passed
@marevol
marevol deleted the claude/add-autocad-version-support-011CUwVLNsqfRNQ7sg4nAH8R branch February 5, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants