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.
> 
> 
> 
> 
> 


Reply via email to