Wsparcie #6056

Wyciąganie metadanych z korpusów Poliqarpowych - Code review

Added by Radosław Warzocha about 8 years ago. Updated about 8 years ago.

Status:ZamkniętyStart date:15 Sep 2014
Priority:NiskiDue 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 about 8 years ago

Mam na wstępie kilka uwag:
  1. 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ć).
  2. 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 about 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 about 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

Also available in: Atom PDF