Results 36 comments of Devin Rousso

In C++, exposing the declared variable inside the `else` makes more sense as you can create custom `operator bool()`, meaning that the variable could actually still have useful data. AFAIK,...

I was thinking that both `a` and `b` would throw if used inside an `else` (or `else if`). ```js if (const a = 0) { // NOT_REACHED a // 0...

From the last line of the Overview: > ... and the variable would only be visible inside the if/while block. ```js if (const a = Math.random() < 0.5) { console.log(a);...

Although this isn't "technically" correct, when I see something like: ```js if (let data = foo.data) { ... } ``` I think that `data` is actually "part" of the `{...

From @michaelficarra's [comment](https://github.com/dcrousso/JS-Declarations-in-Conditionals/issues/3#issuecomment-537664355): > @ljharb Of course, that is good general practice, but we should not ignore that the proposal was presented as essentially "I am a C++ programmer and...

So I just added an options page. It didn't really add too much more to it, but it was a fun challenge. Please let me know if this is more...

this is in `Source/WebInspectorUI/UserInterface/Views/`. in this case i think only `WI.KeyboardShortcut` and `WI.Color` are requirements so we probably could, but not sure if it's worth it FWIW this isn't even...

> Ah yes, it's definitely important to not introduce a sudden behavior change with this fix. Although when I tested it I thought it's quite natural because, with this fix's...

> How do we like the following being our experimental settings text: "Show case insensitive code completion"? how about we flip it around as "Case Sensitive Autocomplete" with the default...

there may be some subtle shifts in behavior with case insensitive autocomplete for situations like `navigator` as that would now potentially also match `Navigator` (same for most things attached to...