Several of these messages appear:

# Geany-INFO: Couldn't convert from ISO-8859-8-I to UTF-8 (Conversion from character set “ISO-8859-8-I” to “UTF-8” is not supported).

What that points out is that Geany has a fixed list of conversions, which are exposed to user selection, but not all systems support all conversions, as you might guess my iconv -l does not contain ISO-8859-8-I. If I try to open an ASCII file (which is valid for all 8859s) with ISO-8859-8-I I get The file xxxx is not valid ISO-8859-8-I which is wrong2, does g_convert() return a better error we could show (like the one from the test)?

g_convert() reports the message you see in the info message inside the parentheses. So yes, we could display it, it's a Mere Matter of Programming™, to report that error all the way up to the caller that will show the error.

As of ISO-8859-8-I it seems understandable not to be supported, as IIUC it's the same thing that should just be rendered differently (or not, which makes matter worse).

Anyhow, yes, this could be improved, and ideally we'd probably not display the encoding in the list if it's not supported by the library we use (iconv), as it makes little sense then. But I'd argue this should probably be a separate improvement :)


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3716/c1848957533@github.com>