aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--include/mgl/gl.h1
-rw-r--r--include/mgl/gl_macro.h2
-rw-r--r--src/gl.c1
-rw-r--r--src/graphics/font.c14
-rw-r--r--src/graphics/image.c15
-rw-r--r--src/graphics/texture.c4
-rw-r--r--src/window/window.c2
-rw-r--r--tests/X11.jpgbin0 -> 19567 bytes
-rw-r--r--tests/main.c4
9 files changed, 30 insertions, 13 deletions
diff --git a/include/mgl/gl.h b/include/mgl/gl.h
index c9a709d..1d19339 100644
--- a/include/mgl/gl.h
+++ b/include/mgl/gl.h
@@ -27,7 +27,6 @@ typedef struct {
void (*glGenTextures)(int n, unsigned int *textures);
void (*glDeleteTextures)(int n, const unsigned int *textures);
void (*glTexImage2D)(unsigned int target, int level, int internalFormat, int width, int height, int border, unsigned int format, unsigned int type, const void *pixels);
- void (*glCompressedTexImage2D)(unsigned int target, int level, unsigned int internalformat, int width, int height, int border, int imageSize, const void *data);
void (*glBindTexture)(unsigned int target, unsigned int texture);
void (*glTexParameteri)(unsigned int target, unsigned int pname, int param);
void (*glHint)(unsigned int target, unsigned int mode);
diff --git a/include/mgl/gl_macro.h b/include/mgl/gl_macro.h
index c47c8fb..aa264ee 100644
--- a/include/mgl/gl_macro.h
+++ b/include/mgl/gl_macro.h
@@ -31,6 +31,8 @@
#define GL_UNSIGNED_BYTE 0x1401
#define GL_ALPHA 0x1906
+#define GL_LUMINANCE 0x1909
+#define GL_LUMINANCE_ALPHA 0x190A
#define GL_RGB 0x1907
#define GL_RGBA 0x1908
diff --git a/src/gl.c b/src/gl.c
index c54bbbf..efcfbd4 100644
--- a/src/gl.c
+++ b/src/gl.c
@@ -42,7 +42,6 @@ int mgl_gl_load(mgl_gl *self) {
{ &self->glGenTextures, "glGenTextures" },
{ &self->glDeleteTextures, "glDeleteTextures" },
{ &self->glTexImage2D, "glTexImage2D" },
- { &self->glCompressedTexImage2D, "glCompressedTexImage2D" },
{ &self->glBindTexture, "glBindTexture" },
{ &self->glTexParameteri, "glTexParameteri" },
{ &self->glHint, "glHint" },
diff --git a/src/graphics/font.c b/src/graphics/font.c
index 2ed54d2..c017755 100644
--- a/src/graphics/font.c
+++ b/src/graphics/font.c
@@ -10,6 +10,11 @@
/* TODO: Test and fix .tcc files */
+static unsigned int to_div2_ceil(unsigned int value) {
+ const uint32_t v = value;
+ return v + (v & 1);
+}
+
int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int character_size) {
self->texture.id = 0;
@@ -47,8 +52,13 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
/* TODO: Optimize */
/* Find optimal size for atlas, starting from small to large */
for(int i = 0; i < 4; ++i) {
- self->font_atlas.width = (14 + (8 * i)) * self->character_size;
- self->font_atlas.height = (14 + (8 * i)) * self->character_size;
+ /*
+ This to_div2_ceil is needed because otherwise for character sizes such as 33 which are not divisable by 2
+ causes the font texture to be skewed. I dont know why this happens. Maybe a bug in stbtt?
+ TODO: Figure out why it happens.
+ */
+ self->font_atlas.width = (14 + (8 * i)) * to_div2_ceil(self->character_size);
+ self->font_atlas.height = (14 + (8 * i)) * to_div2_ceil(self->character_size);
unsigned char *new_atlas = realloc(self->font_atlas.atlas, self->font_atlas.width * self->font_atlas.height);
if(!new_atlas) {
fprintf(stderr, "Error: failed to load font %s, error: out of memory\n", filepath);
diff --git a/src/graphics/image.c b/src/graphics/image.c
index 593a3d2..9934b42 100644
--- a/src/graphics/image.c
+++ b/src/graphics/image.c
@@ -1,4 +1,5 @@
#include "../../include/mgl/graphics/image.h"
+#include <assert.h>
#define STBI_NO_PSD
#define STBI_NO_TGA
@@ -19,7 +20,8 @@ static mgl_image_format stbi_format_to_mgl_image_format(int stbi_format) {
case STBI_rgb_alpha:
return MGL_IMAGE_FORMAT_RGBA;
}
- return 0;
+ assert(0);
+ return MGL_IMAGE_FORMAT_RGBA;
}
static size_t mgl_image_format_num_channels(mgl_image_format image_format) {
@@ -30,7 +32,8 @@ static size_t mgl_image_format_num_channels(mgl_image_format image_format) {
case MGL_IMAGE_FORMAT_RGB: return 3;
case MGL_IMAGE_FORMAT_RGBA: return 4;
}
- return 0;
+ assert(0);
+ return 4;
}
/* TODO: Ensure texture is power of 2 if the hardware doesn't support non power of two textures */
@@ -41,12 +44,14 @@ int mgl_image_load_from_file(mgl_image *self, const char *filepath) {
self->height = 0;
int format;
- self->data = stbi_load(filepath, &self->width, &self->height, &format, 0);
+ /* TODO: format is forced to rgba right now because rgb images cause the image to be skewed (bug) */
+ self->data = stbi_load(filepath, &self->width, &self->height, &format, 4);
if(!self->data) {
fprintf(stderr, "Error: failed to load image %s, error: %s\n", filepath, stbi_failure_reason());
mgl_image_unload(self);
return -1;
}
+ format = 4;
self->format = stbi_format_to_mgl_image_format(format);
return 0;
@@ -60,12 +65,14 @@ int mgl_image_load_from_memory(mgl_image *self, const unsigned char *data, size_
self->height = 0;
int format;
- self->data = stbi_load_from_memory(data, size, &self->width, &self->height, &format, 0);
+ /* TODO: format is forced to rgba right now because rgb images cause the image to be skewed (bug) */
+ self->data = stbi_load_from_memory(data, size, &self->width, &self->height, &format, 4);
if(!self->data) {
fprintf(stderr, "Error: failed to load image from memory, error: %s\n", stbi_failure_reason());
mgl_image_unload(self);
return -1;
}
+ format = 4;
self->format = stbi_format_to_mgl_image_format(format);
return 0;
diff --git a/src/graphics/texture.c b/src/graphics/texture.c
index c70c0f9..f6549f1 100644
--- a/src/graphics/texture.c
+++ b/src/graphics/texture.c
@@ -30,8 +30,8 @@ static int mgl_texture_format_to_compressed_opengl_format(mgl_texture_format for
static int mgl_texture_format_to_source_opengl_format(mgl_texture_format format) {
switch(format) {
case MGL_TEXTURE_FORMAT_ALPHA: return GL_ALPHA;
- case MGL_TEXTURE_FORMAT_GRAY: return GL_LUMINANCE8;
- case MGL_TEXTURE_FORMAT_GRAY_ALPHA: return GL_LUMINANCE8_ALPHA8;
+ case MGL_TEXTURE_FORMAT_GRAY: return GL_LUMINANCE;
+ case MGL_TEXTURE_FORMAT_GRAY_ALPHA: return GL_LUMINANCE_ALPHA;
case MGL_TEXTURE_FORMAT_RGB: return GL_RGB;
case MGL_TEXTURE_FORMAT_RGBA: return GL_RGBA;
}
diff --git a/src/window/window.c b/src/window/window.c
index f975c1e..30abebb 100644
--- a/src/window/window.c
+++ b/src/window/window.c
@@ -158,6 +158,7 @@ static mgl_key x11_keysym_to_mgl_key(KeySym key_sym) {
case XK_BackSpace: return MGL_KEY_BACKSPACE;
case XK_Tab: return MGL_KEY_TAB;
case XK_Return: return MGL_KEY_ENTER;
+ case XK_Escape: return MGL_KEY_ESCAPE;
case XK_Delete: return MGL_KEY_DELETE;
case XK_Home: return MGL_KEY_HOME;
case XK_Left: return MGL_KEY_LEFT;
@@ -243,7 +244,6 @@ static bool mgl_window_on_receive_event(mgl_window *self, XEvent *xev, mgl_event
event->type = MGL_EVENT_RESIZED;
event->size.width = self->size.x;
event->size.height = self->size.y;
- event->mouse_move.y = self->cursor_position.y;
return true;
}
return false;
diff --git a/tests/X11.jpg b/tests/X11.jpg
new file mode 100644
index 0000000..95e4793
--- /dev/null
+++ b/tests/X11.jpg
Binary files differ
diff --git a/tests/main.c b/tests/main.c
index 878f8ff..d430ab2 100644
--- a/tests/main.c
+++ b/tests/main.c
@@ -32,7 +32,7 @@ static void draw(mgl_window *window, void *userdata) {
};
mgl_rectangle_draw(context, &rect);
- mgl_shader_program_set_uniform_vec2f(u->shader_program, "resolution", (mgl_vec2f){ window->size.x, window->size.y });
+ mgl_shader_program_set_uniform_vec2f(u->shader_program, "resolution", (mgl_vec2f){ u->texture->width, u->texture->height });
mgl_sprite sprite;
mgl_sprite_init(&sprite, u->texture, 100.0f - 10.0f, 0.0f);
@@ -117,7 +117,7 @@ int main(int argc, char **argv) {
if(mgl_window_create(&window, "mgl", 1280, 720) != 0)
return 1;
- if(mgl_texture_load_from_file(&texture, "tests/X11.png", NULL) != 0)
+ if(mgl_texture_load_from_file(&texture, "tests/X11.jpg", NULL) != 0)
return 1;
if(mgl_font_load_from_file(&font, "/usr/share/fonts/noto/NotoSans-Regular.ttf", 32) != 0)