Re: [xwax-devel] [PATCH 2/2] Test for font rendering

[Olivier G. <ol...@os...>]

On 06/03/13 05:04 PM, Mark Hills wrote:
> On Wed, 6 Mar 2013, Olivier Gauthier wrote:
> 
>> On 21/02/13 04:22 PM, Mark Hills wrote:
>>> On Thu, 21 Feb 2013, Robert-André Vettel wrote:
>>>
>>>> Sounds cool, now for the noobs: how do I apply it :)
>>>> Just insert both patch-parts in interface.c ?
>>>
>>> Save each patch email to a text file, then do
>>>
>>>   $ cd xwax
>>>   $ patch < /path/to/file.patch
>>>
>>> Maybe I'll do some betas/test releases, so that non Git/patch aware users 
>>> can try out some of these tests.
>>>
>>>> And where do I get the fixed libsdl ? I just found the official page via 
>>>> google but there is neither a ppa nor some .debs and I'm not in my 
>>>> compiling-mood right now.
>>>
>>> You might just have to be patient :) It's accepted but that doesn't mean a 
>>> new version has been released. That's one reason why I'm in no rush to 
>>> commit the code for v1.4.
>>
>> I tried libsdl-ttf2.0-dev (2.0.11-2) from debian testing using
>> apt-pinning on my debian stable. It enabled me to compile your
>> interface.c patch, it was giving me errors about TTF_SetFontHinting and
>> TTF_HINTING_NONE been undeclared.
> 
> Strange that nobody has the TTF_SetFontHinting function available. I will 
> need to check what version is needed for it to be there.
> 
> It's fairly crucial as I really want the interface to have good looking 
> fonts that are smoothly scalable on-the-fly (without restarting.)

I tried the other patch "Initial zoom test". It's basically what I do on
every time I compile my xwax, but I changed the numbers manually without
a zoom function.

I changed the 1.5 value to a variable called zoom_scale, and I enabled
to modify the value using CTRL-PLUS and CTRL-MINUS. I had to remove zoom
call from values that touchs the spinner_angle look-up table as GCC was
talking about variably modifing something.

>From there I can resize on the fly the interface using the keyboard,
but, even if the keyboard loop does return "true" to tell the interface
to redraw, some parts are getting mangled. Here's a screen shot.

> 
>> I also needed to modify the includes in your test-ttf so it find SDL.h.
>> Probably a weirdness in the makefile.
>> #include <SDL.h> -> #include <SDL/SDL.h>
>> #include <SDL_ttf.h> -> #include <SDL/SDL_ttf.h>
> 
> It would be interesting to see the output of
> 
>   which sdl-config
>   sdl-config --cflags
> 
> which may explain why you have to do this.
$ which sdl-config
./usr/bin/sdl-config
$ sdl-config --cflags
-I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT

I don't know much about the include process of libsdl in general. I
simply know that installing the dev packages is enough for me to compile
xwax. Now why your test program needed help here is mystery.

>  
>> Here's the output screenshot of the test-ttf on my machine.
> 
> It shows that the clipping bug is there in the version you're using.
> 
> You could try and build from scratch, from: http://hg.libsdl.org/SDL_ttf/
> 

I just tried to build it, but sadly it depends on SDL2.0 which is not
yet there in debian. I built SDL2.0 and then I stopped my endeavors when
I tried to remove libsdl1.2debian from my install, it wanted to tear
apart a bunch of useful packages. Do you know if it's safe the install
libsdl2.0 aside of 1.2 ?

I kind of feel that this version will take quite a long time to get into
distributions.


thanks!

Olivier