[mapserver-commits] [MapServer/MapServer] 3abd4d: mapcairo.c: avoid double scaling of SVG symbols (f...

[Johannes Eberenz]

  Branch: refs/heads/main
  Home:   https://github.com/MapServer/MapServer
  Commit: 3abd4dc9d5dc8c0eb834e2db509e46636f6d4c3c
      https://github.com/MapServer/MapServer/commit/3abd4dc9d5dc8c0eb834e2db509e46636f6d4c3c
  Author: Johannes Eberenz <johanez at users.noreply.github.com>
  Date:   2026-08-11 (Tue, 11 Aug 2026)

  Changed paths:
    M msautotest/renderers/expected/line_marker_svg.png
    M msautotest/renderers/expected/point_svg.png
    M msautotest/sld/expected/map_linemark.png
    M msautotest/sld/expected/sld_linemark.png
    M msautotest/sld/expected/sld_url_linemark.png
    M msautotest/wxs/expected/point_symbol_identification_anchor.png
    M msautotest/wxs/expected/wms_get_map_symbol_ext_svg_sld.png
    M msautotest/wxs/point_symbol_identification.map
    M src/mapcairo.c

  Log Message:
  -----------
  mapcairo.c: avoid double scaling of SVG symbols (fixes: #7430) (#7583)

* mapcairo.c: revert to svg_handle_render_cairo() (fixes: #7430)

Since 8.6, rsvg_handle_render_cairo() (marked as depriceated) was replaced with the newer rsvg_handle_render_document(), however this caused scaling issues (#7430). This PR implements a simple workaround by reverting back to the legacy function, solving the issues.

* fix svg scaling by using rsvg_handle_render_document without cairo_scale

* correctly rendered test result

* update test coordinates for correctly rendeted SVG

* use symbol size for rsvg_handle_render_document and check vor librscg version

This allows cairo and older rsvgr rendering with scale_cairo>
Also fixed incorrect test png file name

* update expected results

---------

Co-authored-by: Jeff McKenna <jmckenna at gatewaygeomatics.com>



To unsubscribe from these emails, change your notification settings at https://github.com/MapServer/MapServer/settings/notifications