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 775293 - schro: #include math.h before schroedinger headers
schro: #include math.h before schroedinger headers
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Windows
: Normal normal
: 1.11.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-11-29 00:08 UTC by Scott D Phillips
Modified: 2016-11-29 09:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[PATCH gst-plugins-bad] schro: #include math.h before schroedinger headers (2.14 KB, patch)
2016-11-29 00:09 UTC, Scott D Phillips
committed Details | Review

Description Scott D Phillips 2016-11-29 00:08:48 UTC
The schroedinger headers unconditionally #define over C99's rint when
compiling with msvc which messes up the later inclusion of math.h.
Including math.h before schroedinger headers avoids getting syntax
errors in math.h
Comment 1 Scott D Phillips 2016-11-29 00:09:42 UTC
Created attachment 340953 [details] [review]
[PATCH gst-plugins-bad] schro: #include math.h before schroedinger  headers
Comment 2 Tim-Philipp Müller 2016-11-29 09:52:30 UTC
Thanks!

commit 70ba84ee1215ca8fc1166f34663f8da8b0adbe7d
Author: Scott D Phillips <scott.d.phillips@intel.com>
Date:   Mon Nov 28 16:02:03 2016 -0800

    schro: #include math.h before schroedinger headers
    
    The schroedinger headers unconditionally #define over C99's rint when
    compiling with msvc which messes up the later inclusion of math.h.
    Including math.h before schroedinger headers avoids getting syntax
    errors in math.h
    
    https://bugzilla.gnome.org/show_bug.cgi?id=775293