Dnia 2011-06-05, nie o godzinie 12:44 +0200, Tomasz Wasilczyk pisze: > Plik dcc.c:421 - zmienna uint16_t port jest przyrównywana do -1 (sypie > warningiem w libpurple). Można by chociaż rzutować tą -1 na uint16_t.
Ten kod i tak nie jest już pewnie przez nikogo używany do niczego sensownego, więc czego byśmy nie zrobili, nie będzie źle. Ale skoro zachęcaliśmy do podawania wartości -1, która zostanie automagicznie rzutowana na uint16_t, to rzutowanie przy porównaniu ma chyba najwięcej sensu. Dorzuciłem też komentarz o wynikającym z tego problemie z portem 65535, zgodnie z tym co napisał Jaku. > Zamiast typu unsigned int można by było używać typów socklen_t i > size_t (szczegóły w załączniku). To dotyczy prawdopodobnie większej > ilości zmiennych. Jasne. Skoro i tak już używamy socklen_t w dcc7.c, to czemu nie. Commitnięte. > Kompilacja pod Windows - libpurple wycina ifdefami część include-ów, > aby całość ładnie się kompilowała pod tym systemem. Wiem, że libgadu > nie ma oficjalnego wsparcia pod okienkami, ale parę ifdefów dużo nie > popsuje ;]. Szczegóły w linku do dużego diffa z różnicami. Jeśli już, to wolałbym wpakować wszystkie sieciowe #include'y do nowego pliku network.h i tam zrobić #ifdefy tylko jeden raz. Plus zawartość compat.h, bo idealnie do sieciowych zależności by pasowało. Co Ty na to? > Kompilacja pod SunOS i HPUX - jeden ifdef - szczegóły w załączniku. Nie lepiej naprawić to jakoś w configure? Wyjątku dla cygwina też bym się chętnie pozbył, ale nawet nie pamiętam dlaczego się tam pojawił. Pozdr, Wojtek _______________________________________________ libgadu-devel mailing list libgadu-devel@lists.ziew.org http://lists.ziew.org/mailman/listinfo/libgadu-devel
