Skip to content
Snippets Groups Projects
  1. Dec 10, 2018
  2. Nov 27, 2018
    • Arseny Kapoulkine's avatar
      tests: Only use load_file_special_folder test on macOS · 7664bbf9
      Arseny Kapoulkine authored
      The behavior on Linux is very different between kernel versions, and it
      triggers an unexpected OOM during sanitizer runs because somehow the
      size is reported to be LONG_MAX. It's not clear that it helps us cover
      any paths we don't cover otherwise - it would be nice to be able to test
      failing to load a multi-gigabyte file on a 32-bit system, but we can't
      do this easily atm anyway.
      7664bbf9
    • Arseny Kapoulkine's avatar
      Enable config=sanitize in Travis CI · 1a9c3f66
      Arseny Kapoulkine authored
      This commit changes sanitize configuration to fail on the first error
      and ignore floating-point division and overflow "errors" that trigger
      when we test the corresponding functionality. This makes it possible to
      run this on all commits - if new UB or memory safety issues are introduced,
      asan/ubsan will catch them.
      1a9c3f66
  3. Nov 24, 2018
    • Arseny Kapoulkine's avatar
      Fix Wdouble-promotion warnings · f9a2a7d1
      Arseny Kapoulkine authored
      We had a few places in test code and library source where we used an
      implicit float->double cast; while it should preserve the value exactly,
      gcc/clang implement this warning to make sure uses of double are intentional.
      
      This change also adds the warning to Makefile to make sure we don't
      regress on this warning.
      
      Fixes #243.
      f9a2a7d1
  4. Nov 20, 2018
    • Arseny Kapoulkine's avatar
      Escape TAB character in attribute values with 	 · aac75cd2
      Arseny Kapoulkine authored
      This change modifies the table entries for ctx_special_attr to treat TAB
      character as special, which makes the output code escape it.
      
      Before this change, trying to use TAB in an attribute value would output
      it verbatim; during subsequent parsing, pugixml - and other compliant
      parsers - would apply attribute-value normalization, turning the TAB
      into a space and losing the original value.
      
      Using 	 fixes this; if an input document has 	 in an attribute
      value, that gets unescaped into \t during parsing and escaped back into
      	 during output, which means we can now roundtrip values like this.
      
      Fixes #242.
      aac75cd2
  5. Nov 16, 2018
  6. Nov 12, 2018
  7. Oct 24, 2018
    • Arseny Kapoulkine's avatar
      XPath: Workaround Coverity false positive · d9fadc74
      Arseny Kapoulkine authored
      Coverity hits a similar false positive to what clang static analyzer hit
      - it assumes that since optimize() checks _right for being nullptr,
      optimize_self() might hit _right=nullptr in the ast_op_equal case which
      is impossible.
      
      Contributes to #236.
      d9fadc74
  8. Oct 16, 2018
    • Lipsa, Dan's avatar
      Remove warning in Visual Studio (#235) · 273fa0ab
      Lipsa, Dan authored
      The following warning is removed:
      Visual Studio 14.0
      1. warning C4275: non dll-interface class 'std::exception' used as
         base for dll-interface class 'vtkpugixml::xpath_exception'
      273fa0ab
  9. Sep 25, 2018
    • Arseny Kapoulkine's avatar
      Work around clang --analyze warnings · 81c82588
      Arseny Kapoulkine authored
      clang doesn't understand the invariants guaranteed for specific AST node
      types and, when seeing null pointer checks in optimize(), assumes any
      pointers in the node might be null. Work around this by adding explicit
      - redundant - null pointer checks.
      81c82588
    • Arseny Kapoulkine's avatar
      XPath: Refactor xpath_node_set short buffer optimization · e3b5e9ce
      Arseny Kapoulkine authored
      This change replaces xpath_node_set single element storage with a
      single-element array in hopes that this would silence Coverity false
      positive about getting a singleton pointer.
      
      Additionally, it refactors _assign member to unify small and large
      buffer codepaths since they are basically identical.
      
      Fixes #233 (hopefully)
      e3b5e9ce
  10. Aug 14, 2018
  11. Aug 08, 2018
  12. Jul 30, 2018
  13. Jul 28, 2018
  14. Jul 24, 2018
    • Arseny Kapoulkine's avatar
      Add .gitattributes file · f3139f4c
      Arseny Kapoulkine authored
      This makes sure the contents of tests/data/ folder does not go through
      newline conversion, which breaks tests that rely on some files having LF
      and some files having CR+LF.
      
      Fixes #222.
      f3139f4c
  15. Jul 23, 2018
  16. Jun 26, 2018
  17. Jun 20, 2018
  18. May 17, 2018
  19. Apr 27, 2018
    • Arseny Kapoulkine's avatar
      Move CMake build postfix setup behind an off-by-default USE_POSTFIX · 51322cff
      Arseny Kapoulkine authored
      This setup can interfere with existing workflows in two ways:
      
      - If the target application used CMake and configured custom postfixes,
      this change would override them
      
      - If the target application did *not* use CMake, it'd have to abide by
      these conventions even if the target configuration used is unexpected -
      for example, the default "preferred" configuration is frequently
      RelWithDebugInfo, not Release, which now has a postfix.
      
      Fixes #198.
      51322cff
  20. Apr 15, 2018
  21. Apr 12, 2018
  22. Apr 11, 2018
  23. Apr 09, 2018
    • Eli Schwartz's avatar
      cmake: always install the pkg-config file (#193) · daeb8013
      Eli Schwartz authored
      There's really never a reason to *not* want this installed. If an option
      is needed to specify installing in a versioned subdirectory, this option
      should be explicitly described rather than hidden in something else.
      
      As an added bonus, this makes the CMake install code slightly *less*
      complicated.
      daeb8013
  24. Apr 04, 2018
  25. Apr 03, 2018
Loading