[nem-pl] Re: [svn] nemerle/trunk: lib/tree.n ncc/Makefile ncc/testsuite/Makefile ncc/testsuite/match-literals-string-eq...

[Aleksander M.]

Witam,

Thursday, January 15, 2004, 9:51:59 AM, napisałeś:


MM> Wcięcia! Co kawałek masz inne. Sugeruje dostować się do systemu wcinania
MM> w stdlib, example:

dostosuje sie ;p
 
MM> Spacje dookoła operatorów, szczególnie ':' oraz ','.

patrz wyzej :)

MM> Ok, teraz bardziej konstruktywne uwagi:

MM> Dlaczego ta funkcja nazywa się _get a nie get? Nazwanie jej tak nie
MM> ukrywa żadnych nazw (to przecież funkcja lokalna). Powoduje za to
MM> ignorowanie warninga o nie używanej zmiennej (nie są one generowane dla
MM> zmiennych o nazwie zaczynającej się '_').

Hmm no to takiego nawyku nabylem ;) bo czasami ciezko ze
slowotworstwem  ;p ale dostosuje sie rowniez (a o warningach to nie
pomyslalem)

MM> Zamiast Red()/Black() zrobiłbym flagę is_red (tudzież is_black),
MM> porównywanie będzie szybsze a obiekty mniejsze.

super ;)


MM> Ten komparator na pewno nie może siedzieć w każdym nodzie, jeśli to ma
MM> być pamięciowo efektywne. Tak, wiem, tak to działa w tree.n, ale tak
MM> nie ma działać.

hmm no ja po implementacji tych drzew to chcialem to uruchomic zeby
dzialalo - wiec juz Map itp to jest copypaste z poprawkami.
Ale rowniez inaczej to wymysle :)


>> +(*
>> +BEGIN-INPUT
>> +END-INPUT
>> +*)

ehh to bylo pozno :)




-- 
Pozdrowienia,
 Aleksander M.                            mailto:elfwarr at tenet.pl