Merge pull request #6 from progsource/make-sure-that-all-block-parsers-are-finished

Make sure that all block parsers are finished

.travis.yml

@@ -0,0 +1,18 @@
+dist: trusty
+sudo: false
+language: cpp
+compiler: g++
+install: export CXX="g++-6"
+addons:
+  apt:
+    sources:
+      - ubuntu-toolchain-r-test
+    packages:
+      - g++-6
+
+script:
+  - mkdir tmp
+  - cd tmp
+  - cmake ..
+  - make
+  - ../build/MaddyTests

CMakeLists.txt

@@ -1,4 +1,4 @@
-# This project is licensed under the MITlicense. For more information see the
+# This project is licensed under the MIT license. For more information see the
 # LICENSE file.
 
 cmake_minimum_required(VERSION 2.8)
@@ -27,7 +27,7 @@ file(GLOB_RECURSE MADDY_TESTS_FILES ${CMAKE_CURRENT_SOURCE_DIR}/tests/maddy/*.cp
 
 set(
   CMAKE_CXX_FLAGS
-  "${CMAKE_CXX_FLAGS} -g -std=c++${MADDY_CPP_VERSION} -Wall -Wpedantic -Wextra -Wno-ignored-qualifiers -fno-rtti -fno-exceptions"
+  "${CMAKE_CXX_FLAGS} -g -std=c++${MADDY_CPP_VERSION} -Wall -Wpedantic -Wextra -Wno-ignored-qualifiers -fno-rtti -fno-exceptions -fsanitize=address -fno-omit-frame-pointer"
 )
 
 # ------------------------------------------------------------------------------

README.md

@@ -1,7 +1,8 @@
 # maddy
 
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Version: 1.0.1](https://img.shields.io/badge/Version-1.0.0-brightgreen.svg)](https://semver.org/)
+[![Version: 1.0.3](https://img.shields.io/badge/Version-1.0.3-brightgreen.svg)](https://semver.org/)
+[![Build Status](https://travis-ci.org/progsource/maddy.svg?branch=master)](https://travis-ci.org/progsource/maddy)
 
 maddy is a C++ Markdown to HTML **header-only** parser library.
 
@@ -26,7 +27,7 @@ fitting my needs. So I simply wrote my own one.
 
 The supported syntax can be found in the [definitions docs](docs/definitions.md).
 
-## HowTo use
+## How to use
 
 To use maddy in your project, simply add the include path of maddy to yours
 and in the code, you can then do the following:
@@ -42,6 +43,25 @@ std::shared_ptr<maddy::Parser> parser = std::make_shared<maddy::Parser>();
 std::string htmlOutput = parser->Parse(markdownInput);
 ```
 
+## How to run the tests
+
+*(tested on Linux with
+[git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git) and
+[cmake](https://cmake.org/install/) installed)*
+
+Open your preferred terminal and type:
+
+```shell
+git clone https://github.com/progsource/maddy.git
+cd maddy
+git submodule update --init --recursive
+mkdir tmp
+cd tmp
+cmake ..
+make
+make test # or run the executable in ../build/MaddyTests
+```
+
 ## How to contribute
 
 There are different possibilities:

docs/definitions.md

@@ -58,7 +58,7 @@ results in
 
 ```
 
-* unorederd
+* unordered
   * list
   * items
     * in
@@ -69,14 +69,18 @@ results in
 results in
 ```html
 <ul>
-  <li>list</li>
-  <li>items
+  <li>unordered
     <ul>
-      <li>in</li>
-      <li>an</li>
+      <li>list</li>
+      <li>items
+        <ul>
+          <li>in</li>
+          <li>an</li>
+        </ul>
+      </li>
+      <li>hierarchy</li>
     </ul>
   </li>
-  <li>hierarchy</li>
 </ul>
 ```
 
@@ -89,13 +93,17 @@ results in
 * items
 
 ```
+
 results in
+
 ```html
+
 <ol>
   <li>ordered</li>
   <li>list</li>
   <li>items</li>
 </ol>
+
 ```
 
 ```
@@ -108,7 +116,9 @@ results in
   * hierarchy
 
 ```
+
 results in
+
 ```html
 <ol>
   <li>ordered</li>
@@ -116,7 +126,9 @@ results in
     <ol>
       <li>items</li>
       <li>in
-        <ol>an</ol>
+        <ol>
+          <li>an</li>
+        </ol>
       </li>
       <li>hierarchy</li>
     </ol>

include/maddy/emphasizedparser.h

@@ -41,7 +41,7 @@ public:
   void
   Parse(std::string& line) override
   {
-    static std::regex re("(?!`|<code>)\\*(?!`|</code>)([^\\*]*)\\*(?!`|</code>)");
+    static std::regex re("(?!.*`.*|.*<code>.*)\\*(?!.*`.*|.*<\\/code>.*)([^\\*]*)\\*(?!.*`.*|.*<\\/code>.*)");
     static std::string replacement = "<em>$1</em>";
 
     line = std::regex_replace(line, re, replacement);

include/maddy/parser.h

@@ -93,6 +93,18 @@ public:
       }
     }
 
+    // make sure, that all parsers are finished
+    if (currentBlockParser)
+    {
+      std::string emptyLine = "";
+      currentBlockParser->AddLine(emptyLine);
+      if (currentBlockParser->IsFinished())
+      {
+        result += currentBlockParser->GetResult().str();
+        currentBlockParser = nullptr;
+      }
+    }
+
     return result;
   }
 

include/maddy/strikethroughparser.h

@@ -39,7 +39,7 @@ public:
   void
   Parse(std::string& line) override
   {
-    static std::regex re("(?!`|<code>)\\~\\~(?!`|</code>)([^\\~]*)\\~\\~(?!`|</code>)");
+    static std::regex re("(?!.*`.*|.*<code>.*)\\~\\~(?!.*`.*|.*<\\/code>.*)([^\\~]*)\\~\\~(?!.*`.*|.*<\\/code>.*)");
     static std::string replacement = "<s>$1</s>";
 
     line = std::regex_replace(line, re, replacement);

include/maddy/strongparser.h

@@ -41,7 +41,7 @@ public:
   void
   Parse(std::string& line) override
   {
-    static std::regex re("(?!`|<code>)\\*\\*(?!`|</code>)([^\\*\\*]*)\\*\\*(?!`|</code>)");
+    static std::regex re("(?!.*`.*|.*<code>.*)\\*\\*(?!.*`.*|.*<\\/code>.*)([^\\*\\*]*)\\*\\*(?!.*`.*|.*<\\/code>.*)");
     static std::string replacement = "<strong>$1</strong>";
 
     line = std::regex_replace(line, re, replacement);

tests/maddy/test_maddy_emphasizedparser.cpp

@@ -23,8 +23,8 @@ TEST(MADDY_EMPHASIZEDPARSER, ItReplacesMarkdownWithEmphasizedHTML)
 
 TEST(MADDY_EMPHASIZEDPARSER, ItDoesNotParseInsideInlineCode)
 {
-  std::string text = "some text `*bla*` text testing *it* out";
-  std::string expected = "some text `*bla*` text testing <em>it</em> out";
+  std::string text = "some text `*bla*` `/**text*/` testing *it* out";
+  std::string expected = "some text `*bla*` `/**text*/` testing <em>it</em> out";
   auto emphasizedParser = std::make_shared<maddy::EmphasizedParser>();
 
   emphasizedParser->Parse(text);

tests/maddy/test_maddy_strikethroughparser.cpp

@@ -23,8 +23,8 @@ TEST(MADDY_STRIKETHROUGHPARSER, ItReplacesMarkdownWithStrikeThroughHTML)
 
 TEST(MADDY_STRIKETHROUGHPARSER, ItDoesNotParseInsideInlineCode)
 {
-  std::string text = "some text `~~bla~~` text testing <code>~~it~~</code> out";
-  std::string expected = "some text `~~bla~~` text testing <code>~~it~~</code> out";
+  std::string text = "some text `~~bla~~` ` ~~text~~ ` testing <code>~~it~~</code> out";
+  std::string expected = "some text `~~bla~~` ` ~~text~~ ` testing <code>~~it~~</code> out";
   auto strikeThroughParser = std::make_shared<maddy::StrikeThroughParser>();
 
   strikeThroughParser->Parse(text);

tests/maddy/test_maddy_strongparser.cpp

@@ -34,8 +34,8 @@ TEST(MADDY_STRONGPARSER, ItReplacesEmphasizedMarkdownNotWithStrongHTML)
 
 TEST(MADDY_STRONGPARSER, ItDoesNotParseInsideInlineCode)
 {
-  std::string text = "some text *bla* text testing `**it**` out";
-  std::string expected = "some text *bla* text testing `**it**` out";
+  std::string text = "some text **bla** `/**text**/` testing `**it**` out";
+  std::string expected = "some text **bla** `/**text**/` testing `**it**` out";
   auto strongParser = std::make_shared<maddy::StrongParser>();
 
   strongParser->Parse(text);

tests/main.cpp

@@ -8,7 +8,7 @@
 // -----------------------------------------------------------------------------
 
 int main (int argc, char** argv) {
-  ::testing::GTEST_FLAG(throw_on_failure) = false;
+  ::testing::GTEST_FLAG(throw_on_failure) = true;
   ::testing::InitGoogleMock(&argc, argv);
   return RUN_ALL_TESTS();
 }