vscode 1.52.1

With javascript.preferences.importModuleSpecifierEnding set to "js":

Given a file ./foo.js that exports foo, typing

import foo from "./

brings up a suggestion for ./foo, without the extension.

In contrast, simply typing foo and letting auto import do the work correctly adds

import foo from "./foo.js";

at the top of the file, re

Hello.

Since the switch from requests to native-request UrlFileManager does not handle redericts anymore. This leads to an error here (and possibly in other similar code) if response body is empty (like often happens with 301 responses). The error is

Warning: Empty body (HTTP 301) returned by

### Example
let unreachableProxy = "http://127.0.0.1:8888"
try:
  let client = newHttpClient(proxy=newProxy(unreachableProxy), timeout=1)
  var res = client.getContent("https://github.com")
  echo res
except:
  echo "Timed out"

Current Output

timeout variable is ignored, program stuck until default socket timeout will happen

Expected Output

Timed out a

If A is a constant that divides 2**256 (so basically of the form 2**N), then the two expressions should be the same. mod(mul(X, Y), A) costs 10 and mulmod(X, Y, A) costs 8 (excluding the shorter deploy code.)

This is a follow-up to #4097 where we used an AtomicInteger as a poor-mans replacement for a Semaphore with tryAcquire (or maybe even drainPermits).

Should suppress --strict warnings.