sync with mlabbe:devel

.clang-format

@@ -0,0 +1,57 @@
+# main
+IndentWidth: 4
+BreakBeforeBraces: Linux
+AlwaysBreakAfterDefinitionReturnType: true
+UseTab: Never
+AllowShortIfStatementsOnASingleLine: false
+ColumnLimit: 80
+IndentCaseLabels: false
+
+# for macros
+AlignEscapedNewlinesLeft: false
+IndentPPDirectives: AfterHash
+
+# i love alignment
+BinPackArguments: false
+AllowAllParametersOfDeclarationOnNextLine: true
+BinPackParameters: false
+AlignTrailingComments: true
+AlignConsecutiveDeclarations: true
+AlignAfterOpenBracket: Align
+
+# most flow control must be indented for easy eye scanning
+AllowShortBlocksOnASingleLine: false
+AllowShortCaseLabelsOnASingleLine: false
+AllowShortLoopsOnASingleLine: true
+
+# braces
+BraceWrapping:
+  BeforeElse: false
+  AfterControlStatement: false
+  AfterFunction: false
+  AfterStruct: false
+  AfterUnion: false
+  IndentBraces: false
+
+# other
+AlignOperands: true
+MaxEmptyLinesToKeep: 3
+KeepEmptyLinesAtTheStartOfBlocks: false
+PointerAlignment: Left
+DerivePointerAlignment: false
+SpaceAfterCStyleCast: false
+SpaceBeforeAssignmentOperators: true
+SpaceBeforeParens: ControlStatements
+SpaceInEmptyParentheses: false
+SpacesBeforeTrailingComments: 2
+SpacesInCStyleCastParentheses: false
+SpacesInParentheses: false
+SpacesInSquareBrackets: false
+IncludeBlocks: Preserve
+SortIncludes: false
+BreakBeforeBinaryOperators: None
+ReflowComments: true
+
+
+# penalties
+PenaltyExcessCharacter: 2

