have a global "are diagnostic asserts enabled" macro

RESOLVED WORKSFORME

Status

()

Core
MFBT
RESOLVED WORKSFORME
a year ago
a year ago

People

(Reporter: jcristau, Unassigned, Mentored)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lang=C++])

(Reporter)

Description

a year ago
after bug 1357799 code that needs to check whether MOZ_DIAGNOSTIC_ASSERT expands to anything needs to check something like "defined(DEBUG) || !defined(RELEASE_OR_BETA) || defined(EARLY_BETA_OR_EARLIER)" which gets unwieldy.  Having a single macro for this purpose would make things easier to understand.
Mentor: sledru
Keywords: good-first-bug
Whiteboard: [lang=C++]

Comment 1

a year ago
Hi @SylvestreLedru,
I am currently studying in my final year of Computer Science and Engineering 
SRM University ,India

I would would like to take this as my first bug fix , please consider on assigning this to me.
Thanks !!
Flags: needinfo?(sledru)

Comment 2

a year ago
Hi @SylvestreLedru,
I am currently studying in my second year of Cybernetics Engineering Systems and Information Technologies
UNIPA University ,Italy

I would would like to take this as my first bug fix , please consider on assigning this to me.
Thanks !!

Comment 3

a year ago
@SylvestreLedru
please provide the info regarding the bug
Actually, we might not need to do that, sorry :/
Flags: needinfo?(sledru)
Keywords: good-first-bug
I was implemented in bug 1365460
(I didn't marked it as duplicate because 1365460 is very different)
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.