Conversation
Parts not done (flagging as follow-up, not silently skipped): - `dmfcore_parse_finish_cb`-level "reject whole file" behavior for the strict policy isn't wired yet - today the policy only changes logging verbosity at the existing call sites, not the control flow. - DMF XML keyword for referencing `dmf_stdlib` methods by name (e.g. `conversion="scale_format"`) isn't parsed by `dmfsnmp.c` yet - the registry exists, but nothing in the XML grammar consumes it. - old functions in `snmp-ups-helpers.c` moved up to `dmf_stdlib.c`, but not yet removed from the former file, to avoid breaking existing code until the new functions are fully wired in. - probably more mapping subdrivers can be converted to use the new `dmf_stdlib` functions, but that is left for follow-up work. Co-authored-by: Claude Sonnet Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…allocating based on it Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…rop) Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
jimklimov
added
enhancement
DMF
|
A ZIP file with standard source tarball and another tarball with pre-built docs for commit 729d37f is temporarily available: NUT-tarballs-PR-3614.zip. |
jimklimov
marked this pull request as draft
September 2, 2026 21:19
jimklimov
added
the
AI
jimklimov
changed the title
|
❌ Build nut 2.8.5.5201-FTY failed (commit 1fa6146874 by @jimklimov) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
After some analysis, a lot of active data manipulations can be done by relatively few methods predefined in C and used in classic mappings same as in DMF variants -- to the point that many use-cases would not employ LUA at all.
This PR reshuffles related code and recipes to aid in this effort (there would be more work to actually benefit from this in XML DMF handling). Burned through a month's allowance in CoPilot already in the first couple of days, and another in the last day of August, so maybe will follow up much later or find another provider to pick up the torch. Still, thanks to GitHub for sponsoring maintainers to have an allowance at all.
This PR also fixes a couple of potential memory errors in older DMF code, found while CoPilot read through the feature code.