walkero
/
nghttp2
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

src/HtmlParser: Code cleanup

This commit is contained in:
Tatsuhiro Tsujikawa 2013-10-02 23:08:47 +09:00
parent a9ba00914e
commit d90e0d4262
2 changed files with 39 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

74
src/HtmlParser.cc
View File

@ -36,7 +36,7 @@ ParserData::ParserData(const std::string& base_uri)
HtmlParser::HtmlParser(const std::string& base_uri) HtmlParser::HtmlParser(const std::string& base_uri)
: base_uri_(base_uri), : base_uri_(base_uri),
parser_ctx_(0), parser_ctx_(nullptr),
parser_data_(base_uri) parser_data_(base_uri)
{} {}
@ -80,10 +80,10 @@ void start_element_func
const xmlChar *name, const xmlChar *name,
const xmlChar **attrs) const xmlChar **attrs)
{ {
ParserData *parser_data = reinterpret_cast<ParserData*>(user_data); auto parser_data = reinterpret_cast<ParserData*>(user_data);
if(util::strieq(reinterpret_cast<const char*>(name), "link")) { if(util::strieq(reinterpret_cast<const char*>(name), "link")) {
const char *rel_attr = get_attr(attrs, "rel"); auto rel_attr = get_attr(attrs, "rel");
const char *href_attr = get_attr(attrs, "href"); auto href_attr = get_attr(attrs, "href");
if(!href_attr) { if(!href_attr) {
return; return;
} }
@ -93,13 +93,13 @@ void start_element_func
add_link(parser_data, href_attr, REQ_PRI_MEDIUM); add_link(parser_data, href_attr, REQ_PRI_MEDIUM);
} }
} else if(util::strieq(reinterpret_cast<const char*>(name), "img")) { } else if(util::strieq(reinterpret_cast<const char*>(name), "img")) {
const char *src_attr = get_attr(attrs, "src"); auto src_attr = get_attr(attrs, "src");
if(!src_attr) { if(!src_attr) {
return; return;
} }
add_link(parser_data, src_attr, REQ_PRI_LOWEST); add_link(parser_data, src_attr, REQ_PRI_LOWEST);
} else if(util::strieq(reinterpret_cast<const char*>(name), "script")) { } else if(util::strieq(reinterpret_cast<const char*>(name), "script")) {
const char *src_attr = get_attr(attrs, "src"); auto src_attr = get_attr(attrs, "src");
if(!src_attr) { if(!src_attr) {
return; return;
} }
@ -111,38 +111,38 @@ void start_element_func
namespace { namespace {
xmlSAXHandler saxHandler = xmlSAXHandler saxHandler =
{ {
0, // internalSubsetSAXFunc nullptr, // internalSubsetSAXFunc
0, // isStandaloneSAXFunc nullptr, // isStandaloneSAXFunc
0, // hasInternalSubsetSAXFunc nullptr, // hasInternalSubsetSAXFunc
0, // hasExternalSubsetSAXFunc nullptr, // hasExternalSubsetSAXFunc
0, // resolveEntitySAXFunc nullptr, // resolveEntitySAXFunc
0, // getEntitySAXFunc nullptr, // getEntitySAXFunc
0, // entityDeclSAXFunc nullptr, // entityDeclSAXFunc
0, // notationDeclSAXFunc nullptr, // notationDeclSAXFunc
0, // attributeDeclSAXFunc nullptr, // attributeDeclSAXFunc
0, // elementDeclSAXFunc nullptr, // elementDeclSAXFunc
0, // unparsedEntityDeclSAXFunc nullptr, // unparsedEntityDeclSAXFunc
0, // setDocumentLocatorSAXFunc nullptr, // setDocumentLocatorSAXFunc
0, // startDocumentSAXFunc nullptr, // startDocumentSAXFunc
0, // endDocumentSAXFunc nullptr, // endDocumentSAXFunc
&start_element_func, // startElementSAXFunc &start_element_func, // startElementSAXFunc
0, // endElementSAXFunc nullptr, // endElementSAXFunc
0, // referenceSAXFunc nullptr, // referenceSAXFunc
0, // charactersSAXFunc nullptr, // charactersSAXFunc
0, // ignorableWhitespaceSAXFunc nullptr, // ignorableWhitespaceSAXFunc
0, // processingInstructionSAXFunc nullptr, // processingInstructionSAXFunc
0, // commentSAXFunc nullptr, // commentSAXFunc
0, // warningSAXFunc nullptr, // warningSAXFunc
0, // errorSAXFunc nullptr, // errorSAXFunc
0, // fatalErrorSAXFunc nullptr, // fatalErrorSAXFunc
0, // getParameterEntitySAXFunc nullptr, // getParameterEntitySAXFunc
0, // cdataBlockSAXFunc nullptr, // cdataBlockSAXFunc
0, // externalSubsetSAXFunc nullptr, // externalSubsetSAXFunc
0, // unsigned int initialized 0, // unsigned int initialized
0, // void * _private nullptr, // void * _private
0, // startElementNsSAX2Func nullptr, // startElementNsSAX2Func
0, // endElementNsSAX2Func nullptr, // endElementNsSAX2Func
0, // xmlStructuredErrorFunc nullptr, // xmlStructuredErrorFunc
}; };
} // namespace } // namespace
@ -158,7 +158,7 @@ int HtmlParser::parse_chunk(const char *chunk, size_t size, int fin)
return -1; return -1;
} else { } else {
if(fin) { if(fin) {
return parse_chunk_internal(0, 0, fin); return parse_chunk_internal(nullptr, 0, fin);
} else { } else {
return 0; return 0;
} }

1
src/HtmlParser.h
View File

@ -74,7 +74,6 @@ namespace nghttp2 {
class HtmlParser { class HtmlParser {
public: public:
HtmlParser(const std::string& base_uri) {} HtmlParser(const std::string& base_uri) {}
~HtmlParser() {}
int parse_chunk(const char *chunk, size_t size, int fin) { return 0; } int parse_chunk(const char *chunk, size_t size, int fin) { return 0; }
const std::vector<std::string>& get_links() const { return links_; } const std::vector<std::string>& get_links() const { return links_; }
void clear_links() {} void clear_links() {}