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.

Sign up for GitHub

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

Jump to bottom

cleanup/nullable-ize IField-derived classes #888

Merged

gbirchmeier merged 2 commits into connamara:master from gbirchmeier:field-class-cleanup

Sep 25, 2024

Member

gbirchmeier commented Sep 25, 2024

Finished cleaning up all the nullable-enable warnings in the "QuickFIX" csproj!

deprecate lower-case-starting function names (renamed to upper-case-starting)
deprecate Field.Obj (renamed to Value)
deprecate Field.getValue/setValue (just use Value getter/setter)
major cleanup of MessageTests.cs
started cleaning up SessionTests.cs, but that needs more work

gbirchmeier added 2 commits

September 25, 2024 13:40


          cleanup IField-derived classes

b6ab1b5


          delete "#nullable enable" from QuickFIX proj files

8d14aa6

gbirchmeier merged commit c28b406 into connamara:master

2 checks passed

gbirchmeier deleted the field-class-cleanup branch

September 25, 2024 20:09

gbirchmeier mentioned this pull request

fix Examples deprecations & nullable-enables #889

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet