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

CornerPinSurface has a huge bug #5

Open
clankill3r opened this issue Dec 12, 2019 · 0 comments
Open

CornerPinSurface has a huge bug #5

clankill3r opened this issue Dec 12, 2019 · 0 comments

Comments

@clankill3r
Copy link

https://github.com/davidbouchard/keystone/blob/master/src/deadpixel/keystone/CornerPinSurface.java#L53

The following values can not be static!

// Daniel Wiedeman: made them public static
public static int TL; // top left
public static int TR; // top right
public static int BL; // bottom left
public static int BR; // bottom right

Let's say I create a CornerPinSurface with a low resolution and after that I create a second CornerPinSurface with a high resolution. TL, TR, BL and BR are overwritten!
They should just be public, nothing more.

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

No branches or pull requests

1 participant