Hi,

I need for someone to explain to me what's meant with the following
code, taken from vscreen.c, the ? function:

  for (nptr = Scr->VirtualScreens; nptr != NULL; nptr = nptr->next) {
    virtualScreen *vs;
    char *geometry = (char*) nptr->name;
    int x, y, w, h;

    if ((x < 0) || (y < 0) || (w > scr->rootw) || (h > scr->rooth)) {
      twmrc_error_prefix ();
      fprintf (stderr, "InitVirtualScreens : invalid geometry : %s\n", 
geometry);
      continue;
    }
    vs = (virtualScreen*) malloc (sizeof (virtualScreen));
    vs->x = x;
    vs->y = y;
    vs->w = w;
    vs->h = h;


As you can see, the body of the for loop immediately uses x, y, w and
h.  However, those are not initialized, so they might contain any kind
of value.  Bottom line, there's something missing, and I honestly have
no clue what.

Claude, since Since this is tied in to your experiments with the
Xinerama extension, I'd like to nudge you into action a little bit.
What's missing?

My options are to either just let this be, and see if anyone turns in
a good enough bug report (patches welcome!), or to turn of virtual
screen support entirely.

Cheers,
Richard

-----
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.

-- 
Richard Levitte                         [EMAIL PROTECTED]
                                        http://richard.levitte.org/

"When I became a man I put away childish things, including
 the fear of childishness and the desire to be very grown up."
                                                -- C.S. Lewis

Reply via email to