1538081 - add a C++ macro that acts like Rust's dbg!()

Status: RESOLVED FIXED

(Core :: XPCOM, enhancement)

Description

[Cameron McCormack (:heycam)]

Lately I've found the Rust dbg!() macro very useful! I'd like something in C++ to use that works the same way -- that is, prints out the expression passed to it, then returns it.

Comment 1

[Cameron McCormack (:heycam)]

Attachment: Bug 1538081 - Part 1: Add MOZ_DBG macro.

Comment 2

[Cameron McCormack (:heycam)]

Happy to hear ideas for a better macro name too.

Comment 3

[Cameron McCormack (:heycam)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e7afa5680fb18f0c39eb46e5d95db31e5c9b64a4

Comment 4

[Cameron McCormack (:heycam)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7d933f22e9d6b4de2f2a61bf81d54cd0e1e953d1

Comment 5

[Cameron McCormack (:heycam)]

Attachment: Bug 1538081 - Part 2: Add MOZ_DEFINE_DBG.

Depends on D24583

Comment 6

[Cameron McCormack (:heycam)]

Attachment: Bug 1538081 - Part 3: Add operator<<(ostream&) definitions for a few common types.

Depends on D25023

Comment 7

[Cameron McCormack (:heycam)]

Attachment: Bug 1538081 - Part 4: Tests.

Depends on D25024

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/113ac188a69e
Part 1: Add MOZ_DBG macro. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/26df801e44db
Part 2: Add MOZ_DEFINE_DBG. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/d6d331abbf0e
Part 3: Add operator<<(ostream&) definitions for a few common types. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/ced61a86c74c
Part 4: Tests. r=froydnj,gerald

Comment 9

[Andrei Ciure[:aciure]]

Backed out 4 changesets (bug 1538081) for causing BaseElf.cpp bustages

push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&selectedJob=237046627&revision=ced61a86c74c6ce2a966b2ac7880cfe3e7a8e0d8

failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=ced61a86c74c6ce2a966b2ac7880cfe3e7a8e0d8&selectedJob=237052662&searchStr=android%2C4.2%2Cx86%2Copt%2Cbuild-android-x86%2Fopt%2C%28b%29

backout: https://hg.mozilla.org/integration/autoland/rev/4e1afd84043705a5e072873af3d4d7957e639ba3

Comment 10

[Cameron McCormack (:heycam)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f5e95bb143f9bd5a5c7198ea42758534aeea72f7

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b9e91906edae
Part 1: Add MOZ_DBG macro. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/3ae52b39cf6f
Part 2: Add MOZ_DEFINE_DBG. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/a01e770ef4f5
Part 3: Add operator<<(ostream&) definitions for a few common types. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/ac91a1c2e3f5
Part 4: Tests. r=froydnj,gerald

Comment 12

[Cameron McCormack (:heycam)]

Attachment: Bug 1538081 - Followup fixes.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/58a56c6b8543
Followup fixes.

Comment 14

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/b9e91906edae
https://hg.mozilla.org/mozilla-central/rev/3ae52b39cf6f
https://hg.mozilla.org/mozilla-central/rev/a01e770ef4f5
https://hg.mozilla.org/mozilla-central/rev/ac91a1c2e3f5
https://hg.mozilla.org/mozilla-central/rev/58a56c6b8543