Rubberduck

Rubberduck copied to clipboard

The grammar doesn't enforce maximum identifier lengths (see #2855), yet we assume that no parser error means the code is compilable. We need an error-level inspection result to pop for...

retailcoder

Inspection Results should show QualifiedMemberName instead of QualifiedModuleName as appropriate

1

Some inspections are module level, but others are specific to members of a module. Inspection Results' Location column should be more specific about the location by including the location's member...

ThunderFrame

https://msdn.microsoft.com/en-us/library/office/gg278477.aspx > The `LayoutEffect` property is read-only and is available only in the `Layout` event. The `Layout` event is only fired for the following controls: - Frame/Frame2 - MultiPage/MultiPage2 -...

ThunderFrame

Object reference could be held by With block

2

A *language opportunity* inspection: ````vb Dim something As Object Set something = New Object With something .Foo = 42 Debug.Print .Bar End With ```` If a local variable is only...

retailcoder

For example: `Excel.Application.HinstancePtr` should be preferred over `Excel.Application.Hinstance`

ThunderFrame

RD needs an inspection for use of `Static` modifiers for procedures.

1

OK, I'll say it. There is no good use case for these. This... ``` Static Sub Foo() Dim bar As Long bar = bar + 1 Debug.Print bar End Sub...

comintern

Inspection for boolean expression parsed to function expecting string parameter

1

While typing some code, I inadvertently made the following typo while copying and pasting. ```vb If Right$(prp.Name "EmMacro", 7) Then ``` When I meant to type: ```vb If Right$(prp.Name,7) "EmMacro"...

ThunderFrame

Inspection for redundant variable initializations

1

If a variable's first assignment (taking code paths into account) is a literal with the default value for that variable's declared type, Rubberduck should suggest removing that redundant assignment. ````vb...

retailcoder

Inspection for inconsistent use of `Option Compare` across Access VBA modules

1

When executing the following procedure, in modules declared with different `Option Compare` statements, the user will get different results: ```vb Sub test() Debug.Print InStr(1, "abc", "ABC") End Sub ``` There...

ThunderFrame

Inspection for use of New in project/module-level object variables

1

https://msdn.microsoft.com/en-us/library/aa241733(v=vs.60).aspx > The object is created the first time the variable is used in code. This incurs a slight increase in overhead on **each** function call. > [The instance] incurs...

ThunderFrame