-
-
Notifications
You must be signed in to change notification settings - Fork 963
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
refactor: FreeImage dependency update #2531
Conversation
There's a bunch of tabs/space mismatch in this PR, can't merge it as is sadly |
@@ -2790,14 +2779,12 @@ public TimeSpan? TimeStamp | |||
get | |||
{ | |||
FIURational[] stamp = GetTagArray<FIURational>("GPSTimeStamp"); | |||
if ((stamp == null) || stamp.Length != 3) | |||
if (stamp is not { Length: 3 }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this one is just more confusing than the previous one; the negative and positive are merged into the same comparison.
Besides those two comments it looks quite good, thanks @Jklawreszuk !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem , I think it is more readable but I understand that it can be confusing, so I reverted it 😀
140b4e8
to
e847bde
Compare
Huh... Maybe Rider uses spaces instead of tabs when I fixed the warnings. Now I formatted with VSCode, so I hope that will do |
Thanks ! |
PR Details
This PR mainly removes the duplicate FreeImageNET source code, which is not used anywhere.
In addition, I decided to refactor code and remove the useless compatibility with
System.Drawing.Bitmap
, which is also not used anywhere.Types of changes
Checklist