[ekg2-devel] display_notify == 2

[Wiesław Ochmiński]

On Wed, Feb 06, 2008 at 10:49:40AM +0100, Michał Górny wrote:
> On Tue, Feb 05, 2008 at 05:09:36PM +0100, =?ISO-8859-2?Q?Wies=B3aw_Ochmi=F1ski_ wrote:
> > Poniższe powoduje, że przy (display_notify==2) wyświetlane są tylko zmiany
> > stanów z dostępnego na niedostępny i na odwrót oraz przejście w stan
> > bliżej nikomu nieznany (EKG_STATUS_UNKNOWN).
> 
> Dzięki za chęci, ale i tak zrobiłem po swojemu ( ;.

Świetnie. Najważniejsze, że zaczęło działać.
 
> A tak na przyszłość:
> 1) pilnuj, żeby kod był zgodny z opisem (gdzie tam na dole masz
> 	cokolwiek o UNK?),

Wzmianka o _UNKNOWN była nawiązaniem do dyskusji na !ekg2 i rzeczywiście
mogła być niejasna. (Początkowo myślałem tylko o usunięciu informacji o
zmianach z _AVAIL na _AWAY i odwrotnie, bo zasugerowałem się tym co do
tej pory było w tym miejscu (i nie działało)).

> 2) upraszczaj wyrażenia logiczne, XOR nie boli,

Warunek w if nie był mojego autorstwa ((C) darkjames). Ja zrobiłem tylko
łatę z tego, ale broniłbym tamtego zapisu, bo uważam, że upraszczanie
wszelkich wyrażeń logicznych w czasach istnienia kompilatorów
optymalizujących jest czynnością na tyleż nudną co zbędną, zwłaszcza
jeśli chodzi o fragment kodu wykonywany raz na (statystycznie) parę
minut. Czytelność kodu w takim przypadku jest dużo istotniejsza,
aczkolwiek jest to uwaga dość luźno związana z czytelnym przecież
warunkiem XOR.

> 3) w core staramy się unikać wskazywania konkretnych statusów, raczej
> 	operujemy na grupach (makro EKG_STATUS_IS*).

Nie wiedziałem, teraz wiem i dziękuję za zwrócenie uwagi.

    Pozdrawiam,
	Wiechu
-- 
 Wiesław Ochmiński, Polska Północna, Malbork.