Opened 14 years ago

Closed 12 years ago

#3343 closed defect (wontfix)

Add SVG Symbol Capabilities

Reported by: theduckylittle Owned by: zjames
Priority: normal Milestone: FUTURE
Component: MapServer C Library Version: svn-trunk (development)
Severity: normal Keywords:
Cc: assefa, zjames, tbonfort, nhermann

Description

Add the SVG capabilities detailed in GSoC_SVG_Symbols_SECOND_DRAFT_OF_THE_MS_RFC.

I've deviated from the RFC in one important way, instead of creating new SVG and SVGPATH directives I re-used the "IMAGE" directive. "IMAGE" points to the SVG file in the same way that "IMAGE" points to the image/pixmap file for PIXMAP type SYMBOLS.

Change History (8)

comment:1 by theduckylittle, 14 years ago

Status: newassigned

Add initial version to AGG renderer in r9866.

comment:2 by tomkralidis, 14 years ago

FYI I'm getting a build error on trunk (fc12):

In file included from renderers/agg/include/agg_svg_parser.h:24,
                 from mapagg.cpp:82:
renderers/agg/include/agg_svg_path_renderer.h:300: error: extra qualification 'mapserver::svg::path_renderer::' on member 'get_path_storage'
make: *** [mapagg.o] Error 1

comment:3 by theduckylittle, 14 years ago

Also reported by tomkralidis:

./libmapserver.a(mapagg.o): In function `loadSymbolSVG':
/home/tkralidi/foss4g/mapserver/svn/trunk/mapserver/mapagg.cpp:226: undefined reference to `mapserver::svg::path_renderer::path_renderer()'
/home/tkralidi/foss4g/mapserver/svn/trunk/mapserver/mapagg.cpp:227: undefined reference to `mapserver::svg::parser::parser(mapserver::svg::path_renderer&)'
/home/tkralidi/foss4g/mapserver/svn/trunk/mapserver/mapagg.cpp:234: undefined reference to `mapserver::svg::parser::parse(char const*)'
/home/tkralidi/foss4g/mapserver/svn/trunk/mapserver/mapagg.cpp:274: undefined reference to `mapserver::svg::parser::~parser()'
/home/tkralidi/foss4g/mapserver/svn/trunk/mapserver/mapagg.cpp:274: undefined reference to `mapserver::svg::parser::~parser()'
collect2: ld returned 1 exit status
make: *** [shp2img] Error 1

Solution:

Run autoconf.

comment:4 by tomkralidis, 14 years ago

FYI I get some build warnings (fc12):

renderers/agg/src/agg_svg_parser.cpp: In member function 'void mapserver::svg::parser::parse(const char*)':
renderers/agg/src/agg_svg_parser.cpp:246: warning: format '%d' expects type 'int', but argument 4 has type 'XML_Size'
renderers/agg/include/agg_svg_path_tokenizer.h: In constructor 'mapserver::svg::path_tokenizer::path_tokenizer()':
renderers/agg/include/agg_svg_path_tokenizer.h:103: warning: 'mapserver::svg::path_tokenizer::m_last_command' will be initialized after
renderers/agg/include/agg_svg_path_tokenizer.h:102: warning:   'double mapserver::svg::path_tokenizer::m_last_number'
renderers/agg/src/agg_svg_path_tokenizer.cpp:37: warning:   when initialized here
renderers/agg/src/agg_svg_path_tokenizer.cpp: In member function 'bool mapserver::svg::path_tokenizer::parse_number()':
renderers/agg/src/agg_svg_path_tokenizer.cpp:123: warning: suggest parentheses around '&&' within '||'

comment:5 by tbonfort, 13 years ago

Cc: assefa zjames tbonfort added
Milestone: 6.0 releaseFUTURE

this won't be happening in 6.0.

comment:6 by assefa, 13 years ago

Owner: changed from theduckylittle to zjames
Status: assignednew

setting this to 6.2

comment:7 by nhermann, 13 years ago

Cc: nhermann added

comment:8 by tbonfort, 12 years ago

Resolution: wontfix
Status: newclosed

closing as svg symbols are supported through cairo

Note: See TracTickets for help on using tickets.