Toshiaki Kameyama comments

Results 10 comments of


                                            Toshiaki Kameyama

KTIJ-20639 "Remove explicit type" breaks code for Long variables

Fixed

KTIJ-19745 False positive "Redundant 'suspend' modifier" with suspend lambda in constructor whish is used inside suspend method

I ran the `Redundant 'suspend' modifier` inspection on the entire intellij-community and found a warning in the android project.

KTIJ-19745 False positive "Redundant 'suspend' modifier" with suspend lambda in constructor whish is used inside suspend method

It seems that this issue has been resolved by 52a8d75c2cf545bee87579a8856249fc5847921e. This PR just adds a test for the issue.

KTIJ-19680 'Replace with a for loop' and backward conversion could work for 'forEachIndexed'

Fixed, please take another look. > 1. If the `for` loop has no braces, the loop body is lost after the conversion to `forEachIndexed`. 7d580831829c3f57d7c3b1e017cb13ec458ee0ff > 2. If the `forEachIndexed`...

KTIJ-19680 'Replace with a for loop' and backward conversion could work for 'forEachIndexed'

I'll resolve conflicts.

KTIJ-19680 'Replace with a for loop' and backward conversion could work for 'forEachIndexed'

> I'll resolve conflicts. Done

`UseAnyOrNoneInsteadOfFind` seemingly false positive for nullable collections

I believe this case isn't false positive and should be replaced with `list?.any { it == 4 } == true`.

KTIJ-19745 False positive "Redundant 'suspend' modifier" with suspend lambda in constructor whish is used inside suspend method

Added `IGNORE_FE10_BINDING_BY_FIR` to the test.

KTIJ-19680 'Replace with a for loop' and backward conversion could work for 'forEachIndexed'

Created a new [PR](https://github.com/JetBrains/intellij-community/pull/2311) for [KTIJ-19679](https://youtrack.jetbrains.com/issue/KTIJ-19679).

KTIJ-19680 'Replace with a for loop' and backward conversion could work for 'forEachIndexed'

Created a new [PR](https://github.com/JetBrains/intellij-community/pull/2317) for [KTIJ-19680](https://youtrack.jetbrains.com/issue/KTIJ-19680).