You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am working on adding support for CSSHPB to Nitro but running into a significant issue in completing it.
The CSSHPB tag utilizes the DEVVER in the DES Subheader to determine how some fields should be parsed. The current TRECursor does not have access to this field as nitf_TRECursor_getTREPair only has access to the nitf_TRE object.
I am currently thinking of adding to nitf_TRE an optional pointer to a nitf_DESubheader then modifying nitf_TRECursor_getTREPair to lookup subheader fields such as DESVER if it fails to find values in the hash allowing it to be used in NITF_IF cases.
The des sub header would be populated by adding an optional additional argument to nitf_TRE_construct and nitf_TRE_createSkeleton of nitf_DESubheader
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
I am working on adding support for
CSSHPBto Nitro but running into a significant issue in completing it.The
CSSHPBtag utilizes theDEVVERin the DES Subheader to determine how some fields should be parsed. The currentTRECursordoes not have access to this field asnitf_TRECursor_getTREPaironly has access to thenitf_TREobject.I am currently thinking of adding to
nitf_TREan optional pointer to anitf_DESubheaderthen modifyingnitf_TRECursor_getTREPairto lookup subheader fields such asDESVERif it fails to find values in the hash allowing it to be used inNITF_IFcases.The des sub header would be populated by adding an optional additional argument to
nitf_TRE_constructandnitf_TRE_createSkeletonofnitf_DESubheaderSo
nitro/modules/c/nitf/source/NitfReader.c
Line 866 in 298536f
Would change to:
I would appreciate the communities input on this.
Beta Was this translation helpful? Give feedback.
All reactions