Dnia 2011-04-23, sob o godzinie 01:16 +0200, Bartosz Brachaczek pisze:
> Przeglądając kod libgadu, natrafiłem na niezgodny z faktycznym
> działaniem opis funkcji gg_read_line(). Dokumentacja mówi, że w
> przypadku powodzenia zwraca ona buf, podczas gdy w rzeczywistości
> zwraca ona miejsce w buf wskazujące na Null-a. Jednak libgadu nigdzie
> w kodzie nie polega na znaczeniu wskaźnika zwracanego przez tę
> funkcję, więc jest to sprawa raczej kosmetyczna.

Ta funkcja i tak prędzej czy później zniknie (na gałęzi new-api nie jest
używana), więc nie ma sensu jej naprawiać, a co najwyżej poprawić
dokumentację. I tak też zrobiłem.

> Natomiast drugą rzeczą, jaka rzuciła mi się w oczy co do tej funkcji,
> jest zwracanie NULL w przypadku gdy ret == 0, czyli gdy read() napotka
> na EOF. Wydaje mi się, że EOF w tym przypadku nie jest błędem i można
> spokojnie zwrócić buf.

Ta funkcja jest używana do nagłówków HTTP, a tam zamknięcie połączenia
zwykle nie jest niczym dobrym ;)

Pozdr,
Wojtek

_______________________________________________
libgadu-devel mailing list
libgadu-devel@lists.ziew.org
http://lists.ziew.org/mailman/listinfo/libgadu-devel

Reply via email to