cverbiest

I'll try it. Is the parameter name ok ? On Tue, Jan 22, 2019, 19:32 Gilles QUERRET Might be done. Interested in opening a pull request ? > > —...

I tried adding  to the exclude keywords but that does not help. Can this rule be updated

I agree with Jurjen, In general `trim(x, "\")` is valid code. It could be a use case for a custom rule. Unless you are absolutely sure you will never have...

I agree , unless the method is called DeleteObject ;-) . So there should be an option to suppress issue.

Some comments, with unintended risk of hijacking the issue. ``` FIND FIRST ttFoo. FOR EACH lbFoo WHERE lbFoo.cField1 = "1":U AND ttFoo.cField2 = lbFoo.cField2 ``` Is valid for the compiler,...

@lneumeier I totally agree Progress should provide better performance and a better compiler. Sonar-openedge & OEDT are two great products helping us to cope with some of the shortcomings of...

It's related to https://openedge.ideas.aha.io/ideas/OPENEDGE-I-714. I stopped using procedure editor because of this and wrote my own basic editor that runs code without clean-up

Maybe an "unused valid object" rule ? Fired whenever an object is checked for validity without being used. If created I think valid-handle and valid-object should be treated equally. cave-at,...

With my suggestion the unused valid object rule would flag object a on the second if. ```if valid-handle( a ) then delete object a. /* this if get flagged */...

Could we use the ABLDuck documentation syntax to mark a method as obsolete ? https://github.com/Riverside-Software/pct/wiki/ABLDuck#deprecated ``` /** * @deprecated */