-
Notifications
You must be signed in to change notification settings - Fork 522
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
Remove System.Drawing.Imaging references from ArtData #5080
base: p58-wip
Are you sure you want to change the base?
Conversation
} | ||
|
||
public static Rectangle MeasureBound(Bitmap bmp) | ||
public static Rectangle MeasureBound(ArtInfo bmp) |
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.
You may want to rename argument to something else as it's no longer bitmap bmp
{ | ||
Measure(bmp, out var xMin, out var yMin, out var xMax, out var yMax); | ||
|
||
return new Rectangle(xMin, yMin, xMax - xMin, yMax - yMin); | ||
} | ||
|
||
public static Size MeasureSize(Bitmap bmp) | ||
public static Size MeasureSize(ArtInfo bmp) |
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.
You may want to rename argument to something else as it's no longer bitmap bmp
@@ -1221,7 +1221,7 @@ public void SetVisibility(Mobile m, bool? state) | |||
} | |||
} | |||
|
|||
public static Bitmap GetBitmap(int itemID) | |||
public static ArtInfo GetBitmap(int itemID) |
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.
Is it really GetBitmap() at this point? :)
Removed reference to System.Drawing.Imaging from ArtData.
Simplified it to only read dimensions and bounds (no access to pixel colors)