1406206 - Clean up unneeded else in NS_INTERFACE_MAP_ENTRIES_CYCLE_COLLECTION

Status: RESOLVED FIXED

(Core :: XPCOM, enhancement)

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

clang-tidy is complaining about an extra else in NS_INTERFACE_MAP_ENTRIES_CYCLE_COLLECTION. Not hurting anything since it's a macro, but could be cleaned up anyways.

https://bugzilla.mozilla.org/show_bug.cgi?id=1403819#c7

Comment 1

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Attachment: Bug 1406206 - Remove extraneous else block from cycle collection macros;

clang-tidy is complaining about an extra else in
NS_INTERFACE_MAP_ENTRIES_CYCLE_COLLECTION. Not hurting anything, but
could be cleaned up anyways.

Review commit: https://reviewboard.mozilla.org/r/227876/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227876/

Comment 2

[Nathan Froyd [:froydnj]]

Comment on attachment 8959013 [details]
Bug 1406206 - Remove extraneous else block from cycle collection macros;

https://reviewboard.mozilla.org/r/227876/#review233852

Stealing review!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fbdaeb6f64ca
Remove extraneous else block from cycle collection macros; r=froydnj

Comment 4

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/fbdaeb6f64ca