955704 - Add MOZ_FINAL to avoid lots of clang warnings

Status: RESOLVED FIXED

(Chat Core :: General, defect)

Description

[Florian Quèze [:florian]]

*** Original post on bio 2255 at 2013-11-23 18:33:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***

Comment 1

[Florian Quèze [:florian]]

Attachment: Patch

*** Original post on bio 2255 as attmnt 3077 at 2013-11-23 18:33:00 UTC ***

Compiling purplexpcom causes clang to output plenty of noisy warnings like:
purpleAccount.cpp
/home/florian/buildhg/hg.instantbird.org/purple/purplexpcom/src/purpleAccount.cpp:87:1: warning: 
      delete called on 'purpleChatRoomField' that has virtual functions but non-virtual destructor
      [-Wdelete-non-virtual-dtor]
NS_IMPL_ISUPPORTS1_CI(purpleChatRoomField, prplIChatRoomField)
^
../../../mozilla/dist/include/nsIClassInfoImpl.h:163:3: note: expanded from macro
      'NS_IMPL_ISUPPORTS1_CI'
  NS_IMPL_RELEASE(_class)                                                     \
  ^
../../../mozilla/dist/include/nsISupportsImpl.h:506:40: note: expanded from macro 'NS_IMPL_RELEASE'
  NS_IMPL_RELEASE_WITH_DESTROY(_class, delete (this))
                                       ^
../../../mozilla/dist/include/nsISupportsImpl.h:486:5: note: expanded from macro
      'NS_IMPL_RELEASE_WITH_DESTROY'
    _destroy;                                                                 \
    ^


These are easy to fix, we just have to add MOZ_FINAL where needed.

Comment 2

[Patrick Cloke [:clokep]]

Comment on attachment 8354858 [details] [diff] [review]
Patch

*** Original change on bio 2255 attmnt 3077 at 2013-11-23 18:35:58 UTC ***

I didn't test this, but it looks fine. rs=me.

Comment 3

[Patrick Cloke [:clokep]]

*** Original post on bio 2255 at 2013-11-23 19:10:42 UTC ***

http://hg.instantbird.org/instantbird/rev/769b8e95e228