Install flake8-bugbear plugin and fix resulting warnings. NFC (emscri…

…pten-core#17173)

tests: Add `@requires_ninja` decorator. NFC (emscripten-core#17110)

Also rename the existing `require_node` and `require_v8` decorators.

See emscripten-core#17101

Remove `source-map` dependency from terser bundle (emscripten-core#17067

)

When updating terser to 4.1.0 in emscripten-core#17035 I inadvertently introduced a
dependency on the node source-map module.  Rebuilding after patching
our terser fork removed this:

emscripten-core/terser@b3ace3b

This happens to be available during testing because it one of our dev
decpendencies but we not include it in our end-user/production
dependencies.

This kind of error is hard to find since the `node_modules` for our test
setup is the same as the node_modules for emscripten itself.  It migth
be worth attempting to tease them appart if this kind of thing ever
shows up again.

Fix flakiness in `other.test_unistd_close_noderawfs` (emscripten-core…

…#16888)

By looking only for file descriptors pointing to the current test
directory (i.e. exclude the internal file descriptors of Node).

And forward stdin/stderr to `/dev/null`, we are only interested
in `stdout`.

So:
```
find /proc/<PROCESS>/fd -lname "<DIR>*" -printf "%l\n" || true
```

Instead of:
```
find /proc/<PROCESS>/fd -lname "<DIR>*" -printf "%l
" || true
```

Fixes: emscripten-core#16887.

[WasmFS] Fix error code of getcwd() when |size| is too small (emscrip…

…ten-core#16818)

ENAMETOOLONG is not the right return value for getcwd in that
case (but it is for getwd - note no "c" - in another case, according to
the shared man page, which is perhaps what confused us before).
The correct code is ERANGE, which I guess makes sense as the
|size| param is out of range (we are not hitting an absolute "name is
too long for the OS" error).

Update struct info test after emscripten-core#16770. NFC (emscripten-…

…core#16773)

I jumped the gun and landed that PR before all the tests had run.

Format and cleanup site/source/docs/api_reference/advanced-apis.rst. …

…NFC (emscripten-core#16523)

- Use hyperlinks in footnotes to make the source easier to read
- Reformat at 80 columns
- Remove redundant/out-of-date note about "The small number of options
  that developers may have cause to change" (this is covered above
  and also not so true anymore since we have settings_internal)

Fix Closure compile of small builds that use legacy DYNCALLS mode (em…

…scripten-core#13883)

* Fix Closure compile of small builds that use legacy DYNCALLS mode but do not export any wasm function with signature 'v' or 'vi'.

* Address review

* Update tests/test_other.py

* Address review

* Remove duplicate hasExportedFunction

* Use hasExportedFunction()

* Generalize closure optimization to makeDynCall macro

* Clean up Closure warnings

Co-authored-by: Alon Zakai <alonzakai@gmail.com>

Fix default_setting handle of settings with NO_ prefix on the command…

… line (emscripten-core#16365)

Fixes: emscripten-core#16348

Make shared structs for EH/SjLj thread-local (emscripten-core#16292)

These global variables are used as a medium to communicate between the
compiler-generated code and the libraries, so they should be
thread-local to support threads.