gh-91810: ElementTree: Change default encoding in XML declaration to UTF-8

[methane]

Fix #91810

[serhiy-storchaka]

Look at function _get_writer() when encoding is "unicode". If file_or_filename is not a file, it opens a file with the default encoding (which was usually the locale encoding). If file_or_filename is a file, it was a good chance that it is sys.stdout or a file opened with the default encoding. It made sense to use the locale encoding as the declared encoding. You could be right in more cases than wrong.

Now, it would be more correctly to derive the declared encoding from the file encoding. Make _get_writer() returning a pair: the write function and the declared encoding (which should be getattr(file, "encoding", None) or "utf-8" by default).

[methane]

You are right. There are a few cases that locale encoding is right choice.
But I still think UTF-8 should be the default encoding for XML.

Now, it would be more correctly to derive the declared encoding from the file encoding. Make _get_writer() returning a pair: the write function and the declared encoding (which should be getattr(file, "encoding", None) or "utf-8" by default).

I don't like this type of idea that works correctly just on some cases.
There are many cases that output don't have encoding. UTF-8 by default is the my favorite behavior.

Anyway, I don't want to discuss more on this pull request. Please continue on the issue instead.