Age | Commit message (Collapse) | Author |
|
|
|
build: only attempt to install MSVC system libraries on Windows
|
|
Newer versions of CMake attempt to query the system for information about the VS
2017 installation. Unfortunately, this query fails on non-Windows systems when
cross-compiling:
cmake_host_system_information does not recognize <key> VS_15_DIR
CMake will not find these system libraries on non-Windows hosts anyways, and we
were silencing the warnings, so simply omit the installation when
cross-compiling to Windows.
|
|
Closes #251.
|
|
|
|
|
|
These affect both parsing and writing commonmark.
|
|
|
|
|
|
add link parsing corner cases to regressions
|
|
|
|
With current HTML escaping, sometimes we may produce an XML tag like
<code_block> </code_block>
Many XML parsers consider these spaces insignificant and strip them
out but we need this. There's actually a test case like this in
spec.txt (search "A code block can have all empty lines as its
content:") and without proper hinting, an external xml->html converter
will fail the spec.
XML standard covers this case. If xml:space is "preserve", then
whitespaces are significant and should be kept. Add this hint for
text, code, code_block, html_inline and html_block tags.
|
|
|
|
Added regression test.
See commonmark/CommonMark#527.
|
|
- commented out the (already ignored) "many references" test, which
times out
- reduced the iterations for a couple other tests
|
|
|
|
We were needlessly redoing things we'd already done.
Now we skip the work if the first nonspace is greater
than the current offset.
This fixes pathological slowdown with deeply nested
lists (#255). For N = 3000, the time goes from over
17s to about 0.7s.
Thanks to @mity for diagnosing the problem.
|
|
To conform to recent spec change.
|
|
|
|
See commonmark/CommonMark#487.
|
|
This fixes a recently added failing spec test case.
Previously spaces were being allowed in unquoted attribute
values; no we forbid them.
|
|
See commonmark/CommonMark#497.
|
|
|
|
Allow the `width` parameter to be generated too so we get better fuzz-coverage.
|
|
We currently discard fuzz test-cases that are empty but empty inputs are valid
markdown. This improves the fuzzing coverage slightly.
|
|
For empty documents, `->size` is zero so
`renderer.buffer->ptr[renderer.buffer->size - 1]` will cause an out-of-bounds
read. Empty buffers always point to the global `cmark_strbuf__initbuf` buffer
so we read `cmark_strbuf__initbuf[-1]`.
|
|
|
|
Test static-only build
|
|
|
|
|
|
Rename the tests to reflect that they use the library, not the
executable.
|
|
Thanks @mohitsinghs
|
|
Generate export header for static-only build
|
|
Fixes issue #247.
|
|
|
|
This allows us to include tests that we don't yet know
how to pass.
|
|
Tests must complete in 8 seconds or are errors.
|
|
|
|
Add more pathological tests.
|
|
> 32 nested balanced parens in a link is bananas
|
|
Add sourcepos info for inlines
|
|
|
|
|
|
|
|
Include GNUInstallDirs in src/CMakeLists.txt
|
|
Fixes installation error under some CMake versions, notably kalakris'
CMake backport often used with Travis.
|
|
|
|
Due to a mistake, 0.28.1 installed libcmark.a into include/.
Closes #238.
|
|
|
|
For some reason this wasn't getting set in processing
libcmark.pc.in, and we were getting the wrong entry in libcmark.pc.
(See #236)
The new approach sets an internal libdir variable to
lib${LIB_SUFFIX}. This variable is used both to set the
install destination and in the libcmark.pc.in template.
Closes #236.
However, I'd welcome comments from @juhp who originally
added CMAKE_INSTALL_LIBDIR in #185. I think that the new
system should work fine with Fedora, since LIB_SUFFIX will
be set appropriately, but some testing is in order.
|