« first day (2509 days earlier)      last day (671 days later) » 

Duga
Duga
12:02 AM
RELOAD!
[rubberduck-vba/Rubberduck] 1 issue comment.
[Minesweeper] Games Played: 107, Bombs Used: 73, Moves Performed: 14030, New Users: 13
 
 
3 hours later…
Duga
Duga
2:49 AM
 
Duga
Duga
3:12 AM
[rubberduck-vba/Rubberduck] build for commit 947deea6 on unknown branch: AppVeyor build failed
BUILD FAILURE!
 
Mathieu Guindon
Mathieu Guindon
Argh
 
Duga
Duga
 
Duga
Duga
3:49 AM
[rubberduck-vba/Rubberduck] build for commit d032b222 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] retailcoder pushed commit 96329362 to next: inspection is looking for Sheets.Item and Sheets._Default member calls
[rubberduck-vba/Rubberduck] retailcoder pushed commit fae2b92d to next: disable tests for SheetAccessUsingString pending a way to mock ITypeLib API determining the document module supertypes.
[rubberduck-vba/Rubberduck] retailcoder pushed commit 947deea6 to next: fixes broken tests
[rubberduck-vba/Rubberduck] retailcoder pushed commit d032b222 to next: fix broken test
Merge pull request #5726 from retailcoder/fixStringSheet

Tweaks worksheet inspection to look for Sheets.Item and Sheets._Default member calls; adjusted tests to correctly load worksheet/workbook supertypes.
 
Mathieu Guindon
Mathieu Guindon
is it weird if I merge the version bump to [next] but not yet into [main]?
til [main] in SE chat is a magic link to Code Review
 
Duga
Duga
4:20 AM
[rubberduck-vba/Rubberduck] build for commit 236805ad on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] retailcoder pushed commit adf018e9 to next: Bump version to 2.5.2
[rubberduck-vba/Rubberduck] retailcoder pushed commit 0c84c2d0 to next: updated splash, deleted corrupt designer
[rubberduck-vba/Rubberduck] retailcoder pushed commit b7490612 to next: updated splash background
[rubberduck-vba/Rubberduck] retailcoder pushed commit 426c846b to next: tweak version label bgcolor
[rubberduck-vba/Rubberduck] retailcoder pushed commit c009f175 to next: made version label transparent, load image into picturebox
Merge pull request #5706 from retailcoder/bump252

Bump version to 2.5.2
 
Duga
Duga
4:44 AM
[rubberduck-vba/Rubberduck] build for commit edd5fb38 on next: AppVeyor build succeeded
 
 
3 hours later…
Duga
Duga
7:19 AM
> Version 2.5.2.5814
OS: Microsoft Windows NT 10.0.19042.0, x64
Host Product: Microsoft Office x64
Host Version: 16.0.13901.20400
Host Executable: EXCEL.EXE

The **ImplicitActiveSheetReference** test seems to be broken.

The following code:

```
With shNotOK
.Range(.Range("A2").Resize(UBound(arrNotOK, 1), UBound(arrNotOK, 2)).Address).Formula = arrNotOK
End With
```
where **shNotOK** is
```
Set shNotOK = ActiveWorkBook.Sheets("NotOK")
```
and **arrNotOK** is two-dimentional
 
Duga
Duga
7:51 AM
 
Duga
Duga
8:09 AM
 
Duga
Duga
8:28 AM
> This addition broke the `ImplicitActiveSheetReferenceInspection`. We either need to make this list virtual or remove this base class entirely.

Ref #5735
 
 
2 hours later…
FreeMan
FreeMan
10:33 AM
@Hosch250 Howdy!! Good to see you.
 
 
1 hour later…
Duga
Duga
11:40 AM
> Noticed more "false alarms" for code like this one:
```
sh.Range(....
```
where sh is some worksheet.
> Noticed some more "false alarms" for code like this one:
```
sh.Range(....
```
where sh is some worksheet.
 
Mathieu Guindon
Mathieu Guindon
fml
 
Duga
Duga
> Noticed some more "false alarms" for code like this one:
```
sh.Range(....
```
where sh is a variable for some worksheet.
> Noticed some more "false alarms" for code like this one:
```
sh.Range(....
```
where **sh** is a variable for some worksheet.
> Noticed some more "false alarms" for code like this one:
```
sh.Range(....
```
where **sh** is some variable, pointing to some worksheet.
> Noticed some more "false alarms" for code like this one:
```
sh.Range(....
```
where **sh** is variable, pointing to some worksheet.
 
