aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--pugl/detail/win.c26
-rw-r--r--pugl/detail/x11.c2
-rw-r--r--pugl/detail/x11_gl.c2
-rw-r--r--wscript14
4 files changed, 26 insertions, 18 deletions
diff --git a/pugl/detail/win.c b/pugl/detail/win.c
index 953c620..cd7a781 100644
--- a/pugl/detail/win.c
+++ b/pugl/detail/win.c
@@ -395,35 +395,35 @@ constrainAspect(const PuglView* const view,
RECT* const size,
const WPARAM wParam)
{
- const float minAspect = view->min_aspect_x / (float)view->min_aspect_y;
- const float maxAspect = view->max_aspect_x / (float)view->max_aspect_y;
- const int w = size->right - size->left;
- const int h = size->bottom - size->top;
- const float a = w / (float)h;
+ const float minA = (float)view->min_aspect_x / (float)view->min_aspect_y;
+ const float maxA = (float)view->max_aspect_x / (float)view->max_aspect_y;
+ const int w = size->right - size->left;
+ const int h = size->bottom - size->top;
+ const float a = (float)w / (float)h;
switch (wParam) {
case WMSZ_TOP:
- size->top = (a < minAspect ? (LONG)(size->bottom - w * minAspect) :
- a > maxAspect ? (LONG)(size->bottom - w * maxAspect) :
+ size->top = (a < minA ? (LONG)(size->bottom - w * minA) :
+ a > maxA ? (LONG)(size->bottom - w * maxA) :
size->top);
break;
case WMSZ_TOPRIGHT:
case WMSZ_RIGHT:
case WMSZ_BOTTOMRIGHT:
- size->right = (a < minAspect ? (LONG)(size->left + h * minAspect) :
- a > maxAspect ? (LONG)(size->left + h * maxAspect) :
+ size->right = (a < minA ? (LONG)(size->left + h * minA) :
+ a > maxA ? (LONG)(size->left + h * maxA) :
size->right);
break;
case WMSZ_BOTTOM:
- size->bottom = (a < minAspect ? (LONG)(size->top + w * minAspect) :
- a > maxAspect ? (LONG)(size->top + w * maxAspect) :
+ size->bottom = (a < minA ? (LONG)(size->top + w * minA) :
+ a > maxA ? (LONG)(size->top + w * maxA) :
size->bottom);
break;
case WMSZ_BOTTOMLEFT:
case WMSZ_LEFT:
case WMSZ_TOPLEFT:
- size->left = (a < minAspect ? (LONG)(size->right - h * minAspect) :
- a > maxAspect ? (LONG)(size->right - h * maxAspect) :
+ size->left = (a < minA ? (LONG)(size->right - h * minA) :
+ a > maxA ? (LONG)(size->right - h * maxA) :
size->left);
break;
}
diff --git a/pugl/detail/x11.c b/pugl/detail/x11.c
index 70341ea..8b4c136 100644
--- a/pugl/detail/x11.c
+++ b/pugl/detail/x11.c
@@ -452,7 +452,7 @@ puglRequestAttention(PuglView* view)
root,
False,
SubstructureNotifyMask | SubstructureRedirectMask,
- (XEvent*)&event);
+ &event);
}
PuglStatus
diff --git a/pugl/detail/x11_gl.c b/pugl/detail/x11_gl.c
index 7fcf169..929d3a5 100644
--- a/pugl/detail/x11_gl.c
+++ b/pugl/detail/x11_gl.c
@@ -25,8 +25,8 @@
#include <GL/gl.h>
#include <GL/glx.h>
-#include <stdlib.h>
#include <stdio.h>
+#include <stdlib.h>
typedef struct {
GLXFBConfig fb_config;
diff --git a/wscript b/wscript
index e239f36..582d306 100644
--- a/wscript
+++ b/wscript
@@ -207,13 +207,21 @@ def lint(ctx):
shell=True)
cmd = ("clang-tidy -p=. -header-filter=.* -checks=\"*," +
+ "-bugprone-suspicious-string-compare," +
"-clang-analyzer-alpha.*," +
+ "-cppcoreguidelines-avoid-magic-numbers," +
"-google-readability-todo," +
+ "-hicpp-multiway-paths-covered," +
+ "-hicpp-signed-bitwise," +
+ "-hicpp-uppercase-literal-suffix," +
"-llvm-header-guard," +
+ "-misc-misplaced-const," +
"-misc-unused-parameters," +
- "-hicpp-signed-bitwise," + # FIXME?
- "-readability-else-after-return\" " +
- "../pugl/detail/*.c ../test/*.c")
+ "-readability-else-after-return," +
+ "-readability-magic-numbers," +
+ "-readability-uppercase-literal-suffix\" " +
+ "../pugl/detail/*.c")
+
subprocess.call(cmd, cwd='build', shell=True)
# Alias .m files to be compiled like .c files, gcc will do the right thing.