Issue 2517: add xslt render lib in cloud storage #2659
Conversation
|
Hi! Thank you for contributing! |
| @@ -0,0 +1,43 @@ | |||
| #include "xslt-render.h" | |||
There was a problem hiding this comment.
мы дефис в именах файлов не используем, используем _
| return out.Str(); | ||
| } | ||
|
|
||
| void TestAddChilds(auto&& name, auto&& value) |
There was a problem hiding this comment.
Childs -> Children
и не вижу поводов передавать rvalue reference - почему бы не сделать так, как мы в остальном коде делаем - pass by value, а дальше std::move
There was a problem hiding this comment.
это не rvalue, а forward reference
childs на children заменил
|
|
||
| //////////////////////////////////////////////////////////////////////////////// | ||
|
|
||
| Y_UNIT_TEST_SUITE(XsltRenderTest) |
| @@ -0,0 +1,37 @@ | |||
| #pragma once | |||
|
|
|||
| #ifdef THROW | |||
There was a problem hiding this comment.
если от этих дефайнов вообще никак невозможно избавиться, то нужно их спрятать в xml_document.cpp, чтобы они через хедер не проникали в остальной наш код
сделать TXmlNodeWrapper через какой-нибудь pimpl
There was a problem hiding this comment.
ну там проблема как раз в том, что есть хедеры, которые инклудят конфиги и вот это defs.h, где внутри тот же макрос THROW, поэтому раз я хочу использовать xml-document, то мне надо убрать макрос именно в хедере
There was a problem hiding this comment.
я могу конечно написать враппер для работы со всей либой cpp/xml/document, но звучит хуже, чем эта кучка ифдефов
| xmlCleanupGlobals(); | ||
| xmlCleanupParser(); | ||
| } | ||
| }; |
There was a problem hiding this comment.
пустой строки не хватает после };
|
|
||
| #include "xml-document.h" | ||
|
|
||
| #include <contrib/libs/libxml/include/libxml/globals.h> |
There was a problem hiding this comment.
опять же лучше сделать через pimpl, чтоб этих жутких хедеров не было в наших хедерах - чтоб они в остальной код через include-ы не заезжали
github-actions
bot
removed
the
ok-to-test
| class TXmlNodeWrapper final | ||
| { | ||
| public: | ||
| enum ESource |
|
|
||
| explicit TXmlNodeWrapper(std::unique_ptr<TData> data); | ||
|
|
||
| std::unique_ptr<TData> Data = nullptr; |
There was a problem hiding this comment.
явно инициализировать кажется нет смысла
#2517
часть от пр #2588
здесь создание либы для генерации xml
добавил тесты (тест кейс взял из репы либы https://github.com/GNOME/libxslt/blob/master/tests/REC/test-12.2-1.out)
добавил структуру TXslRenderer, чтобы можно было один раз передать и распрасить xslt