Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IFFPdu improvements #5 and Unsafe equalsImpl for object list attributes #8 #15

Merged
merged 5 commits into from
Dec 20, 2023

Conversation

mimmi-matalamaki-insta-fi
Copy link
Contributor

@mimmi-matalamaki-insta-fi mimmi-matalamaki-insta-fi commented Nov 29, 2023

We have modified IFFPdu that all five layers (1-5) can be added to IFFPdu.

Also fixed:

We have fixed issue:

@mimmi-matalamaki-insta-fi mimmi-matalamaki-insta-fi changed the title Iff pdu improvements #6 and Unsafe equalsImpl for object list attributes #8 IFFPdu improvements #5 and Unsafe equalsImpl for object list attributes #8 Nov 29, 2023
@brutzman
Copy link
Contributor

brutzman commented Dec 11, 2023

Thank you very much for this careful modification.

@terry-norbraten and I are looking at it closely, since it will be our first effort to handle a pull request in this library.

We expect to clone your branch of opendis7-source-generator changes to test before accepting. We will also run your new test class in opendis7-java: test/IFFPduTest.java. If everything works, we will merge those changes (and not the other opendis7-java source changes, which should get autogenerated). Otherwise we will provide further questions/suggestions.

Am expecting we might rename Mode5StatusRecord class as IFFMode5StatusRecord but that step can wait until everything else works.

We will also be looking at available string constants for PDU names, in order to avoid potential typos.

Your professional and patience is appreciated, we will ask questions if we have them.

@terry-norbraten terry-norbraten merged commit 67b7862 into open-dis:master Dec 20, 2023
@terry-norbraten
Copy link
Collaborator

Special thanks to mimmi-matalamaki-insta-fi and to arttu-salminen-insta-fi for the IFFPdu layers expansion and the other mentioned fixes for #8, #11 and #12.

@mimmi-matalamaki-insta-fi
Copy link
Contributor Author

We are happy that you merged our pull-request to part of your master branch code and that we have managed to bring value of this project.

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.

4 participants