Skip to content

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

Draft
wants to merge 44 commits into
base: 2.x
Choose a base branch
from
Draft

2.0.0 🚀 : #11

wants to merge 44 commits into from

Conversation

squid233
Copy link
Member

@squid233 squid233 commented Jan 8, 2022

No description provided.

import java.nio.IntBuffer;

/**
* @author squid233
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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) {
Copy link

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) ->
Copy link

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]");
Copy link

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant