Wsparcie #6056
Wyciąganie metadanych z korpusów Poliqarpowych - Code review
| Status: | Zamknięty | Start date: | 15 Sep 2014 | |
|---|---|---|---|---|
| Priority: | Niski | Due date: | ||
| Assignee: | Adam Radziszewski | % Done: | 0% | |
| Category: | - | |||
| Target version: | - |
Description
branch: poliqarp_metadata_retrieving
Wprowadziłem niewielkie modyfikacje w poliqarpclient.h/.cpp w celu pobierania metadanych. W funkcji pobierającej dokumenty dołożyłem flagę get_metadata. Metadane pobierane są jako słownik i następnie klucz po kluczy wpisywane jako atrybuty chunka. Testy przechodzą.
Proszę o krótki przegląd i ew. wmerdżowanie do mastera.
History
#1 Updated by Adam Radziszewski over 8 years ago
- Celowa była zmiana domyślnego zachowania w CMake? Teraz domyślnie instaluje PoliqarpReadera. Dla naszego zespołu to na pewno wygodniejsze, ale można by się zastanawiać nad kwestiami licencyjnymi. Instalacja go sprawia, że całość jest na GPL-u (bez niego na LGPL-u). Jeśli to zachowanie ma być domyślne, to być może warto by na Wiki zaznaczyć, że jeśli chcesz LGPL, to wyłącz Poliqarpa, bo on jest domyślnie (np. tam gdzie jest słowo „caution” albo to całe jakoś przeredagować).
- Bardzo przydałoby się choćby kilka słów komentarza przy metodzie
get_next_document, które wyjaśniłyby co to za metadane są i gdzie zostaną wstawione.
#2 Updated by Adam Radziszewski over 8 years ago
Skądinąd więcej uwag nie mam i kod nie wzbudza moich kontrowersji, więc można go będzie „wmerdżować” w mastera.
#3 Updated by Radosław Warzocha over 8 years ago
- Status changed from Przypisany to Zamknięty
Zgodnie z tym, o czym rozmawialiśmy: Komentarz dodany, zmiana domyślnego zachowania w CMake'u wycofana.
Jeśli chodzi o readery i writery XMLowe i pochodne, to okazuje się, że już powinny czytać i zapisywać atrybuty chunków. CCLowy nawet dba o to, żeby nie zapisać nic, poza tym, co jest w DTD.
Zmiany wmerdżowane do mastera.
Zamykam