[Pidgin] #11431: Signals no longer work in Perl plugins
Pidgin
trac at pidgin.im
Thu Mar 4 12:12:27 EST 2010
#11431: Signals no longer work in Perl plugins
----------------------+-----------------------------------------------------
Reporter: dharding | Owner: MarkDoliner
Type: defect | Status: new
Milestone: 2.7.0 | Component: plugins
Version: 2.6.6 | Resolution:
Keywords: perl |
----------------------+-----------------------------------------------------
Comment(by dharding):
Replying to [comment:10 MarkDoliner]:
> The original code behaved correctly, right? It just had a spurious
warning?
As best as I can tell, yes. I don't have a x86-64 system that I can test
Pidgin on, but running a simple test program on x86-64 seems to indicate
that the original code worked correctly on both x86 and x86-64, even
though it generated a warning when compiled on the latter platform.
> But I'm leaning toward just adding a cast for that specific case. How
do you feel about the attached patch coupled with
http://developer.pidgin.im/viewmtn/revision/info/67de3fff150c538c01ca66706c29a8f19cbfeff1?
As far as I can tell that should work - my testing indicates that it
should give the correct behavior on both platforms without any warnings.
It has been awhile since I delved into the complexities of C array/pointer
conversions, so I haven't figured out how on platforms where va_list is an
array, a va_list can be passed to another function as either a va_list or
a va_list* and still behave correctly - but it seems that it does in fact
behave correctly, so adding a cast seems the smallest fix to deal with the
warning.
--
Ticket URL: <http://developer.pidgin.im/ticket/11431#comment:11>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list