12.9.0.43 - Code Analysis - Wrong Rule 5401 Avoid using an IN OUT parameter as IN only

Toad for Oracle - Beta
1

The following code shows “Rule 5401 Avoid using an IN OUT parameter as IN only” which is wrong:

CREATE TYPE test_type AS OBJECT ( test_member NUMBER );

CREATE PROCEDURE test_proc(
in_test IN PLS_INTEGER – marked with Rule 5401 Avoid using an IN OUT parameter as IN only.
)
AS
v_test_type test_type;
BEGIN
v_test_type.test_member := in_test;
END;

2

Peter,

That and 5402 both have some issues. They are old legacy rules that have not been migrated to the XPATH rule engine. The issue is with variable scope and they need some redesign.

Thanks for reporting.

Greg

From: Peter Lang [mailto:bounce-PeterLang@toadworld.com]

Sent: Wednesday, March 23, 2016 5:04 AM

To: toadoraclebeta@toadworld.com

Subject: [Toad for Oracle - Beta Discussion Forum] 12.9.0.43 - Code Analysis - Wrong Rule 5401 Avoid using an IN OUT parameter as IN only.

12.9.0.43 - Code Analysis - Wrong Rule 5401 Avoid using an IN OUT parameter as IN only.

Thread created by Peter Lang

The following code shows “Rule 5401 Avoid using an IN OUT parameter as IN only” which is wrong:

CREATE TYPE test_type AS OBJECT ( test_member NUMBER );

CREATE PROCEDURE test_proc(

in_test IN PLS_INTEGER – marked with Rule 5401 Avoid using an IN OUT parameter as IN only.

)

AS

v_test_type test_type;

BEGIN

v_test_type.test_member := in_test;

END;

To reply, please reply-all to this email.

Stop receiving emails on this subject.

Or
Unsubscribe from Toad for Oracle - Beta notifications altogether.

Toad for Oracle - Beta Discussion Forum

Flag
this post as spam/abuse.

3

But this one was clearly a typo in the code base. I fixed it.

Thanks for letting us know.

Andre

4

It is not really fixed, the issue appears still in the actual Beta 12.9.0.46 from 31.03.2016.

5

Well, actually I meant “fixed for one of the next betas”. The fix is in component QP5.dll version 5.294. Because it’s a component, not Toad itself, it may take up to a couple of weeks before you get it.