-
Notifications
You must be signed in to change notification settings - Fork 0
2.0.0 🚀 : #11
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
base: 2.x
Are you sure you want to change the base?
Conversation
import java.nio.IntBuffer; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.overrun.glutils.util.math.Maths.ceilDiv; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL11.*; | ||
|
||
/** | ||
* @author Erik Duijs |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL11.*; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL12.*; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.system.MemoryUtil.memFree; | ||
import static org.overrun.glutils.tex.Images.getRGB; | ||
import static org.overrun.glutils.tex.TexParam.glLinear; | ||
import static org.overrun.glutils.tex.TexParam.glNearest; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static java.util.Objects.requireNonNull; | ||
import static org.lwjgl.stb.STBImage.stbi_failure_reason; | ||
import static org.overrun.glutils.FilesReader.getInputStream; | ||
import static org.overrun.glutils.GLUtils.getLogger; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
|
||
/** | ||
* Some methods for calling GL functions quickly. | ||
* | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL12.*; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL11.glDeleteTextures; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.overrun.glutils.tex.Images.loadFsAwt; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import java.util.StringJoiner; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import java.util.StringJoiner; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
|
||
import java.util.StringJoiner; | ||
|
||
import static java.lang.Integer.compare; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
import static org.lwjgl.opengl.GL11.*; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
package org.overrun.glutils; | ||
package org.overrun.glutils.tex.stitch; | ||
|
||
import java.util.Map; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
public float fps = 0; | ||
public float passedTime = 0; | ||
/** | ||
* @see #passedTime |
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.
MissingSummary: A summary fragment is required; consider using the value of the @see block as a summary fragment instead. (details)
(at-me in a reply with help
or ignore
)
package org.overrun.glutils.timer; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
package org.overrun.glutils.callback; | ||
package org.overrun.glutils.timer; | ||
|
||
import java.util.function.Supplier; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
|
||
/** | ||
* compatibility layer | ||
* | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
@@ -23,7 +23,7 @@ | |||
* | |||
*/ | |||
|
|||
package org.overrun.glutils; | |||
package org.overrun.glutils.util; | |||
|
|||
/** | |||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
package org.overrun.glutils.util; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
package org.overrun.glutils.util; | ||
|
||
/** | ||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
@@ -23,19 +23,21 @@ | |||
* | |||
*/ | |||
|
|||
package org.overrun.glutils.callback; | |||
package org.overrun.glutils.util.math; | |||
|
|||
/** | |||
* @author squid233 |
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.
MissingSummary: A summary line is required on public/protected Javadocs. (details)
(at-me in a reply with help
or ignore
)
|
||
int level = 0; | ||
while (!done) { | ||
if (image != data) { |
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.
ReferenceEquality: Comparison using reference equality instead of value equality (details)
(at-me in a reply with help
or ignore
)
if (cb != null) { | ||
cb.free(); | ||
} | ||
bufFrame = framebuffer = new Framebuffer((hWnd, width, height) -> |
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.
StaticAssignmentInConstructor: This assignment is to a static field. Mutating static state from a constructor is highly error-prone. (details)
(at-me in a reply with help
or ignore
)
@@ -46,15 +45,15 @@ public static void drawText(String text, | |||
FontTexture font, | |||
@Nullable ColorFunction fgColor, | |||
boolean bottomToTop) { | |||
String[] ln = text.split("[\\r\\n]"); | |||
var ln = text.split("[\\r\\n]"); |
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.
StringSplitter: String.split(String) has surprising behavior (details)
(at-me in a reply with help
or ignore
)
No description provided.