Am 04.06.2014 12:23, schrieb Mark Kettenis: >> Date: Wed, 04 Jun 2014 12:07:25 +0200 >> From: walter harms <[email protected]> >> >> Do not use variables before checked for NULL. >> >> >> Signed-off-by: wharms <[email protected]> >> >> --- >> modules/lc/def/lcDefConv.c | 35 +++++++++++++++++++++++++---------- >> 1 file changed, 25 insertions(+), 10 deletions(-) >> >> diff --git a/modules/lc/def/lcDefConv.c b/modules/lc/def/lcDefConv.c >> index 3cd5c22..2cae504 100644 >> --- a/modules/lc/def/lcDefConv.c >> +++ b/modules/lc/def/lcDefConv.c >> @@ -181,14 +182,16 @@ def_wcstombs( >> XPointer *args, >> int num_args) >> { >> - const wchar_t *src = (const wchar_t *) * from; >> + const wchar_t *src; >> char *dst = (char *) *to; >> State state = (State) conv->state; >> char ch[MB_LEN_MAX]; >> int unconv = 0; >> >> if (from == NULL || *from == NULL) >> - return 0; >> + return 0; >> + >> + src = (const wchar_t *) * from; > > While there, you should probably remove the space after the '*'. > There are a couple of more occurances further down.
you mean: src = (const wchar_t *) *from; ? re, wh _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
