[PATCH:libX11 7/8] Add missing stdio header for printf(3) in DEBUG build.

Alan Coopersmith alan.coopersmith at oracle.com
Sun Jun 2 16:00:16 PDT 2013


On 06/ 2/13 01:04 PM, Thomas Klausner wrote:
> On Sun, Jun 02, 2013 at 10:02:39PM +0200, walter harms wrote:
>>
>>
>> Am 02.06.2013 20:49, schrieb Thomas Klausner:
>>> ---
>>>   src/xcms/cmsMath.c | 4 ++++
>>>   1 file changed, 4 insertions(+)
>>>
>>> diff --git a/src/xcms/cmsMath.c b/src/xcms/cmsMath.c
>>> index 70b0675..487eb3f 100644
>>> --- a/src/xcms/cmsMath.c
>>> +++ b/src/xcms/cmsMath.c
>>> @@ -35,6 +35,10 @@ in this Software without prior written authorization from The Open Group.
>>>   #include "Xlibint.h"
>>>   #include "Xcmsint.h"
>>>
>>> +#ifdef DEBUG
>>> +#include <stdio.h>
>>> +#endif
>>> +
>>
>> I guess libX11 can take the overhead even when DEBUG is nodef
>> just include.
>>
>> #include <stdio.h>  /* debug code needs that */
>
> Fine with me. Should I send an updated patch for that?

No need - the original patch seems fine to me.

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list