.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,32 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Compilation Environment**
+
+ - OS (eg: OSX 10.14, Ubuntu 18.04):
+ - Compiler (eg: GCC, Clang):
+ - Compiler Version (eg: MSVC 2017):
+ - Build directory used (eg: `build/gmake_linux`:
+ - Have I attempted to reproduce the problem on the `devel` branch?
+
+**Describe the bug**
+
+_A clear and concise description of what the bug is._
+
+**Compilation output**
+
+_If compiling with a makefile, paste the output of `make verbose=1` which includes Makefile steps._
+
+**Additional context**
+
+_Add any other context about the problem here._
+
+**User Description**
+
+_Are you using NFD as an individual, a small company (less than ten employees) or a larger organization?

README.md

@@ -10,7 +10,7 @@ Features:
  - Simple universal file filter syntax.
  - Paid support available.
  - Multiple file selection support.
- - 64-bit and 32-bit friendly.
+ - x64, x86 and arm64 (Linux) support
  - GCC, Clang, Xcode, Mingw and Visual Studio supported.
  - No third party dependencies for building or linking.
  - Support for Vista's modern `IFileDialog` on Windows.
@@ -34,7 +34,7 @@ int main( void )
     if ( result == NFD_OKAY ) {
         puts("Success!");
         puts(outPath);
-        free(outPath);
+        NFD_Free(outPath);
     }
     else if ( result == NFD_CANCEL ) {
         puts("User pressed cancel.");
@@ -77,6 +77,38 @@ release | what's new                  | date
 <i></i> | remove deprecated scons     | aug 2019
 <i></i> | fix mingw compilation       | aug 2019
 <i></i> | -Wextra warning cleanup     | aug 2019
+1.2.0   | defaultPath can specify files now         | jan 2021
+<i></i> | extension automatically added when saving | jan 2021
+<i></i> | GTK dialog focus bugfix     | jan 2021
+<i></i> | Macos dialog focus bugfix   | jan 2021
+<i></i> | Added NFD_Free()            | jan 2021
+<i></i> | Fix vs2019 debug assert     | jan 2021
+<i></i> | Fix zenity debug assert     | jan 2021
+<i></i> | add clang-format            | jan 2021
+<i></i> | Linux arm64 support         | jan 2021
+<i></i> | Apple Silicon support       | jan 2021
+<i></i> | Macos fat binary builds     | jan 2021
+
+## Usage Survey ##
+
+Do you use Native File Dialog?  It helps me to understand how.
+
+There is a totally optional usage survey.  Share how you use Native File Dialog.  Big and small projects, public and private may share.
+
+[Click here to fill out the usage survey.](https://forms.gle/ApWCFsXeCVxpg4XLA "Usage survey").
+
+
+### Breaking and Notable Changes ###
+
+There are no ABI breaking changes in NFD's history.
+
+#### 1.2.0 ####
+
+ - If argument `filterList` is specified, a default extension is appended to the filename if the user does not take an action to specify one.  Previously no extension was added on the GTK3 and Win32 implementations, but was added on MacOS.
+
+ - Argument `defaultPath` sometimes failed to display the specified directory if a file was included in the `defaultPath` but the file did not exist.  In 1.2.0, polyfill was added to display the directory even if the file doesn't exist. 
+
+ - `NFD_Free()` was added, and can be used in place of `free()`.  This was done to facilitate usage of NFD in a DLL on Windows.  Otherwise, there is no difference.
 
 ## Building ##
 
@@ -90,12 +122,14 @@ Previously, NFD used SCons to build.  As of 1.1.6, SCons support has been remove
 
 ### Makefiles ###
 
-The makefile offers up to four options, with `release_x64` as the default.
+The makefile offers up to six options, with `release_x64` as the default.
 
     make config=release_x86
     make config=release_x64
+    make config=release_arm64
     make config=debug_x86
     make config=debug_x64
+    make config=debug_arm64
 
 ### Compiling Your Programs ###
 
@@ -111,7 +145,7 @@ On Linux, you have the option of compiling and linking against GTK.  If you use
 
 #### Linux Zenity ####
 
-Alternatively, you can use the Zenity backend by running the Makefile in `build/gmake_linux_zenity`.  Zenity runs the dialog in its own address space, but requires the user to have Zenity correctly installed and configured on their system.
+Alternatively, you can use the Zenity backend by running the Makefile in `build/gmake_linux_zenity`.  Zenity runs the dialog in its own address space, but requires the user to have Zenity correctly installed and configured on their system.  For a full list of Zenity limitations, see "Known Limitations" below.
 
 #### MacOS ####
 
@@ -156,12 +190,15 @@ I accept quality code patches, or will resolve these and other matters through s
 
  - No support for Windows XP's legacy dialogs such as `GetOpenFileName`.
  - No support for file filter names -- ex: "Image Files" (*.png, *.jpg).  Nameless filters are supported, however.
- - GTK Zenity implementation's process exec error handling does not gracefully handle numerous error cases, choosing to abort rather than cleanup and return.
- - GTK 3 spams one warning per dialog created.
+ - GTK 3 sometimes spams one warning per dialog created, depending on how GTK3 was built.
+ - The GTK Zenity backend (non-default) is a lightweight alternative intended only for small programs:
+   - It errors out if Zenity is not installed on the user's system
+   - This backend's process exec error handling does not gracefully handle numerous error cases, choosing to abort rather than cleanup and return.
+   - Unlike the other backends, the Zenity backend does not return implied extensions from filterlists. [#95](https://github.com/mlabbe/nativefiledialog/issues/95 "Issue 95")
 
 # Copyright and Credit #
 
-Copyright &copy; 2014-2019 [Frogtoss Games](http://www.frogtoss.com), Inc.
+Copyright &copy; 2014-2021 [Frogtoss Games](http://www.frogtoss.com), Inc.
 File [LICENSE](LICENSE) covers all files in this repo.
 
 Native File Dialog by Michael Labbe
@@ -173,6 +210,8 @@ Tomasz Konojacki for [microutf8](http://puszcza.gnu.org.ua/software/microutf8/)
 
 [Tom Mason](https://github.com/wheybags) for Zenity support.
 
+Various pull requests and bugfixes -- thanks to the original authors.
+
 ## Support ##
 
 Directed support for this work is available from the original author under a paid agreement.

build/gmake_linux/Makefile

@@ -22,6 +22,13 @@ ifeq ($(config),release_x86)
   test_opendialogmultiple_config = release_x86
   test_savedialog_config = release_x86
 endif
+ifeq ($(config),release_arm64)
+  nfd_config = release_arm64
+  test_pickfolder_config = release_arm64
+  test_opendialog_config = release_arm64
+  test_opendialogmultiple_config = release_arm64
+  test_savedialog_config = release_arm64
+endif
 ifeq ($(config),debug_x64)
   nfd_config = debug_x64
   test_pickfolder_config = debug_x64
@@ -36,6 +43,13 @@ ifeq ($(config),debug_x86)
   test_opendialogmultiple_config = debug_x86
   test_savedialog_config = debug_x86
 endif
+ifeq ($(config),debug_arm64)
+  nfd_config = debug_arm64
+  test_pickfolder_config = debug_arm64
+  test_opendialog_config = debug_arm64
+  test_opendialogmultiple_config = debug_arm64
+  test_savedialog_config = debug_arm64
+endif
 
 PROJECTS := nfd test_pickfolder test_opendialog test_opendialogmultiple test_savedialog
 
@@ -86,8 +100,10 @@ help:
 	@echo "CONFIGURATIONS:"
 	@echo "  release_x64"
 	@echo "  release_x86"
+	@echo "  release_arm64"
 	@echo "  debug_x64"
 	@echo "  debug_x86"
+	@echo "  debug_arm64"
 	@echo ""
 	@echo "TARGETS:"
 	@echo "   all (default)"

build/gmake_linux/nfd.make

@@ -31,6 +31,8 @@ ifeq ($(config),release_x64)
   define PRELINKCMDS
   endef
   define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
   endef
 all: prebuild prelink $(TARGET)
 	@:
@@ -58,6 +60,37 @@ ifeq ($(config),release_x86)
   define PRELINKCMDS
   endef
   define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
+  endef
+all: prebuild prelink $(TARGET)
+	@:
+
+endif
+
+ifeq ($(config),release_arm64)
+  RESCOMP = windres
+  TARGETDIR = ../lib/Release/arm64
+  TARGET = $(TARGETDIR)/libnfd.a
+  OBJDIR = ../obj/arm64/Release/nfd
+  DEFINES += -DNDEBUG
+  INCLUDES += -I../../src/include
+  FORCE_INCLUDE +=
+  ALL_CPPFLAGS += $(CPPFLAGS) -MMD -MP $(DEFINES) $(INCLUDES)
+  ALL_CFLAGS += $(CFLAGS) $(ALL_CPPFLAGS) -O2 -Wall -Wextra -fno-exceptions `pkg-config --cflags gtk+-3.0`
+  ALL_CXXFLAGS += $(CXXFLAGS) $(ALL_CPPFLAGS) -O2 -Wall -Wextra -fno-exceptions `pkg-config --cflags gtk+-3.0`
+  ALL_RESFLAGS += $(RESFLAGS) $(DEFINES) $(INCLUDES)
+  LIBS +=
+  LDDEPS +=
+  ALL_LDFLAGS += $(LDFLAGS) -s
+  LINKCMD = $(AR) -rcs "$@" $(OBJECTS)
+  define PREBUILDCMDS
+  endef
+  define PRELINKCMDS
+  endef
+  define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
   endef
 all: prebuild prelink $(TARGET)
 	@:
@@ -85,6 +118,8 @@ ifeq ($(config),debug_x64)
   define PRELINKCMDS
   endef
   define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
   endef
 all: prebuild prelink $(TARGET)
 	@:
@@ -112,6 +147,37 @@ ifeq ($(config),debug_x86)
   define PRELINKCMDS
   endef
   define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
+  endef
+all: prebuild prelink $(TARGET)
+	@:
+
+endif
+
+ifeq ($(config),debug_arm64)
+  RESCOMP = windres
+  TARGETDIR = ../lib/Debug/arm64
+  TARGET = $(TARGETDIR)/libnfd_d.a
+  OBJDIR = ../obj/arm64/Debug/nfd
+  DEFINES += -DDEBUG
+  INCLUDES += -I../../src/include
+  FORCE_INCLUDE +=
+  ALL_CPPFLAGS += $(CPPFLAGS) -MMD -MP $(DEFINES) $(INCLUDES)
+  ALL_CFLAGS += $(CFLAGS) $(ALL_CPPFLAGS) -g -Wall -Wextra -fno-exceptions `pkg-config --cflags gtk+-3.0`
+  ALL_CXXFLAGS += $(CXXFLAGS) $(ALL_CPPFLAGS) -g -Wall -Wextra -fno-exceptions `pkg-config --cflags gtk+-3.0`
+  ALL_RESFLAGS += $(RESFLAGS) $(DEFINES) $(INCLUDES)
+  LIBS +=
+  LDDEPS +=
+  ALL_LDFLAGS += $(LDFLAGS)
+  LINKCMD = $(AR) -rcs "$@" $(OBJECTS)
+  define PREBUILDCMDS
+  endef
+  define PRELINKCMDS
+  endef
+  define POSTBUILDCMDS
+	@echo Running postbuild commands
+	@echo "\n *** Do you use Native File Dialog?  Please take the user survey to help development: https://forms.gle/ApWCFsXeCVxpg4XLA" ***
   endef
 all: prebuild prelink $(TARGET)
 	@: