On Fri, 6 May 2011 13:33:21 +0200, Alessio Treglia <ales...@debian.org>
wrote:
> Hi Matthias,
> 
> [...]
>> + src/projectM-libvisual/actor_projectM.cpp |    2 +-
>> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> +
>> +--- projectm.orig/src/projectM-libvisual/actor_projectM.cpp
>> ++++ projectm/src/projectM-libvisual/actor_projectM.cpp
>> +@@ -209,7 +209,7 @@ extern "C" int lv_projectm_events (VisPl
>> +  * we aren't with projectm, so just ignore :) */
>> + extern "C" VisPalette *lv_projectm_palette (VisPluginData *plugin)
>> + {
>> +-      return NULL;
>> ++      return (VisPalette *) NULL;
>> + }
>> +
>> + /* This is where the real rendering happens! This function is what we
>> call, many times
> 
> the above patch makes projectm build fine with GCC4.6 in sid.
> Your review would be appreciated, let me know what you think.
Looks excellent, thanks! (Applied it upstream too)
I didn't know how to fix this, and I still find it very strange... Why do
GCC need a cast of a NULL pointer?
Anyway, I think we should upload this soon, I only want to take a second
look at bug #621789 before.
Cheers,
  Matthias



_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to