Add command line support for EMF and WMF export

Bug #1244749 reported by David Mathog on 2013-10-25
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Wishlist
David Mathog

Bug Description

suv sent an email in which Martin Owens and/or Krzysztof Kosiński noted that there was no command line support for the new cross platform EMF/WMF features.

The attached patch adds that. It removes the WIN32 conditionals and adds "-m" and "--export-wmf" for WMF.

EMF still uses the same "-M" and "--export-emf" as it did before lp988601 was merged back into trunk.

AIl 4 of "-M", "-m", "--export-emf", and "--export-wmf" were tested one at a time on linux. All produced the expected output file, with no errors or warnings noted.

All 4 modes were tested in a DOS window under Windows XP, again successfully producing the expected output, with no errors or warnings noted.

One of you might want to rearrange the code in main.cpp after the patch is applied slightly. Both EMF and WMF eventually go through do_export_win_metafile_common() via do_export_emf() and do_export_wmf(), which are just wrappers around the first one. They could probably just as easily call the common method directly, but I did not test that.

David Mathog (mathog) wrote :
David Mathog (mathog) wrote :

Clarification - the patch removes the WIN32 conditionals related to EMF export in main.cpp.
No other WIN32 conditionals are affected.

su_v (suv-lp) on 2013-10-25
tags: added: cli emf exporting
tags: added: wmf
su_v (suv-lp) on 2013-10-25
Changed in inkscape:
assignee: nobody → David Mathog (mathog)
importance: Undecided → Wishlist
status: New → In Progress
su_v (suv-lp) wrote :

On 2013-10-25 20:23 +0200, David Mathog wrote:
> AIl 4 of "-M", "-m", "--export-emf", and "--export-wmf" were tested one
> at a time on linux. All produced the expected output file, with no
> errors or warnings noted.

Also tested successfully on OS X 10.7.5.

su_v (suv-lp) on 2013-10-26
summary: - Add command line support for EMF and WMF
+ Add command line support for EMF and WMF export
David Mathog (mathog) wrote :

Please commit.

jazzynico (jazzynico) wrote :

Committed revision 12809.
Thanks for the patch!

Changed in inkscape:
milestone: none → 0.49
status: In Progress → Fix Committed
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers