nx-libs/nx-X11/lib/X11/XDefaultOMIF.c, branch 3.6.x

libNX_X11: Fully drop loadable i18n module support in libNX_X11.

2017-03-03T12:45:25+00:00

- Support for i18n in libNX_X11 is static only, has ever been. - Thus, the files XlcSL.[co], XlcDL.[co], XDefaultIMIF.[co], XDefaultOMIF.[co] and lcDynamic.[co] become obsolete.

XDefaultOMIF: Remove comments referring to ancient Sun bug ids

2016-10-19T19:40:30+00:00

Signed-off-by: Alan Coopersmith Reviewed-by: Adam Jackson Backported-to-NX-by: Ulrich Sibiller

XDefaultOMIF: additional code simplification

2016-10-19T19:40:30+00:00

Don't need to test for a case that we already returned for, don't need to store a count that will only ever be 1 if we didn't return, don't need to increment pointers to allow storing more than one item when we can only ever possibly do one. Signed-off-by: Alan Coopersmith Reviewed-by: Adam Jackson Backported-to-NX-by: Ulrich Sibiller

XDefaultOMIF: replace strlen+Xmalloc+strcpy with strdup

2016-10-19T19:40:30+00:00

Code seems to have been originally written to handle appending multiple strings, but only ever operates on a single string. Signed-off-by: Alan Coopersmith Reviewed-by: Adam Jackson Backported-to-NX-by: Ulrich Sibiller

Delete #if 0 hunks of code

2016-10-19T19:40:29+00:00

Signed-off-by: Alan Coopersmith Backported-to-NX-by: Ulrich Sibiller

libX11: rm redundante NULL checks

2016-10-19T19:40:29+00:00

This patch removes the last remaining NULL checks for Xfree() Signed-off-by: Harms Reviewed-by: Peter Hutterer Signed-off-by: Peter Hutterer Backported-to-NX-by: Ulrich Sibiller

Remove even more casts of return values from Xmalloc/Xrealloc

2016-10-19T19:40:28+00:00

Signed-off-by: Alan Coopersmith Backported-to-NX-by: Ulrich Sibiller

init_om: remove unneeded extra copy of string to local buffer

2016-10-19T19:40:28+00:00

Strings from the supported_charset_list[] were being copied one by one to a stack buffer, and then strdup called on that buffer. Instead, just strdup the original string, without the local copy, and use a more traditional for loop, so it's easier to figure out what the code is doing (cleaning up a gcc const-cast warning in the process). Signed-off-by: Alan Coopersmith Reviewed-by: Matthieu Herrb Backported-to-NX-by: Ulrich Sibiller

Sun's copyrights are now owned by Oracle

2016-10-19T19:40:27+00:00

Signed-off-by: Alan Coopersmith Backported-to-NX-by: Ulrich Sibiller

Convert malloc(strlen()); strcpy() sets to strdup

2016-10-19T19:40:25+00:00

Signed-off-by: Alan Coopersmith Reviewed-by: Jeremy Huddleston Backported-to-NX-by: Ulrich Sibiller