[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: IceWM tray bug (Was: Re: icewm + kaffeine)



#include <hallo.h>
* Stanislav Maslovski [Tue, Oct 13 2009, 12:50:51PM]:
> On Mon, Oct 12, 2009 at 10:55:47PM +0000, Eduard Bloch wrote:
> > On Sat, Oct 10, 2009 at 11:35:14PM +0700, Vladislav Naumov wrote:
> > > Самый уловистый оказался psi:
> > > хоть неубиваемого окна он не даёт, зато _стабильно_ валит трей на
> > > третьем запуске. То есть раз запустили-вышли, два запустили-вышли, три
> > 
> > Ну и вот, я постояно пользуюсь пси и таких еффектор с анстеибл-версией только
> > один раз заметил. В чем фишка? Переставлю сейчас локаль на русский, может тогда
> > проявится скоро.
> 
> Локаль тут наврядли играет роль. Попробуй лучше (с исходно пустым
> треем!) многократно позапускать/поприбивать psi или gajim, как
> Владислав и я делали.

Ага, попробовал. Баг в самом деле обнавружил, только вероятность не
такая большая (5..10%) но если ето race то разница допустима. Просмотрел
исходники по быстрому, побочных еффектов от вашего пача не ожидаю
(конечно кроме того что пытались первоначально с этим измененем
исправить). Версию 1.2.37+1.3.4pre2-5 с ним только что закинул в
incomming, там еще пару интерессных фиксов ждали.

> > Если будет очевидно, что этот коммит в самом деле был безсмусленый,
> > тогда может и выкину. А сейчас еще не ясно, и руки не добрались до
> > проверки (простите, просто нету времени!).
> 
> Он не совсем бессмысленный. Марко пытался исправить баг, который
> проявляется при рестарте icewm. Суть бага в том, что после рестарта
> (у меня это происходит только с пустым треем) на месте трея
> появляется черный квадрат (он может быть и не черным. Технически,
> это пустое окно, которое не перерисовывается по Expose events).
> 
> Исправление, которое он придумал, с флагом fManaged и дополнительным
> вызовом requestDock() из trayChanged(), на мой взгляд никуда не
> годится, так как явно приводит к race и к гораздо более неприятному
> багу.
> 
> Я почитал исходники вчера еще немного, и мне кажется, что я близок к
> пониманию, как оно всё в целом работает, так что я думаю, что смогу
> предложить другое исправление для бага при рестарте.

Хорошо, благодарю заранее. Если нужно будеть осторожно протестировать,
есть возможность вставить экспериментальные добавки в пакет
icewm-experimental.

MfG,
Eduard.

-- 
<Salz> jjFux: Ted hieß ja früher auch Walther
<Salz> winkiller: hm... es sind 8... die 7 kandidaten und NOTA
<Madkiss> Ist der jetzt eigentlich eine gespaltene Persönlichkeit, bei der aber
  beide Teile bekloppt sind?


Reply to: