1629192 - Add some uses of selection statements with initializers to the tree

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task)

Description

[Jeff Walden [:Waldo]]

In other words, if (T v = ...; cond) and switch (T v = ...; expr).

Some fumbling regexp searches in mozilla-central indicate no uses of either syntax yet. And it's not on the "Using" page yet, either. gcc 7 (we require 7.1) and clang 3.9 (we require 5.0) and MSVC 2017 15.3 (we use fold expressions that require MSVC 2017 15.5) were the first versions to provide support, so we should be good...but there's no test like a clean try-run and actual in-tree use to prove fitness. Let's add some uses!

Comment 1

[Jeff Walden [:Waldo]]

Attachment: Bug 1629192 - Add uses of C++17's new |if (T v = ...; cond)| and |switch (T v = ...; value)| syntax to verify their usability with current compiler requirements. r=sfink!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dafc4c96797f
Add uses of C++17's new |if (T v = ...; cond)| and |switch (T v = ...; value)| syntax to verify their usability with current compiler requirements.  r=sfink

Comment 3

[Cristian Brindusan [:cbrindusan]]

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

Comment 4

[Jeff Walden [:Waldo]]

I updated https://developer.mozilla.org/en-US/docs/Mozilla/Using_CXX_in_Mozilla_code to note this syntax is safe to use.