code inspection bug with ++

return findLink(firstLink, toModel, i, ++depth);

The value changed at ++depth at line 200 is never used.

hope it's ok to post it here - is quicker than having
to first search if someone else already posted it to
tracker ... ;)

2 comments
Comment actions Permalink

On 21-04-2004 15:22, Michael Damberger wrote:

return findLink(firstLink, toModel, i, ++depth);

The value changed at ++depth at line 200 is never used.

hope it's ok to post it here - is quicker than having
to first search if someone else already posted it to
tracker ... ;)


That one is reported very often. It's a feature apparently;-)
see: http://www.intellij.net/tracker/idea/viewSCR?publicId=11942

Bas

0
Comment actions Permalink

ok, read the disuccsion in the tracker.

in my opinion the warning should not occur.
it is nothing wrong with the code.

I WANT to increase depth by 1 and then pass it as parameter, which is exactly what happens.
I DO NOT CARE if depth is not used any more AFTER the method call.
And I do also not care that I could also write depth+1
instead.

But I do care about a warning for not a real problem at
all - understandable?

M.








0

Please sign in to leave a comment.