Re: [ANNOUNCE] Sawfish 1.6.1 "Astral". : sawfish restart bugs, short report.

[Teika Kazura <teika lavabit com>]

On Wed, 20 Jan 2010 20:11:11 +0100, Janek Kozicki wrote:
> Oh, but also I noticed that some windows have problems when moving
> them around. They are displayed in their proper coordinates, but
> when I click on them to move them, then suddenly they land on far
> off-screen coordinates. Fortunately with grow-pack I can quickly
> summon them back.

On Wed, 20 Jan 2010 21:41:27 +0100, Janek Kozicki wrote:
>> > (1600*2+1920)*7 = 35840
> 
> I verified it even more precisely. This breaks exactly after 32768th
> pixel is exceeded. Therefore: after 'sawfish restart' all windows
> (except the maximized ones) are in their correct position. But if I
> try to move windows that are far-right (after 2^15 pixels) they behave
> wrongly. Which can be quickly fixed with grow-pack.

That's nasty. Why? You know, in Xinerama.h, there's in fact short:

typedef struct {
   int   screen_number;
   short x_org;
   short y_org;
   short width;
   short height;
} XineramaScreenInfo;

I don't know what exactly happens in C integer comparison, but as far
as I tested the other day, it's harmless. No? (These short values
are accessed in "find-head" in src/functions.c.)

Teika (Teika kazura)