sys.readFile for Node.js: the encoding parameter is ignored, so charset compiler option is ignored on Node.js

[denvned]

Current implementation of sys.readFile for Node.js ignore the encoding parameter. This effectively makes the charset compiler option ignored on Node.js. If it is "by desing", then it probably should be documented.

[mhegazy]

it is by design. the encoding is an option for engines that support it. node's read file does not support the whole encoding ranges. What sort of documentation you want? just a comment in the code, or an error message that this compiler option is not supported on this engine?

[denvned]

At least here I think. An error message and a description in the help output would be nice also. But this all is not so critical for me. Probably I will send a PR later.

[mhegazy]

A PR would definitely be appreciated!

[jakebailey]

charset was removed in #57527.