Freeflow
Freeflow
I'm playing with twinbasic. I converted a solution to Project Euler Question 14 from VBA to twinBasic (I just had to change debug.print statements to Msgbox due to preview restrictions). The VBA code takes 51 minutes to run to completion. twinBasic took just 3 seconds.!!!!!
3
 
Duga
Duga
12:06 PM
> Thanks for reporting this.

The cause of the issue is the change from the referenced commit. It will likely occur for all calls to the members on variables of type `Excel.Worksheet`.
 
FreeMan
FreeMan
@Freeflow that's... an improvement!
Reminds me of a place I worked at back in '96. They'd hired a contractor to write a rather large program in VB (6?, 5?, I dunno) and it took forever to run. The rather acerbic owner/boss was unhappy with that and demanded the contractor make it faster. When he was unable to do so, he hired another to rewrite the thing in a different language that would be faster.
The new contractor wrote it in Delphi, but couldn't get it completed and was summarily caned canned. He then called me over (I was hired just after delivery of the rewrite) and said something to the effect of "you're a programmer, you make this work!"
I was able to get it working, and it did run significantly faster than the VBx version, however, it was rather... poorly written. I ended up bailing from that company before lawsuits and/or criminal proceedings were launched (failure to deliver on his outlandish promises anything to get the sale, as well as failure to properly dispose of PII). Don't know what happened to the project, but I believe the company was purchased, so the boss probably walked away with a very fat payout.
Note: "back in '96" is 1996, not 1896, you wippersnappers! Get off my lawn!!
 
Duga
Duga
12:26 PM
> Thanks, I'll push a hotfix and add a bunch of tests - somehow I expected the build to fail if it broke the inspection, ... But this could only have been properly tested with the other changes made in the mock parser amd test setup.
 
Hosch250
Hosch250
12:58 PM
@FreeMan hi.
@Freeflow sounds too good to be true. Hope there's not a bug involved in the runtime :)
 
FreeMan
FreeMan
Welcome back! Congrats on the new house.
I was just wondering a week or so ago if anyone had heard from you or Comintern.
Maybe he'll stop by soon, too!
 
Hosch250
Hosch250
Maybe!
 
Duga
Duga
 
BZngr
BZngr
1:21 PM
@Hosch250 Good to see you here again!
 
Duga
Duga
> This patch makes the ImplicitActiveSheetReference inspection tests to now correctly set the document module supertypes in the test code, and adds a test that confirms the fix for the referenced issue.
 
Mathieu Guindon
Mathieu Guindon
1:42 PM
hello!
 
Hosch250
Hosch250
@BZngr You too.
Hi, Mat.
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit 26a29f8d on unknown branch: AppVeyor build succeeded
 
FreeMan
FreeMan
BTW, @Hosch250, you're going to have to update your profile pic - I'm sure the puppy is much bigger now! ;)
 
Hosch250
Hosch250
She's 9 now.
But she's still cute.
 
Duga
Duga
1:58 PM
 
M.Doerner
M.Doerner
@MathieuGuindon You merge too fast.
@Hosch250 Welcome back!
 
Duga
Duga
 
Mathieu Guindon
Mathieu Guindon
@M.Doerner oops, sorry!
 
Duga
Duga
 
Hosch250
Hosch250
@M.Doerner Thanks.
 
Duga
Duga
2:24 PM
[rubberduck-vba/Rubberduck] build for commit b341d6ee on next: AppVeyor build succeeded
 
Duga
Duga
2:39 PM
 
Hosch250
Hosch250
2:56 PM
Have you guys considered using GitHub actions?
I know what you have works, but they are pretty cool.
 
Mathieu Guindon
Mathieu Guindon
it does
from what I understand it should be possible to build my contentupdater app on the cloud and somehow hook it up so that it automatically runs and updates the website records as soon as a PR is merged and built on [next] or [main]
 
Duga
Duga
> This PR updates the installer bitmap resources (including the ugly old v1.x Ducky.ico stray icon!), and adds an extra test that was in scope for #5736
 
Hosch250
Hosch250
I believe GitHub has a webhook you could connect to.
 
Mathieu Guindon
Mathieu Guindon
@Duga ooh missing the xml-doc edit
 
