DECRQM returns crap

[pjanx]

Problem description

DECRQM is implemented wrong. It reports "set" when the feature is unknown, and reports "unknown" when a feature is known.

                                int internalBit = mapDecSetBitToInternalBit(mode);
                                if (internalBit == -1) {
                                    value = isDecsetInternalBitSet(internalBit) ? 1 : 2; // 1=set, 2=reset.

The fix appears to be trivial: change == to !=.

Additional information

• Termux application version: master
• Android OS version: —
• Device model: —

[Grimler91]

Thanks for noticing, this bug has apparently been there since the first commit:

termux-app/app/src/main/java/com/termux/terminal/TerminalEmulator.java

Line 729 in a18ee58

if (internalBit == -1) {

[ghost]

Part of v0.100 release.