After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 609215 - [rsvg] fix compilation with MSVC
[rsvg] fix compilation with MSVC
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
0.10.17
Other Windows
: Normal normal
: 0.10.18
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-02-07 06:31 UTC by David Hoyt
Modified: 2010-02-09 07:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patchs gstrsvgdec.c (927 bytes, patch)
2010-02-07 06:31 UTC, David Hoyt
none Details | Review

Description David Hoyt 2010-02-07 06:31:49 UTC
Created attachment 153187 [details] [review]
Patchs gstrsvgdec.c

Visual C++ doesn't like #if's inside a statement it seems. The attached patch fixed the problem.
Comment 1 Tim-Philipp Müller 2010-02-07 10:19:54 UTC
Thanks for the patch.

Could you next time please supply a patch in git format-patch format? git format-patch -1 will create a patch from the last commit. (That way I won't have to type a commit message and your author name, and don't have to fix up the indenting).

commit beeb8fa8ecfc315fc6a04dd684d6616e012c6ea9
Author: David Hoyt <dhoyt@llnl.gov>
Date:   Sun Feb 7 11:09:45 2010 +0100

    rsvgdec: fix compilation with MSVC
    
    MSVC doesn't like #if #else within statements.
    
    Fixes #609215.
Comment 2 David Hoyt 2010-02-08 16:57:49 UTC
I apologize - I don't use git - I download the tarballs instead. Is there another program that would put them in the desired format? I don't mind hand editing the patches either so they match what you expect.
Comment 3 Tim-Philipp Müller 2010-02-09 07:59:27 UTC
Ah well, in that case don't worry about it. Hand-editing is likely too much hassle, but you could always do something like this:

  git config --global user.name "Your Name"
  git config --global user.email "you@foobar.com"

  git init .
  git add .
  git commit -m 'Import 0.10.25 tarball' 

on a freshly unpacked tarball. You can then modify your files and do

  git add dir/file1 file2 file3
  git commit

which will pop up a dialog where you can type a commit message, and then

  git format-patch -1

which will create a file called 0001-xyz.patch