[ekg2-devel] Pola zamiast act w window_t

[Jakub Zawadzki]

On Wed, Feb 13, 2008 at 04:42:37PM +0100, Michał Górny wrote:
> On Sun, Feb 10, 2008 at 11:03:37PM +0100, GiM wrote:
> > c) łatwo _bardzo_ (o czym już pisał dj) o błędy,
> Z drugiej strony, trochę żal, że gcc nie ostrzega przed wpisaniem
> w pole bitowe większej wartości.

Awykonywalne. masz foo z polem bitowym bar:1,
robisz foo.bar = baz();

powiedzmy jeszcze ze baz() jest w innym .o

BTW.
window_t->lock ma na pewno byc int, (jak wywolujemy komende to
zwiekszamy w->lock, wiec jak komenda wywoluje komende, to sie popsuje)

> > d) gcc i tak to przerabia jak chce, więc zysk pamięciowy
> >    może mieć przebicie na czas (no to chyba zawsze jest taki
> >    trade pamięć vs. czas),
> 
> IMO akurat w przypadku komunikatora internetowego, który raczej
> programem o wysokim priorytecie nie jest, akurat powinniśmy kierować się
> przede wszystkim ku minimalizacji zużycia pamięci. Nie robimy tyle
> rzeczy, żeby naprawdę nam zależało na oszczędności czasu procesora;
> za to zdarza nam się całymi dniami w wielu kopiach siedzieć w pamięci.

Bo ekg2 tak bardzo zuzywa pamieci...
Zreszta procesora tez.

(darkjam   7100  0.0  3.5 26384 15812 pts/3   S+   Jan01  49:27  |   \_ekg2

 49 minuty procesora, przez 1,5 msc to duzo? [to sa 2 pentiumy II 266 Mhz]
)

Powtorze pytanie: chcesz zamieniac w strukturach z 'int' na 'char',
wszedzie gdzie mozliwe? Potem mozna isc krok dalej i zamieniac zmienne z
int na char.

Jesli nie, to nie jestes konsekwentny.

> > g) ewentualne zwiększanie pola bitowego (e.g. bo chcemy kolejny
> >    ficzer) to psucie ABI
> 
> Jeśli chcemy kolejny ficzer, to raczej nie zwiększamy pola, tylko robimy
> nowe, adekwatnie nazwane. Przecież o to chodzi, nie? Jeśli dodamy je na
> końcu, to bodajże ABI nic się nie stanie. Swoją drogą, o ile się
> nie mylę, to grupa pól bitowych i tak wyrównywana jest do określonego
> pochodnego typu, więc do pewnego stopnia możńa dokładać nowe bity
> w środku.

w ircu/mucu w->act 0x3 bedzie wykorzystane jak ktos napisze
wiadomosc na kanale ale highlitujac nas (i wolalbym zeby to bylo w w->act)

na w->act 0x4 na razie nie mam pomyslu, ale wtedy przekroczymy 2 bity 
(moze jak sie pojawi jakis specyficzny tekst, ale nie do nas?,
everything possible)

Grupa pol bitowych nie jest wyrownywana dopoki nie zrobisz :0, albo
kiedy nie wstawisz jakiegos typu ktory wymusi wyrownania.

Mnie najbardziej zalezy zeby w w->act byla TYLKO informacje nt. czy cos sie 
pojawilo w okienku, i jak wazne; 

Przy okazji fajnie by bylo nie popsuc przy okazji niczego 
(wiec im mniej zmienimy, tym prawd. popsucia czegos maleje), ale jesli
uwazasz ze tak bedzie lepiej (i niczego nie popsujesz), to OK, 
mozesz komitowac.
Mozesz nawet dac: Approved-by-But-with-Comments: Jakub Zawadzki

Wypadaloby tez migrowac pozostale strukturki na pola bitowe, bo w koncu
coding-style powinien byc mniej-wiecej spojny.

Pozdrawiam.