1 files changed, 27 insertions, 23 deletions

diff --git a/src/xml.cpp b/src/xml.cpp
index 44b423a..451ee18 100644
--- a/src/xml.cpp
+++ b/src/xml.cpp
@@ -1,18 +1,19 @@
 // Standard C++

+#include <array>

 #include <memory>

 #include <sstream>

+#include <span>

 

-// Projekt

+// Project

 #include "xml.hpp"

 

 // Expat XML Parser

-#define XML_STATIC              // force static linkage

 #include "expat.h"

 

 using namespace std;

 

 void

-XML::parse(istream& in)

+XML::parse(istream& ins)

 {

         unique_ptr<XML_ParserStruct, decltype(&XML_ParserFree)> parser(XML_ParserCreate(nullptr), XML_ParserFree);

 

@@ -22,54 +23,57 @@ XML::parse(istream& in)
         XML_SetCdataSectionHandler(parser.get(), cdata_start, cdata_end);

         XML_SetCharacterDataHandler(parser.get(), data);

 

-        char buffer[2048];

-        for ( bool done = false; !done ; ) {

-                in.read(buffer, sizeof buffer);

-                done = (in.gcount() < sizeof buffer);

+        static const auto buffer_size = 2048;

 

-                if ( XML_Parse(parser.get(), buffer, static_cast<int>(in.gcount()), done) != XML_STATUS_OK ) {

+        array<char, buffer_size> buffer{};

+

+        for ( bool done = false; !done; ) {

+                ins.read(buffer.data(), buffer_size);

+                done = ins.gcount() < buffer_size;

+

+                if ( XML_Parse(parser.get(), buffer.data(), static_cast<int>(ins.gcount()), static_cast<int>(done)) != XML_STATUS_OK ) {

                         stringstream msg;

 

                         msg << XML_ErrorString(XML_GetErrorCode(parser.get()));

                         msg << " in line " << XML_GetCurrentLineNumber(parser.get());

 

-                        throw runtime_error( msg.str() );

+                        throw runtime_error(msg.str());

                 }

         }

 }

 

 void

-XML::tag_start(void* me, const char* name, const char* attr[])

+XML::tag_start(void* self, const char* name, const char** attr)

 {

-        vector<pair<string, string>> attr_temp;

+        vector<tuple<string_view, string_view>> attr_temp;

 

-        for ( size_t i = 0; attr[i]; i += 2 )

-                attr_temp.push_back( make_pair<string, string>(attr[i], attr[i+1]) );

+        for ( auto i = 0; attr[i] != nullptr; i += 2 ) {

+                attr_temp.emplace_back(attr[i], attr[i + 1]);

+        }

 

-        reinterpret_cast<XML*>(me)->handle_start(name, attr_temp);

+        static_cast<XML*>(self)->handle_start(name, attr_temp);

 }

 

 void

-XML::tag_end(void* me, const char* name)

+XML::tag_end(void* self, const char* name)

 {

-        reinterpret_cast<XML*>(me)->handle_end(name);

+        static_cast<XML*>(self)->handle_end(name);

 }

 

 void

-XML::cdata_start(void* me)

+XML::cdata_start(void* self)

 {

-        reinterpret_cast<XML*>(me)->handle_cdata_start();

+        static_cast<XML*>(self)->handle_cdata_start();

 }

 

 void

-XML::cdata_end(void* me)

+XML::cdata_end(void* self)

 {

-        reinterpret_cast<XML*>(me)->handle_cdata_end();

+        static_cast<XML*>(self)->handle_cdata_end();

 }

 

 void

-XML::data(void* me, const char* dat, int len)

+XML::data(void* self, const char* dat, int len)

 {

-        reinterpret_cast<XML*>(me)->handle_data(vector<char>(dat, dat+len));

+        static_cast<XML*>(self)->handle_character_data({ dat, dat + len });

 }

-