[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