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 667287 - gstpad: Cast function pointer assigments to lhs type (gpointer)
gstpad: Cast function pointer assigments to lhs type (gpointer)
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Mac OS
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-01-04 19:50 UTC by Håvard Graff (hgr)
Modified: 2012-07-09 08:40 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (1.71 KB, patch)
2012-01-04 19:50 UTC, Håvard Graff (hgr) 		reviewed Details | Review

Description Håvard Graff (hgr) 2012-01-04 19:50:57 UTC 
Created attachment 204604 [details] [review]
patch

.
Comment 1 Tim-Philipp Müller 2012-01-04 23:58:07 UTC 
This looks fine, but why is it needed exactly? Anything can be assigned to void* without problems usually (in C anyway).
Comment 2 Akhil Laddha 2012-02-22 11:43:45 UTC 
Håvard, ping, could you please respond to comment#1 ?
Comment 3 Idar Tollefsen 2012-03-21 11:35:41 UTC 
(In reply to comment #1)
> This looks fine, but why is it needed exactly? Anything can be assigned to
> void* without problems usually (in C anyway).

Indeed, but some compilers will issue a warning about the implicit casts.

In this case, it was the SunPro compiler on Solaris. For other compilers, it's probably a matter of how high you set the warning level.
Comment 4 Tim-Philipp Müller 2012-07-09 08:40:11 UTC 
Hrm, ok. So I acknowledge that there's a minor issue here, but I think I'm just going to WONTFIX this at this point, since it's 0.10 specific, and I don't think the patch actually makes the code better in any way. It would probably have to be solved in a slightly more convoluted way, but that also isn't really particularly interesting for 0.10 at this point IMHO.