[PATCH:xmodmap 3/3] Replace chk_malloc + sprintf with asprintf

Alan Coopersmith alan.coopersmith at oracle.com
Sun Nov 13 09:27:03 PST 2011


On 11/12/11 01:14, walter harms wrote:
>     Exit(-1); translates into exit(-1);  (Why ?)
>     this does not work as intended. maybe 1 or EXIT_FAILURE is better here.	

It's what the original code had and I didn't think about changing it.
xmodmap doesn't document exit values, so I think any non-zero value
is about the same here, even if it's a -1 that gets translated to a
unsigned char value.

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list