On Wed 07 May 2014 at 21:57:42 +0200, Rhialto wrote:
>     /* If old_bw is zero, the application has asked not to have any border,
>        so we indeed do not draw any border.  Maybe it would be better to set
>        frame_bw and frame_bw3D to 0 in this case, but I don't know where
>        this is handled.  --Stef  */

Maybe the frame_bw3D thing you refer to is this, from
add_window.c:AddWindow():

    tmp_win->frame_bw3D = Scr->ThreeDBorderWidth;
    if (LookInList(Scr->NoBorder, tmp_win->full_name, &tmp_win->class)) {
        tmp_win->frame_bw = 0;
        tmp_win->frame_bw3D = 0;
    } else if (tmp_win->frame_bw3D != 0) {
        tmp_win->frame_bw = 0;
        Scr->ClientBorderWidth = FALSE;
    } else if (Scr->ClientBorderWidth) {
        tmp_win->frame_bw = tmp_win->old_bw;
    } else {
        tmp_win->frame_bw = Scr->BorderWidth;
    }
    bw2 = tmp_win->frame_bw * 2;


(where I think the Scr->ClientBorderWidth = FALSE; assignment is out of
place - it should probably be done when reading the config file and 
Scr->ThreeDBorderWidth; is set non-zero)

The test you want to do here can probably be based on
Scr->ClientBorderWidth? Maybe it is enough that if
Scr->ClientBorderWidth is TRUE, the border width in the configure
request is ignored?

-Olaf.
-- 
___ Olaf 'Rhialto' Seibert  -- The Doctor: No, 'eureka' is Greek for
\X/ rhialto/at/xs4all.nl    -- 'this bath is too hot.'

Attachment: pgpvH_N900FJ8.pgp
Description: PGP signature

Reply via email to