« first day (2782 days earlier)      last day (398 days later) » 

Duga
Duga
00:00
REFRESH!
[Minesweeper] 101 Games Played. 58 Bombs Used. 14024 Moves Performed. 11 New Users
[rubberduck-vba/Rubberduck] 1 issues opened. 1 issue comments
[Zomis/Games] 76 additions. 34 deletions. 2 commits
 
2 hours later…
Duga
Duga
02:27
[rubberduck-vba/Rubberduck] Se7enSquared forked us into Se7enSquared/Rubberduck
 
6 hours later…
Duga
Duga
08:04
[rubberduck-vba/Rubberduck] interservices created comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> The fact is that in range(sheet2.cells(1,2), sheet2.cells(1,2)), range implicitly reference to sheet2 instead than (the whatever) 'ActiveSheet' as Code Inspector wrongly says.

To see that, try these combinations in the immediate window or in a module while keeping sheet1 active:
activesheet.range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> error 1004
range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> writes "abc" in cell "B1" of sheet2
sheet2.range(sheet2.cells(1,2), she
[rubberduck-vba/Rubberduck] interservices edited comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> The fact is that in range(sheet2.cells(1,2), sheet2.cells(1,2)), range implicitly reference to sheet2 instead than (the whatever) 'ActiveSheet' as Code Inspector says.

To see that, try these combinations in the immediate window or in a module while keeping sheet1 active:
activesheet.range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> error 1004
range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> writes "abc" in cell "B1" of sheet2
sheet2.range(sheet2.cells(1,2), sheet2.cell
[rubberduck-vba/Rubberduck] interservices edited comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> The fact is that in range(sheet2.cells(1,2), sheet2.cells(1,2)), range implicitly reference to sheet2 instead than (the whatever) 'ActiveSheet' as Code Inspector says.

To see that, try these combinations in the **immediate window** or in a **module** while **keeping sheet1 active**:
**activesheet.range**(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> **error 1004**
**range**(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> writes "abc" in cell "B1" of **sheet2**
**sheet2.range*
 
1 hour later…
Duga
Duga
09:17
[rubberduck-vba/Rubberduck] BStrauss3 opened issue #5932: Inappropriate "can be written as a function" opportunity
> Sub Increment(ByRef l as Long)
l = l + 1
End Sub

Usage:

Increment account

Is flagged as "can be written as a function". Not sure this is really any better:

Function Increment(ByVal l as Long) as Long
Increment = l + 1
End Function

Usage:

account = Increment(account)
[rubberduck-vba/Rubberduck] Vogel612 added label feature-inspection-quickfixes to issue #5932: Inappropriate "can be written as a function" opportunity
[rubberduck-vba/Rubberduck] Vogel612 added label support to issue #5932: Inappropriate "can be written as a function" opportunity
[rubberduck-vba/Rubberduck] Vogel612 added label feature-refactorings to issue #5932: Inappropriate "can be written as a function" opportunity
[rubberduck-vba/Rubberduck] Vogel612 created comment on issue #5932: Inappropriate "can be written as a function" opportunity
[rubberduck-vba/Rubberduck] Vogel612 closed issue #5932: Inappropriate "can be written as a function" opportunity
> Considering that this doesn't actually exhibit any unintended behavior, I don't think this issue is actionable. As such I am closing it :)
[rubberduck-vba/Rubberduck] Vogel612 edited comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> The fact is that in `range(sheet2.cells(1,2), sheet2.cells(1,2))`, `range` implicitly reference to `sheet2` instead of (the whatever) `ActiveSheet` as Code Inspector says.

To see that, try these combinations in the **immediate window** or in a **module** while **keeping sheet1 active**:
```
activesheet.range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> **error 1004**
range(sheet2.cells(1,2), sheet2.cells(1,2)) = "abc" ' -> writes "abc" in cell "B1" of **sheet2**
sheet2.range(s
 
3 hours later…
Duga
Duga
12:52
[rubberduck-vba/Rubberduck] retailcoder created comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> A shorter form referring to a single cell would be `wks.Cells(1, 2)` though.

The problem with the implicit inferred-from-inner-calls form, is that it works until it doesn't. Lookup error 1004 on Stack Overflow, you'll see just how frequently it trips people over. Code that does one thing written here but does something else written identically over there, is exactly what Rubberduck is warning about. Agreed that the target should probably not be hard-coded, but "implicit Active sheet reference
 
1 hour later…
Duga
Duga
13:52
[rubberduck-vba/Rubberduck] interservices created comment on issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
> Ok I didn't know that if I put my range(wks.cells,wks.cells) expression in a document (sheet) module, the implicit reference to Me **prevails** on that inferred-from_inner_calls.
This is sufficient for me to give up on this.
Duga
Duga
14:06
[rubberduck-vba/Rubberduck] kaszysz opened issue #5933: Excel is still calculating after use of Intender
[rubberduck-vba/Rubberduck] kaszysz added label bug to issue #5933: Excel is still calculating after use of Intender
> **Rubberduck version information**
The info below can be copy-paste-completed from the first lines of Rubberduck's log or the About box:

Rubberduck version [...]
Operating System: [Windows 11 Pro]
Host Product: [Excel 365]
Host Version: [2112 16.0.14729.20254 64bit]


**Description**
Whole project has been working as I excepted. I had used Intender and after that whole excel's file with vba program is crashing and cant event save it. In right bottom corrner is still
[rubberduck-vba/Rubberduck] BStrauss3 created comment on issue #5932: Inappropriate "can be written as a function" opportunity
> I think that's fair.

I dumped  RD into a large existing project. And 40 hours in I'm still finding it overwhelming.  You might consider offering a "minimal existing project" set to enable at first. But that's a different request.

⁣
-----Burton


On Jan 17, 2022, 3:18 AM, at 3:18 AM, Clemens Lieb ***@***.***> wrote:
>Considering that this doesn't actually exhibit any unintended behavior,
>I don't think this issue is actionable. As such I am closing it :)
>
>--
>Reply to this email directly o
 
2 hours later…
this
this
15:43
Refactoring a module with 5600 lines can be a bit scary - there's 2-3 seconds where module is empty while the rewriter reconstructs the module.
I also had to give up on the encapsulate field refactoring - it was sooo glacial just to navigate the huge list of fields to be encapsulated.
BZngr
BZngr
Wow...is it the only module in the project?
FreeMan
FreeMan
wow indeed! I thought my code was pretty bad...
Duga
Duga
[rubberduck-vba/Rubberduck] kaszysz closed issue #5933: Excel is still calculating after use of Intender
this
this
No, unfortunately, there are others, so it's adding to the work, obviously.
I can understand why - they want to keep it 100% self-contained so it'd just be "drop this module in and voila!"
but gee, not fun to make changes inside.
BZngr
BZngr
16:02
Dropping in a 5600 line module has to make an audible sound.
2
this
this
lol
If I ever have time I may need to refactor it into a twinBASIC project. Then things will be organized far more sensibly and there'll be just one DLL with one public interface.
Vogel612
Vogel612
sigh ... spotbugs is ... weird: github.com/spotbugs/spotbugs/issues/1925
@BZngr depends on your machine... some won't care, some will have to spin up the fan :D
BZngr
BZngr
:)
Duga
Duga
[rubberduck-vba/Rubberduck] Vogel612 created comment on issue #5933: Excel is still calculating after use of Intender
> @kaszysz did you intend to close this issue? If yes, did the problem resolve itself?
BZngr
BZngr
Or did he indent to close the intender issue?
this
this
16:17
must be the best hidden feature of RD. I did not even know it even had an intender.
FreeMan
FreeMan
LOL
Duga
Duga
16:31
[rubberduck-vba/Rubberduck] retailcoder closed issue #5931: wrong case of "member 'Range' implicitly references 'ActiveSheet'"
Mathieu Guindon
Mathieu Guindon
@Duga how is Excel's "calculating" state even related in any way with the VBA project??
this
this
16:52
@MathieuGuindon wouldn't it be a case of having a formula calling a vBA function?
HackSlash
HackSlash
17:18
Isn't the indenter one of the first things RD ever did?
Mathieu Guindon
Mathieu Guindon
17:32
almost, yeah.
Vogel612
Vogel612
we had our own indenter before we got the smartindenter stuff??
Mathieu Guindon
Mathieu Guindon
Actually "calculate" status is an indicator of pending recalc, most likely due to calculation mode being set to manual - something RD couldn't do if it tried
@Vogel612 no but we got the VB6 source in 2015 IIRC =)
this
this
More importantly, when did RD got its intender?!? Why didn't I know about that?
Mathieu Guindon
Mathieu Guindon
lol
FreeMan
FreeMan
Yeah, if I knew RD had an intender, I'd run it on all my code to get it to do what I intended, not what I wrote.
4
Duga
Duga
17:38
[rubberduck-vba/Rubberduck] retailcoder created comment on issue #5933: Excel is still calculating after use of Intender
> Can you confirm Excel's calculation mode is automatic? Rubberduck does not interact with Excel itself, or any other host application... We're going to need more detailed repro steps to replicate this issue.
 
2 hours later…
Duga
Duga
19:34
[rubberduck-vba/Rubberduck] philippetev starred us
2
Duga
Duga
20:21
[rubberduck-vba/Rubberduck] pawelsadlo2 starred us
2
 
3 hours later…
Duga
Duga
23:45
[rubberduck-vba/Rubberduck] 1498 stars vs. [decalage2/oletools] 1886 stars

« first day (2782 days earlier)      last day (398 days later) »