Merge pull request #28 from CleverRaven/master

BathroomTrueOct9

.clang-tidy

@@ -18,6 +18,7 @@ misc-*,\
 modernize-*,\
 performance-*,\
 readability-*,\
+-cata-text-style,\
 -bugprone-misplaced-widening-cast,\
 -bugprone-narrowing-conversions,\
 -bugprone-unused-return-value,\
@@ -61,5 +62,7 @@ FormatStyle:     none
 CheckOptions:
   - key: readability-uppercase-literal-suffix.NewSuffixes
     value: 'L;UL;LL;ULL'
+  - key: cata-text-style.EscapeUnicode
+    value: 0
 
 # vim:tw=0

.github/pull_request_template.md

@@ -50,6 +50,12 @@ The easier you make your solution to understand, the faster it can get merged.
 A clear and concise description of any alternative solutions or features you've considered.
 -->
 
+#### Testing
+<!--
+Describe what steps you took to test that this PR resolved the bug or added the feature, and what tests you performed to make sure it didn't cause any regressions.
+Also include testing suggestions for reviewers and maintainers.
+-->
+
 #### Additional context
 <!--
 Add any other context (such as mock-ups, proof of concepts or screenshots) about the feature or bugfix here.

Makefile

@@ -581,43 +581,43 @@ ifdef TILES
     ODIR = $(ODIRTILES)
   endif
 else
+  ifeq ($(LOCALIZE),1)
+    NCURSES_PREFIX = ncursesw
+  else
+    NCURSES_PREFIX = ncurses
+  endif
+  ifdef OSXCROSS
+    NCURSES_PREFIX = ncurses
+  endif
   # ONLY when not cross-compiling, check for pkg-config or ncurses5-config
   # When doing a cross-compile, we can't rely on the host machine's -configs
   ifeq ($(CROSS),)
-    ifneq ($(shell pkg-config --libs ncurses 2>/dev/null),)
-      HAVE_PKGCONFIG = 1
-    endif
-    ifneq ($(shell which ncurses5-config 2>/dev/null),)
-      HAVE_NCURSES5CONFIG = 1
-    endif
+      ifeq ($(OSXCROSS),)
+        ifneq ($(shell pkg-config --libs $(NCURSES_PREFIX) 2>/dev/null),)
+          HAVE_PKGCONFIG = 1
+        endif
+        ifneq ($(shell which $(NCURSES_PREFIX)5-config 2>/dev/null),)
+          HAVE_NCURSES5CONFIG = 1
+        endif
+      endif
   endif
 
   # Link to ncurses if we're using a non-tiles, Linux build
   ifeq ($(HAVE_PKGCONFIG),1)
-    ifeq ($(LOCALIZE),1)
-      CXXFLAGS += $(shell pkg-config --cflags ncursesw)
-      LDFLAGS += $(shell pkg-config --libs ncursesw)
-    else
-      CXXFLAGS += $(shell pkg-config --cflags ncurses)
-      LDFLAGS += $(shell pkg-config --libs ncurses)
-    endif
+    CXXFLAGS += $(shell pkg-config --cflags $(NCURSES_PREFIX))
+    LDFLAGS += $(shell pkg-config --libs $(NCURSES_PREFIX))
   else
     ifeq ($(HAVE_NCURSES5CONFIG),1)
-      ifeq ($(LOCALIZE),1)
-        CXXFLAGS += $(shell ncursesw5-config --cflags)
-        LDFLAGS += $(shell ncursesw5-config --libs)
-      else
-        CXXFLAGS += $(shell ncurses5-config --cflags)
-        LDFLAGS += $(shell ncurses5-config --libs)
-      endif
+      CXXFLAGS += $(shell $(NCURSES_PREFIX)5-config --cflags)
+      LDFLAGS += $(shell $(NCURSES_PREFIX)5-config --libs)
     else
       ifneq ($(TARGETSYSTEM),WINDOWS)
-        LDFLAGS += -lncurses
+        LDFLAGS += -l$(NCURSES_PREFIX)
       endif
 
       ifdef OSXCROSS
-        LDFLAGS += -L$(LIBSDIR)/ncurses/lib
-        CXXFLAGS += -I$(LIBSDIR)/ncurses/include
+        LDFLAGS += -L$(LIBSDIR)/$(NCURSES_PREFIX)/lib
+        CXXFLAGS += -I$(LIBSDIR)/$(NCURSES_PREFIX)/include
       endif # OSXCROSS
     endif # HAVE_NCURSES5CONFIG
   endif # HAVE_PKGCONFIG