mark more classes as final

RESOLVED FIXED in mozilla35

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: tbsaunde, Assigned: tbsaunde)

Tracking

unspecified
mozilla35
x86_64
Linux
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 8480112 [details] [diff] [review]
mark more classes MOZ_FINAL
Attachment #8480112 - Flags: review?(nfroyd)
Comment on attachment 8480112 [details] [diff] [review]
mark more classes MOZ_FINAL

Review of attachment 8480112 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the HAVE_VISIBILITY changes reverted.

::: memory/mozalloc/mozalloc_abort.h
@@ +14,5 @@
>  // do nothing: it's been defined to __declspec(dllexport) by
>  // mozalloc*.cpp on platforms where that's required
>  #elif defined(XP_WIN)
>  #  define MOZALLOC_EXPORT __declspec(dllimport)
> +#elif __GNUC__ || __clang__

Um, different patch?

::: memory/mozalloc/mozalloc_oom.h
@@ +14,5 @@
>  // do nothing: it's been defined to __declspec(dllexport) by
>  // mozalloc*.cpp on platforms where that's required
>  #elif defined(XP_WIN)
>  #  define MOZALLOC_EXPORT __declspec(dllimport)
> +#elif defined(__GNUC__) || defined(__clang__)

Here too.

::: mfbt/Types.h
@@ +39,5 @@
>   */
>  #if defined(WIN32)
>  #  define MOZ_EXPORT   __declspec(dllexport)
>  #else /* Unix */
> +#  if __GNUC__ || __clang__

Here too.
Attachment #8480112 - Flags: review?(nfroyd) → review+
(Assignee)

Comment 3

4 years ago
> r=me with the HAVE_VISIBILITY changes reverted.

erph, yeah that stuff is a silly mistake on my part.
https://hg.mozilla.org/mozilla-central/rev/26c4989e824c
Assignee: nobody → trev.saunders
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.