Introduce leak sanitizer, fix several string leaks (#59)

* more doc polish

* Free C strings returned from QuickJS

* Add address sanitizer

* Fix some leaks!

* Fix some leaks in test

* unfix an over-fixed test

* doc

* Fix sanitize dingus

* Santizer self test

* Split sanitizer out to own define

* enable -ftlo link time optimization for release build

* Remove extra string copy for QTS_GetString/QTS_Dump

* oops: leak build

* finally get C++ include settings working for emscripten.h

Author: justjake

.github/workflows/main.yml

@@ -42,27 +42,24 @@ jobs:
         uses: actions/cache@v2
         with:
           path: "**/.yarn/cache"
-          key: ${{ hashFiles('**/yarn.lock') }}-${{ hashFiles('**/package.json') }}
+          key: ${{ hashFiles('**/yarn.lock') }}
 
       - name: Install dependencies
         run: yarn install --immutable
         env:
           YARN_ENABLE_SCRIPTS: 0
 
-      - name: Test debug
-        run: make clean && yarn test
+      - name: Build
+        run: yarn build
 
-      - name: Prettier
-        run: yarn run prettier-check
-
-      - name: Build release
-        run: yarn make-release && yarn build && yarn tarball
-
-      - name: Test release
-        run: yarn run mocha
+      - name: Test
+        run: yarn test-all
 
       - name: Test release with NodeJS/Typescript example
         run: yarn smoketest-node
 
       - name: Test release with create-react-app/Typescript example
         run: yarn smoketest-cra
+
+      - name: Prettier
+        run: yarn run prettier-check

.gitignore

@@ -2,9 +2,8 @@ node_modules
 yarn-error.log
 npm-error.log
 
-*.wasm
-*.wasm.map
 *.tsbuildinfo
+.vscode/*.log
 
 /build
 /dist

.prettierignore

@@ -1,11 +1,9 @@
-ts/ffi.ts
-ts/ffi-asyncify.ts
-*.emscripten-module.js
+ts/generated
 quickjs
 .yarn
 doc
 test/json-generator-dot-com-1024-rows.json
-dist/*
+dist
 examples/imports
 examples/typescript-smoketest/*.js
 build

.vscode/c_cpp_properties.json

@@ -2,14 +2,42 @@
   "configurations": [
     {
       "name": "Mac",
-      "includePath": ["${workspaceFolder}/**"],
-      "defines": ["QTS_ASYNCIFY=1", "QTS_DEBUG=1"],
+      "includePath": [
+        "${workspaceFolder}/**",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/system/include/**",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/cache/sysroot/include/**",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/llvm/lib/clang/15.0.0/include/**",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/cache/sysroot/include/SDL",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/cache/sysroot/include/compat",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/llvm/lib/clang/15.0.0/include",
+        "/opt/homebrew/Cellar/emscripten/3.1.7/libexec/cache/sysroot/include"
+      ],
+      "defines": ["QTS_ASYNCIFY=1", "QTS_DEBUG_MODE=1", "__EMSCRIPTEN__=1"],
       "macFrameworkPath": [
         "/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks"
       ],
       "compilerPath": "/usr/bin/clang",
       "cStandard": "c17",
       "cppStandard": "c++98",
+      "intelliSenseMode": "macos-clang-arm64",
+      "compilerArgs": [],
+      "mergeConfigurations": false,
+      "browse": {
+        "path": ["${workspaceFolder}/**"],
+        "limitSymbolsToIncludedHeaders": true
+      },
+      "configurationProvider": "ms-vscode.makefile-tools"
+    },
+    {
+      "name": "Emscripten",
+      "includePath": ["${workspaceFolder}/**"],
+      "defines": [],
+      "macFrameworkPath": [
+        "/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks"
+      ],
+      "compilerPath": "/opt/homebrew/bin/emcc",
+      "cStandard": "c17",
+      "cppStandard": "c++98",
       "intelliSenseMode": "macos-clang-arm64"
     }
   ],

.vscode/settings.json

@@ -1,9 +1,13 @@
 {
-  "cSpell.words": ["Asyncified", "ASYNCIFY", "chonkers", "JSWASM", "runtimes"],
+  "cSpell.words": ["Asyncified", "ASYNCIFY", "chonkers", "JSFFI", "JSWASM", "runtimes"],
   "files.associations": {
     "cutils.h": "c",
     "quickjs.h": "c",
     "quickjs-libc.h": "c",
-    "stdio.h": "c"
+    "stdio.h": "c",
+    "lsan_interface.h": "c",
+    "math.h": "c",
+    "stdbool.h": "c",
+    "emscripten.h": "c"
   }
 }