Bug #72715 character set code endianness dependent on CPU type rather than endianness of CP

Description:
The PowerLinux Migration Toolkit helped find this gem in include/m_ctype.h

/*
  On i386 we store Unicode->CS conversion tables for
  some character sets using Big-endian order,
  to copy two bytes at onces.
  This gives some performance improvement.
*/
#ifdef __i386__
#define MB2(x)                (((x) >> 8) + (((x) & 0xFF) << 8))
#define MY_PUT_MB2(s, code)   { *((uint16*)(s))= (code); }
#else
#define MB2(x)                (x)
#define MY_PUT_MB2(s, code)   { (s)[0]= code >> 8; (s)[1]= code & 0xFF; }
#endif

Which needs some analysis as to the validity on both non-x86 and especially on POWER BE and LE (maybe also MIPS/MIPSel).

How to repeat:
Code analysis.

Suggested fix:
think carefully.. hopefully I'll investigate further and provide further insight/a patch... but if somebody else gets to it first, yay!

I'm mostly filing the bug to keep track of this issue.

Hello Stewart,

Thank you for the bug report.

Thanks,
Umesh