Hacker News Reader

Comment by ithrow | Hacker News Reader

Intellij/Webstorm doesn't "just work", at least for javascript: https://youtrack.jetbrains.com/issue/WEB-57184/wrong-warning...

It's a real turn off that a commercial IDE can't get fundamental stuff like the one show in the bug tracker correct (or fixed in a timely manner). You start to question why you are even paying for the thing.

View on HN

There are some annoyances like the caches and the indexing time when reinstalling node modules but I'm an FEE and I use it everyday productively for FE. The fundamentals are all sound, my favorite feature is the debugger, git integration and test runner.

I can't work out what is actually wrong here - there appears to be very little detail, would you mind explaining?

False warning about unused definition 'salary' when it is actually being used.

Thank you for explaining - I didn't think to click on the thumbnail as I thought the snippet _is_ the expanded thumbnail.

The way I see/read it: it _isn't_ being used in respect to nothing _reads_ it, except within the calculation to raise - there's nothing that reads the result of the raise. It's warning you that you are (potentially) needlessly capturing state. I agree it's a valid warning (for that specific example.)

I appreciate it is just an example to demonstrate the issue but I am unable to deduce a scenario where that wouldn't be a valid warning.

'salary' is part of the constructor's body and the constructor is being used, it shouldn't warn. Look how it just warns about 'salary' but not about 'name', in the image, only 'salary' is gray out. It's not even consistent.

That's just a warning, though. Annoying, but it's not like you can't work because of it.

Reply on news.ycombinator.com