Indeed it is wrong. See http://bugzilla.mozilla.org/show_bug.cgi?id=123187
Thanks,
Doug Turner
Tobias Oberstein wrote:
> i've been wading through nsISupportsImpl.h and by pure accident ran over
> the following snippet (which is from HEAD, but is the same for
> 0.9.7-RELEASE
> but on Line594)
>
> 647 #define NS_IMPL_THREADSAFE_RELEASE(_class)
> \
> 648 nsrefcnt _class::Release(void)
> \
> 649
> {
> \
>
>
> should'nt this read
>
> 647 #define NS_IMPL_THREADSAFE_RELEASE(_class)
> \
> 648 NS_IMETHODIMP_(nsrefcnt) _class::Release(void)
> \
> 649
> {
> \
>
>
> to get the correct calling conventions ??
>
> Toby.
>
>
>
>
>