Toad World® Forums

Code Analysis: 6405 not working any more?


#1

The unused variable is not detected any more:

DECLARE
  unused NUMBER;
BEGIN
  NULL;
END;

I know that this is a legacy rule, has it been disabled?
Is there a new rule instead of it?


#2

Hi Peter,
Sorry for my late answer, it seems that my forum notification setting got lost.
Rule 6405 (“Avoid defining variables that are not referenced inside the program”) is legacy indeed, and internally marked as defective. We need a better implementation. I’m sorry to say that this make take a while.
Thanks,
Andre


#3

Hi Andre,

While the old implementation was not perfect, I would absolutely prefer to have it back for now than to have no check at all.

What do you think?

Thanks, Peter


#4

Peter I understand. Let me look into our options. I’ll get back asap.

Thanks,
Andre


#5

Peter, FYI, we added a “limited” XPath based implementation in upcoming drop 5.341 which will handle scalars inside a program unit. Hope this will do for now.

Thanks,
Andre