4 comments
Comment actions Permalink

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

0
Comment actions Permalink

On 2003/12/23 16:26, Dave Griffith wrote:

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? ;)

Bas

0
Comment actions Permalink

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

0
Comment actions Permalink

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

0

Please sign in to leave a comment.