| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
|
|
| |
feature
The only thing that librsvg-c's tests use from rsvg::test_utils is the
load_svg() function. That function is trivial enough that we can just
copy it to librsvg-c's tests, and thus avoid having a whole feature
specification just for that.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
It was (probably by accident) under build-dependencies before
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
| |
Now that C files are scattered around, we'll just let grcov warn about
their paths which contain "../".
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
| |
Now that https://github.com/mozilla/grcov/issues/845 is fixed, we can
remove the workaround.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
script
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
| |
With clang 15.0.7, its utility libraries were in
/usr/lib64/clang/15.0.7/lib/linux. However, with clang 16, they are
now in /usr/lib64/clang/16/lib/linux (e.g. without minor/micro version
numbers). Extract just the major version from `clang --version`.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
| |
They were being written to ./rsvg/coverage-profiles instead of
./coverage-profiles, which is what the gen-coverage.sh script expects.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Avoids generating lots of duplicated code in tests.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
autotools requires them, it seems, but it has worked so far...?
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
A bunch of unquoted http links and XML element names, mostly.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
|
| |
Do not use a symlink; automake doesn't understand them for distcheck.
The fixtures that are duplicated from rsvg/tests/fixtures are just a
few, and they are small. I prefer having rsvg-convert's fixtures in a
single place instead of referencing something like ../../blahblah.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Otherwise cargo doesn't pick them up as tests.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
The C tests were moved to librsvg-c/tests-c, so update things accordingly
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While casually reading
https://hackmd.io/mxdn4U58Su-UQXwzOHpHag (Analysis of
rustc-benchmarking-data), I noticed that it mentions
https://github.com/Lokathor/tinyvec/issues/161.
The tinyvec crate, with the default set of features, can compile on
rustc < 1.55 by using large implementations of Default for [T; N]. On
Rustc >= 1.55, it can elide this code and just use const generics.
Apparently this reduces compilation time for tinyvec by about half -
which is not a lot in the context of librsvg, but let's make use of
that.
Librsvg currently requires Rust >= 1.64.0 anyway, so we can make use
of tinyvec's rustc_1_55 feature.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
| |
Even though they are not part of the autotools build yet, they do get
referenced in the Cargo workspace.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Autotools doesn't use them except for making release tarballs.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Seems like it was inadvertently renamed.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
| |
* Rename the librsvg-c project's library back to "librsvg_c". Rust
will then build target/whatever/liblibrsvg_c.a.
* Build librsvg_c as an rlib too, so rsvg-convert can use it.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Also, change the crate-type to staticlib in librsvg-c as it was before.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
|
|
|
| |
librsvg-c/src/lib.rs just had "pub mod c_api;", so let's pull
everything one directory up.
This will also make it slightly easier to fix the toplevel Makefile.am.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
| |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
The NMake Makefiles need a small update so that the test sources can be located
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|
|
|
|
|
| |
Closes #950
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/822>
|
|\
| |
| |
| |
| | |
Remove the current viewport as mutable state from DrawingCtx
See merge request GNOME/librsvg!821
|
| |
| |
| |
| | |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/821>
|
| |
| |
| |
| |
| |
| | |
The actual viewport stack is now implied as arguments in the call stack.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/821>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- All the drawing functions get a new argument of type &Viewport. The
idea is that the caller computes a new Viewport and just passes a
reference to it. The viewport stack is implied by the call stack,
instead of being represented as mutable state in DrawingCtx. This
removes DrawingCtx::get_viewport() and ::get_top_viewport().
- Rename Svg::push_viewport() to make_svg_viewport().
- Remove DrawingCtx::get_viewport_for_units(); replace it with Viewport::from_units().
- Remove DrawingCtx::push_coord_units(), which was just being called
in generate_cairo_mask() - this takes care of the TODO that
mentioned exactly that goal.
- Remove DrawingCtx::push_view_box(); replace it with Viewport::with_view_box().
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/821>
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ViewParams was the original set of values that we used since 2018 for
length normalization.
Viewport was my attempt to start unifying the "viewport" concept,
around 2020 and #298.
My current plan is to remove the current viewport from the mutable
part of DrawingCtx, and instead pass the current viewport as function
arguments to the draw() functions.
This commit basically moves the fields and methods from ViewParams
into Viewport, and makes everything use Viewport instead of
ViewParams. Things are a bit funky because of the viewport_stack
field, which is shared via an Option<Weak<>> to the viewport stack in
the DrawingCtx - but that will disappear when the current viewport is
passed as arguments instead of mutable state.
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/821>
|
|\
| |
| |
| |
| | |
CI: bump the container image
See merge request GNOME/librsvg!817
|
| |
| |
| |
| | |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/817>
|
| |
| |
| |
| | |
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/817>
|