Hosch250
Hosch250
You'd build your endpoint, then point GitHub to it, and it would call it.
 
Mathieu Guindon
Mathieu Guindon
3:02 PM
tbh the first step would be to get my GH API creds into BlackBox and upload the damn source code to GitHub already
actually it's already on GH, just on a private repo
 
Duga
Duga
 
FreeMan
FreeMan
^ read csproj and wondered who the heck was posting their homework to the repo. Yeah, never mind, I got it...
 
Mathieu Guindon
Mathieu Guindon
lol
 
Hosch250
Hosch250
There might be a nuget package to help with that.
This looks really useful: medium.com/@aergonaut/…
 
Duga
Duga
3:27 PM
[rubberduck-vba/Rubberduck] build for commit 1a09a8da on unknown branch: AppVeyor build failed
BUILD FAILURE!
 
FreeMan
FreeMan
Mug slapping his head in 3... 2... 1... :D
 
Duga
Duga
> Build 5816 seems to work as it should. Nice work! :)
 
Mathieu Guindon
Mathieu Guindon
3:46 PM
@Duga yeah, seems
@FreeMan called it
 
FreeMan
FreeMan
@MathieuGuindon LOL
 
Mathieu Guindon
Mathieu Guindon
wtf though I did run them locally this time
oh fml
 
this
this
that's not cool. it should have coalesced the blue ! to the parent node
 
Duga
Duga
> Build 5816 seems to work as it should. Nice work! :)

BTW, I like the new splash screen. :)
 
Mathieu Guindon
Mathieu Guindon
@Duga repro'd
 
FreeMan
FreeMan
4:00 PM
@Duga Oh yeah, that does look pretty good!
 
this
this
@MathieuGuindon IDK if relevant but if there's stuff to be deserialized for mock parser, that may need to be updated (or maybe you did that already)
 
SmileyFtW
SmileyFtW
@MathieuGuindon - read the twinBasic article... and see that you are talking about it here. How would it replace VBA in Excel, for instance? Or does it?
Great article, BTW
 
Mathieu Guindon
Mathieu Guindon
thanks!
 
SmileyFtW
SmileyFtW
I find myself using the Factory Pattern A LOT... LOL
 
Mathieu Guindon
Mathieu Guindon
you'll find yourself using constructors a lot in tB! :)
tB wouldn't replace VBA, but makes it easy to build something in a language you already know (plus all the perks of it being a brand new language) and to expose it to a VBA client
and then there's talks going on about how a COM-visible .DLL written in tB can be engineered to not be a PITA to use and deploy
 
this
this
4:13 PM
At this stage, I can see uses for moving the standard library code that you authored into a tB DLL
 
SmileyFtW
SmileyFtW
@MathieuGuindon <Standing on chair>... still over my head...
2
 
this
this
Note, though, it's limited to 32-bit, so wouldn't work in 64-bit Excel unless you want to enable COM remoting.
 
Mathieu Guindon
Mathieu Guindon
@this VBMVVM? that's the idea
 
this
this
Yep
 
FreeMan
FreeMan
@SmileyFtW climbs ladder. Has same problem
 
this
this
4:15 PM
remove all the boilerplate clutter from your VBA project!
 
Mathieu Guindon
Mathieu Guindon
@SmileyFtW tools / references -> then reference your compiled twinBASIC DLL from your VBA project, and invoke tB objects from VBA code
@this my understanding is that this is just a preview limitation; will/should be lifted come release 1
 
SmileyFtW
SmileyFtW
@MathieuGuindon That part makes sense, but "can be engineered to not be a PITA"?
 
Mathieu Guindon
Mathieu Guindon
you know, Wayne and black magic
 
IvenBach
IvenBach
Soon you'll be doing the same kind of stuff for RD Mug. Gotta slay them dragons.
 
Mathieu Guindon
Mathieu Guindon
basically they're looking into ways to reference a tB library from VBA without it breaking all the time
 
this
this
4:18 PM
@MathieuGuindon yes did you see the roadmap?
 
