OpenSystemsDevelopment / qpdf

Fixes #1532.

…df::Dictionary`, update constructors, and streamline logic with `Dictionary` API.

…simplify iteration logic, update to use `Dictionary` API, and streamline error checks.

…with subscript operator, simplify logic, and improve error checks.

…h subscript operator, streamline `repair` with `Dictionary` initializer, and remove redundant semicolon.

…access, and streamline logic.

…y` with subscript operator, and simplify logic.

…`getKey` with subscript operator, and simplify logic.

…ictionary` APIs, replace `getKey` with subscript operator, streamline logic, and simplify bounds checks.

…y, and simplify implementation.

…erator[]`, and simplify null handling logic.

…actor implementation to simplify null handling and align with API conventions.

…ctionary key access, and simplify null handling logic.

- Added `Array::empty()` as a static method for creating empty arrays.
- Removed unused `Array(bool empty)` constructor and defaulted `Array` constructor.
- Updated `NNTree` methods to use `Array::empty()` for clarity and consistency.

…s and `Members` structure

- Simplified `Members` implementation by inlining into `NNTree` context.
- Replaced `std::shared_ptr<Members>` with `std::make_shared` for improved clarity and usage consistency.
- Updated method calls to directly access `Members`' `impl`.
- Improved maintainability and alignment with modern C++ practices.

…idate into `NNTree` implementation

- Merged `QPDFNameTreeObjectHelper` and `QPDFNumberTreeObjectHelper` into `NNTree` source for simplification.
- Adjusted `CMakeLists.txt` and headers to reflect changes.

- Marked `NNTreeIterator` and `NNTreeImpl` as `final` to prevent inheritance.
- Inlined `NNTreeIterator` and `NNTreeImpl` methods to streamline code and reduce redundancy.
- Updated constructors, operators, and utility functions in `NNTreeIterator` and `NNTreeImpl` for improved readability and maintainability.

- Added `const` qualifiers to member functions and parameters where applicable.
- Replaced `QPDFObjectHandle` arrays with `qpdf::Array` for improved type clarity.
- Simplified logic in `compareKeyItem` and `compareKeyKid` by removing redundant checks.

- Simplified the code by directly handling limits validation in `compareKeyKid`.
- Improved clarity by reducing unnecessary method indirection.

- Removed `NNTreeDetails` and moved key validation logic directly into `NNTreeImpl`.
- Replaced type determination with `qpdf_object_type_e` for improved clarity.
- Updated `QPDFNameTreeObjectHelper` and `QPDFNumberTreeObjectHelper` to align with the new `NNTreeImpl` structure.
- Simplified methods accessing tree items and validating keys.

…e memory management, and streamline class definition.

…e()` for re-validation and repair of AcroForm structure.

…()` method, remove redundant helper functions, and streamline related logic for consistency and maintainability.

…ated `acroform()` method and update related references for improved performance, consistency and maintainability.

… validation, improve readability, and reduce redundancy in type checks.

…for improved readability, and remove redundant variables.

…king `validate()` for `names_dest` initialization

…s_dest`, improve memory management, and streamline initialization logic

- Added `validate()` invocation to `m->embedded_files`.
- Introduced `value_validator` to ensure only dictionaries are accepted in `EmbeddedFiles`.
- Enabled consistent value validation and key validation logic.

…rove memory management, and simplify initialization logic.

… in damaged label tests

- Add `validate()` call to `QPDFNumberTreeObjectHelper` for page label validation.
- Introduce stricter enforcement of dictionary values and validation logic.
- Update damaged page label test to remove `EXPECT_FAILURE`.

…fy logic, and enhance consistency with modern C++ practices.

- Introduced a `validate()` method in `QPDFNameTreeObjectHelper`, `QPDFNumberTreeObjectHelper`, and `NNTreeImpl`.
- Ensure proper validation of tree keys and values, detecting invalid entries, unsorted keys, and inconsistencies.
- Added support for auto-repair in case of validation errors.

- Introduced `std::function<bool(QPDFObjectHandle const&)>` to validate tree values.
- Updated constructors and methods across `QPDFNameTreeObjectHelper` and `QPDFNumberTreeObjectHelper` to support value validation.
- Refactored `NNTreeImpl` to check for invalid values during insertion, iteration, and retrieval.
- Improved error handling and reporting for invalid values.

Fixes bug in #1523

Fixes bug in #1523