Aw, hell. This one again. Something about IDEA's new layout stuff is causing the new declaration to be wrapped. Thought I had it licked. I'll look into it. With luck, there will be an IPP bug fix by next week, and this will be in it.
Aw, hell. This one again. Something about IDEA's new layout stuff is causing the new declaration to be wrapped. Thought I had it licked. I'll look into it. With luck, there will be an IPP bug fix by next week, and this will be in it.
Inspection gadgets had a quick fix for conditional expressions, but it isn't there anymore (and it had the same flaw). Was there any reason to remove it? Or am I losing my memory and it was never there in the first place? Or am I going blind and it is there, but I can't see it? ;)
IG never had such a quick fix, because it's doing searches for any conditional expression. IPP searches for conditional expressions it can actually turn to if statements (basically just return of a conditional, assignment to a conditional, and declaration of a single local variable with a conditional initializer, for those keeping track at home). Conditionals which can't be replaced by if statements (for example, as method arguments) are ignored by IPP, but are flagged by IG as a design flaw. Adding the smarts to IG so that it could have quick-fixes for this case (and a couple of similar ones) has been on my TODO list for a while.
Aw, hell. This one again. Something about IDEA's new layout stuff is causing the new declaration to be wrapped. Thought I had it licked. I'll look into it. With luck, there will be an IPP bug fix by next week, and this will be in it.
--Dave
On 2003/12/23 16:26, Dave Griffith wrote:
Inspection gadgets had a quick fix for conditional expressions, but it
isn't there anymore (and it had the same flaw). Was there any reason to
remove it? Or am I losing my memory and it was never there in the first
place? Or am I going blind and it is there, but I can't see it? ;)
Bas
IG never had such a quick fix, because it's doing searches for any conditional expression. IPP searches for conditional expressions it can actually turn to if statements (basically just return of a conditional, assignment to a conditional, and declaration of a single local variable with a conditional initializer, for those keeping track at home). Conditionals which can't be replaced by if statements (for example, as method arguments) are ignored by IPP, but are flagged by IG as a design flaw. Adding the smarts to IG so that it could have quick-fixes for this case (and a couple of similar ones) has been on my TODO list for a while.
--Dave
no problem, just enjoy your holiday ;)
with such a great thing (your plugin) I could live with a lot of more defects,
thanks man,
-m