Mathieu Guindon
Mathieu Guindon
yep
 
 
4 hours later…
this
this
8:11 PM
given the expression like (((lame) is stupid) and some more stupid text i don't care), how do I get regex to match only the (lame) and not (((lame) ? Note: the parentheses are literal and needs to be matched. I'm using something like \((.+)\) which will match the (((lame)
 
Vogel612
Vogel612
if you only want the innermost one, you can go for /\([^\(]+?\)/
 
Mathieu Guindon
Mathieu Guindon
8:31 PM
say you have Bar context in Foo.Bar("thing"); do we have a method that can give us the Foo context?
 
this
this
I think you are supposed ot use the GetAncestor extension method
 
Mathieu Guindon
Mathieu Guindon
hm
 
M.Doerner
M.Doerner
That is not an ancestor, actually.
Foo is in the lExpression of the memberAccessExpression ancestor.
Note however that you might be inside a WithMemberAccessExpression.
 
Mathieu Guindon
Mathieu Guindon
yeah that occurred to me... feels like IdentifierReference is missing a piece about what Declaration is qualifying it
@M.Doerner but that can be recursive, correct? i.e. the lExpression is also a memberAccessExpression?
 
M.Doerner
M.Doerner
It can, but you are interested in the last one anyway, right?
They are boxed right-to-left (seen from the outside in).
 
Mathieu Guindon
Mathieu Guindon
8:41 PM
I think so but my thought stream seems to turn to spaghetti... that does help
 
M.Doerner
M.Doerner
What exactly are you trying?
 
Mathieu Guindon
Mathieu Guindon
to determine the actual qualifying declaration given an IdentifierReference
so given a lExpressionContext, I just want the last child then
 
BZngr
BZngr
Same issue exists for finding the qualifying Declaration of a UserDefinedTypeMember IdentifierReference.
 
Mathieu Guindon
Mathieu Guindon
many places too
we're reinventing that wheel every time are we
 
BZngr
BZngr
Yeah...you can find my wheel inside EF refactoring. :)
 
this
this
8:52 PM
well, that's what you get for missing the "e" in the antlr, right?
 
Mathieu Guindon
Mathieu Guindon
woot, Me works
 
Mathieu Guindon
Mathieu Guindon
9:05 PM
ok, needs a test for With now
 
BZngr
BZngr
 
Mathieu Guindon
Mathieu Guindon
thinking the resolver could simply store the current qualifying object as it creates IdentifierReference instances... we shouldn't need to drill up & down parse trees to do this in inspections and refactorings
 
BZngr
BZngr
That would be great!
 
this
this
who knows. that may help speed up the resolution/inspection
 
BZngr
BZngr
Certainly would not slow it down. :)
 
Mathieu Guindon
Mathieu Guindon
9:27 PM
not pretty, but all green.
 
Duga
Duga
 
this
this
@MathieuGuindon should we expect a Rubberswan then?
@Duga wonders what an excel inspection has to do with installer resources ;-)
 
Duga
Duga
9:43 PM
 
Mathieu Guindon
Mathieu Guindon
@this nothing to see here ;-)
I said 2.5.2 was imminent way too long ago
ship ship ship ship ship ship
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit b589d880 on unknown branch: AppVeyor build failed
BUILD FAILURE!
 
Mathieu Guindon
Mathieu Guindon
fml
did not run all the inspections tests
wait huh
lol of course
bbl
 
M.Doerner
M.Doerner
10:22 PM
I think I can store the qualifying declaration in the resolver. However, one has to be a bit coutious as this also requires updating the cache invalidation.
Hm, we could actually store the qualifying reference. That would give more validation and no further cache invalidation. However, it will be a bit painful for with blocks.
 
this
this
@M.Doerner for With over a variable, that should be easy. But for those over an expression, do we create a declaration representing the temporary variable as the result of that expression?
 
IvenBach
IvenBach
10:41 PM
That change-I-didn't-expect-to-make-a-difference surely did.
 
Duga
Duga
11:22 PM
[rubberduck-vba/Rubberduck] build for commit cff0958b on unknown branch: AppVeyor was unable to build non-mergeable pull request
BUILD FAILURE!
 
IvenBach
IvenBach
@Duga Iven-Self-Note: git fetch followed by git merge upstream/next before pushing for PR.
It's not much, but I'm proud of that PR. First time setting up mocking without blindly copy-pasta-minor-edit to get a test in.
@GregorBrandt Welcome to the pond.
 
Gregor Brandt
Gregor Brandt
Thx
 
IvenBach
IvenBach
11:45 PM
@this I'm getting 3 failing tests locally. Confident it's because I'm using YYYY-MM-DD for my date format.
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit 17e72f56 on unknown branch: AppVeyor build succeeded
 

« first day (2509 days earlier)      last day (671 days later) »