Hello!

Since the last update I implemented the following visitors:

• length_subtraction
• constructor_check
• explicitly_annotated_unittest

Also finished the work on some visitors I started before:

• redundant_attributes
• label_var_same_name
• local_imports

Some of those need the approval from my mentor but they are mostly done.

Also spent some time "debugging" - meaning comparing differences between
the actual D-Scanner and my version when applied to phobos. While some
checks might be subjective, I found scenarios where the actual D-Scanner
is not issueing any warnings where it's supposed to. So in order to be able
to apply my version to phobos I also fixed some style errors in phobos:

• selective_local_imports
• same_name_vars
• redundant_attributes

At least in my view, in all the above situations D-Scanner is
supposed to issue warnings and it's not catching them, so I think
that my implementation, in some particular cases at least, will
be more complete than the actual implementation.

Thank you!

On Monday, 14 November 2022 at 14:35:46 UTC, Lucian Danescu wrote:

Awesome work. This is really needed.