VBA Rubberducking

Hosch250

8:00 PM

Each test needs to test an isolated instance.

Mat's Mug

right

Comintern

[TestClass]
public class RenameTests

Am I missing the static there somewhere?

Mat's Mug

the mock vbe (and its state) in the base class would be shared by all test methods

Hosch250

And what happens when one test deletes a component, and the next one tries to access it?

Mat's Mug

shit hits the fan

Hosch250

8:01 PM

There is an extremely good reason that tests are single isolated blocks.

Comintern

Hmmm...

Hosch250

We'd need to setup/teardown anyway.

Mat's Mug

yep

Comintern

Need some TestRunner classes then.

Hosch250

So, I'm willing to go that route, if we can get it set up so we can add the code in the test...

But really, I think it would be much better to just refine the builder.

Mat's Mug

8:05 PM

@Hosch250 the builder API is really not too far from where it needs to be for that, I think

Comintern

Something more like this:

//Arrange
var tester = new RenameTestRunner();
//Act
var actual = tester.Test(code, selection);
//Assert
Assert.AreEqual(actual, expected);

Mat's Mug

#SoManyOptions

Comintern

^

Hosch250

var builder = new MockVbeBuilder();
IVBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component, selection);
var project = vbe.Object.VBProjects[0];
var module = project.VBComponents[0].CodeModule;
var mockHost = new Mock<IHostApplication>();
mockHost.SetupAllProperties();
->
MockVBE.CreateModule(moduleName, code, selection)

I can definitely make it do that.

Heck, I could even make it add out parameters for the project and vbe itself.

Mat's Mug

Yeah. Needs to be flexible enough to allow adding classes, modules, forms,..

Hosch250

8:07 PM

If we go to C# 7, we could just pass wildcards to those instead of needing overloads.

Mat's Mug

And then the tests themselves need to assert the parse trees instead of the text output

I'm green-lighting C# 6 after .12 is released

Hosch250

That will be much harder.

Comintern

Woot!

Hosch250

@Mat'sMug If you wait until March 7, we can do C# 7 :)

We could even just target C# 7 in the test project.

Mat's Mug

not yet

Hosch250

8:09 PM

Not critical, but it would be fun.

OK, I'll just do a few overloads.

To support stuff like: github.com/rubberduck-vba/Rubberduck/blob/next/RubberduckTests/…

Mat's Mug

6 will be a major improvement. I see 7 as experimental for now

Comintern

^

Hosch250

And this: github.com/rubberduck-vba/Rubberduck/blob/next/RubberduckTests/…

Mat's Mug

@Hosch250 or ASTs

Hosch250

Meh, 7 will be worth it just for the out parameter features.

M.Doerner

8:11 PM

On a slightly different note, I think we should start to actually write unit tests and not scenario tests. There are quite a number of tests that test simple functionallity of a class by running code through the parser instead of building the necessary declarations by hand and only testing the class under test.

Hosch250

@M.Doerner Actually, those are blackbox tests.

Don't delete the blackbox tests, but go ahead and write new whitebox tests.

Comintern

The indenter definitely needs to swing that direction- it already has way too many bizarre configurations to account for.

Mat's Mug

Agreed

That'll speed things up quite a bit, too

Hosch250

Yeah, whitebox tests are much better as a general rule.

Comintern

And a code re-writer will have too many bizarre corner cases to cover with blackboxing.

Hosch250

8:13 PM

My Checkers library uses whitebox tests primarily, but has a set of blackbox tests too.

IvenBach

for https://github.com/rubberduck-vba/Rubberduck/wiki/Unit-Testing#example would it be better to separate `Dim sut As New NumKeyValidator` to be `Dim sut As NumKeyValidator
Set sut = new NumKeyValidator'

M.Doerner

I mean, there should always be some blackbox/scenario tests, but the majority should be whitebox/unit tests.

Mat's Mug

@IvenBach probably, yeah... lol that article probably predates the "self-assigned object variable" inspection

Comintern

Better would be:

With New NumKeyValidator
    'Act:
    'Assert:
    Dim i As Integer
    For i = 0 To 9
        Assert.IsTrue .IsValidKey(Asc(CStr(i)), Value), "Value '" & i & "' was not accepted."
    Next
End With

Mat's Mug

@M.Doerner yup. not sure how we can do that though. Could we inject a bunch of hand-crafted declarations into a mock parser state?

Comintern

8:22 PM

I could mock up a resolver really quick. I could probably have it ready in time for RD 12.0

Hosch250

OK, hold off a sec.

Let's get what we have cleaned up before we jump into this.

Duga

[rubberduck-vba/Rubberduck] IvenBach edited wiki page Unit Testing

Hosch250

I'll get to work on this sometime this/next week.

Or wait, when is 2.0.12 coming out?

Mat's Mug

I think we should do that after the release though

Hosch250

Any day now, I suppose.

Mat's Mug

8:26 PM

@Hosch250 rubberduckvba.wordpress.com

M.Doerner

@Mat'sMug One could actually do that. Although, since the inspection now tend to use a DeclarationFinder one should probably mock that on the RubberduckParserstate and fill it with the desired declarations.

IvenBach

Not if, but when, I find edits like that there's no qualms about me making update?

Hosch250

The 4th? I'll be available more/less after the 6th.

M.Doerner

Do we actually need the _undeclared collection on the DeclarationFinder for anything but collecting the undeclared variables while resolving? If not, I could stop rebuilding it after every refresh.

Mat's Mug

@M.Doerner I think not. Wait what is the undeclared variable inspection using if not that?

M.Doerner

8:31 PM

I think the IsUndeclared flag.

I am using that to rebuilt the collection.

Mat's Mug

Ok

So yeah it wouldn't be used beyond the collecting of declarations then

M.Doerner

Then I will just remove it. I anyway introduced a separate collections for the temporary jobs when I introduced the rebuilding of the '_undeclared` and _unresolved collections.

Duga

[rubberduck-vba/Rubberduck] IvenBach edited wiki page Unit Testing

Mat's Mug

@IvenBach feel free to update the screenshots too :-)

IvenBach

I don't know how to do screenshots on GitHub.

Should Test Explorer>Add>Test Method add the code wherever the cursor is active or should it add it below the currently active method?

M.Doerner

8:42 PM

Is there any reason in particular why all the dictionaries in the DeclarationFinder are concurrent? The only place where we are writing to them is in the constructor.

IvenBach

@Mat'sMug Can you even selectively run tests anymore?

Mat's Mug

I can't remember why "Selected Tests" was removed from the "Run" menu, but yeah it needs to come back

Quack Exchange

1

Q: Applying same formatting to multiple borders in Excel VBA?

Is there a better way to format the cells with borders than what you get when you record a macro? For example, I want to add borders to a cell range. The recorded code is: Range("A1:C19").Select Selection.Borders(xlDiagonalDown).LineStyle = xlNone Selection.Borders(xlDiagonalUp).LineStyle = xl...

1

Q: Fixing Medical Claim Files

With thanks to @Mat'sMug and @Comintern for their encouragement, here is a program I wrote to help my team fix medical claim files. The goal of this program is to make the process of emergency changes to medical claim files as human-proof and rapid as possible, and deliver all the corrected file...

vba

Hosch250

@Mat'sMug Me either. WT*

ThunderFrame

@Comintern I got PropertyBag working in VBA, and I could implement some of the VBRUN interfaces, but I couldn't get much further than that.

Mat's Mug

8:51 PM

I dont know if that's worth dedicated question, but the only trouble i had so far with RD is that in my office's desktop Excel sometimes "complains" that it is not the "default application" for Excel files or something like that. It's not a problem because everything still goes smoothly but is that something expected or should I worry about it? — A.S.H 2 mins ago

did that happen to anyone? I don't see how RD could affect it in any way

Hosch250

Nope, not RD.

ThunderFrame

@Mat'sMug Is there a conflict with another add-in?

M.Doerner

Strange, after introducing selective resolving of references, it consistently takes twice as long to reparse when I did not change anything compared to when I changed something.

Hosch250

No.

ThunderFrame

vbforums.com/…

> Basically, with just these, you could conclude that MSVBVM60.dll IS NEEDED no matter what. All custom controls and Class Modules need the IClassModuleEvt and the HiddenInterface interfaces, all projects need the IProjectControl and IVbeRuntimeHost. All data-binding needs the data interfaces and classes, and all appications need the App class...
>
> Basically, you can't remove MSVBVM60.dll. You CAN remove stdole...but only if you don't need the StdPicture, StdFont, and IUnknown, which is never the case...IUnknown is needed for every program...

All long help threads should have a sticky globally-editable post at the top saying 'DEAR PEOPLE FROM THE FUTURE: Here's what we've figured out so far ...'

3

Comintern

8:55 PM

@ThunderFrame Hmmm... Maybe Marshal.GetIUnknownForObject and then cast?

Oh wait, that's the wrong way, isn't it.

IvenBach

XKCD strikes again!

Comintern

@ThunderFrame The reason you can remove stdole is that it's forward declared in both msvbvm60 and vbe7.

Duga

[rubberduck-vba/Rubberduck] IvenBach opened issue #2763: Unit Test>Test Explorer>Add>Test Method

> Adds the method wherever the cursor is active. This may have it add it to an existing method creating `Public Sub SomeTest()
dim someVar as string
someVar = "someValue"

Public Sub AnotherTest()

End Sub

End Sub'

[rubberduck-vba/Rubberduck] IvenBach edited issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder added label difficulty-01-duckling to issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder added label critical to issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder added label bug to issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder added label feature-unit-testing to issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder commented on issue #2763: Unit Test>Test Explorer>Add>Test Method

> Really?!! Wow, how did we miss that!

[rubberduck-vba/Rubberduck] retailcoder milestoned issue #2763: Unit Test>Test Explorer>Add>Test Method

[rubberduck-vba/Rubberduck] retailcoder commented on issue #2622: Parameter 'Cancel' is never used

[rubberduck-vba/Rubberduck] retailcoder commented on issue #2597: Concurrency issues randomly make tests fail

> ref. #2759 - the tests are currently being ignored, so that the CI builds can be useful in any way.

FreeMan

9:16 PM

@Mat'sMug I've never seen that

FreeMan

9:27 PM

So is Planned for next release the .12 release or the next one after that? I'm confused because there's also a Planned column. Is Planned basically, "yeah, we'll get these, but haven't figured out when yet"?

ThunderFrame

> Whenever we create a class module the VBA compiler also creates a
default interface for that class. The default interface is given the same
name as the class and contains a list of all the public properties, methods,
and so on that we add to the class. When we dimension a variable using Dim
clsTheClass As CClassName, we’re saying that the variable uses the
interface CClassName. When we use code like Set clsTheClass = New
CClassName, we’re creating an object that is a new instance of the class

FreeMan

^^very readable description, thanks!

Mat's Mug

@FreeMan planned for next release would be .12, yes.

ThunderFrame

@Comintern Is that what HiddenInterface is doing?

Mat's Mug

planned is there because otherwise the backlog would be a bit unmanageable, so it serves as some kind of prioritization buffer

And of course, managing the projects and cards ...is a project in itself!

Comintern

9:37 PM

@ThunderFrame I think it's baked in at a lower level that than that. As long as the interfaces are referenced in the type library and registered you can use them. All of the API calls into the library are just that - API calls. Since everything inherits from IUnknown, IDispatch those types will always get pulled in. stdole doesn't really define much that isn't going to forward declared from a bajillion different places.

FreeMan

@Mat'sMug thx

ThunderFrame

I wonder if some of the VBE messages are throwing PropertyBags around...

Quack Exchange

0

Q: Prevent saving for an opened CSV file?

I'm interested in collecting rows from multiple CSV workbooks but a good deal of the data is not useful to me. I've got a roster of some several dozen people but the workbooks house all the data of some several hundred. The method I've used already is that the tool workbook will start by Defin...

vba excel

FreeMan

@QuackExchange @Mat'sMug can you just move it for him?

Mat's Mug

...where to?

FreeMan

9:50 PM

of course, it'll get closed there because it doesn't have any code? SO

but at least at SO, he'll get prompted to post broken code so it can be fixed.

Mat's Mug

rule #1: don't. migrate. crap.

FreeMan

I thought rule #1 was The Doctor lies...

Mat's Mug

if it gets closed on the target site, migration gets rejected and the post bounces right back over here.

FreeMan

ok... never mind then, he'll probably duplicate his post there, then. :)

TTGH, I'm not going to get anything useful done in 8 minutes...

ThunderFrame

Is it just me, or do the accountants that are self-taught VBA programmers sometimes write better Excel code than the Enterpise programmers?

IvenBach

9:57 PM

@ThunderFrame Every now and then the blind pig finds a truffle.

2

I think it comes down to amateurs, including myself, know just enough to make it work and can't overthink or overdesign things.

Comintern

FWIW, I taught myself VBA when I was working in a financial services job. Actually, the only language I have formal schooling in is VB.NET, which I never use...

Mat's Mug

my schooling predates .NET, so I did VB6 and some C++... although I don't recall any C++

Comintern

...and I'm the only one here at work that doesn't use Java style closures in C#. :shudder:

IvenBach

A few friends tried to make fun of me when I was learning Excel. Now I'm hired because of it and they ask me questions regarding it...

Comintern

Granted, I did write a bunch of TRS-DOS BASIC way back in the day...

ThunderFrame

10:01 PM

I had a .NET colleague boast about his VSTO add-in that ran in 5 minutes. A business colleague showed him the same outcome in VBA running in 0.025 seconds. VSTO was just as capable of those times, but because the dev didn't know about reading ranges as arrays (and wasn't familiar with the Excel Object Model), his code was orders of magnitude slower.

IvenBach

@ThunderFrame Well now you know!

Amazing how tweaking just a few small things does wonders

Mat's Mug

@ThunderFrame I bet it left the Excel process lingering in task manager, too

IvenBach

Unit Testing. Is there any convention about Assert.<Test>?

Mat's Mug

?

ThunderFrame

Working on RD now makes me sound like a know-it-all when I talk about the intricacies of line_continuators

Comintern

10:03 PM

LOL

IvenBach

Assert.AreEqual varResult, varExpectedAnswer as opposed to Assert.AreEqual SomeFunctionCall(inp1, inp2, ..., inpN), varExpectedAnswer

ThunderFrame

HR wants to speak to me about my proselytizing on instruction separators being evil

Comintern

"Well you, know, you could do this..." :gives @Comintern a wicked indenter issue:

IvenBach

@ThunderFrame excuse my ignorance but what's an instruction separator?

Mat's Mug

@IvenBach oh. I'd usually have some Dim expected As Whatever and a Dim actual As Whatever locals, and do Assert.AreEqual expected, actual instead of inlining the calls - that way makes it easier to debug

Comintern

10:05 PM

@IvenBach Sub Foo(): Debug.Print "Foo": End Sub <--it's the colon.

ThunderFrame

Do :Loop

Comintern

That's a Dont Do: Loop.

ThunderFrame

^That one will hang the App

IvenBach

@Comintern so ":" = Instruction separator and "_" is a line continuation

Sideways 8 loop?

Comintern

Yep.

ThunderFrame

10:06 PM

yep, unless they're inside square brackets, or double quotes.

Comintern

Or a comment.

A.S.H

Hi guys, Thanks @Comintern for the invitation :)

Comintern

Sometimes.

Hey, @A.S.H

ThunderFrame

hi @A.S.H

Now you know where VBA devs go to ~~die~~ solve the world's problems

puzzlepiece87

@ThunderFrame I'm just like @ThunderFrame's comment and Comintern's background, business guy who never had a programming class.

IvenBach

10:08 PM

Anytime someone posts in Stack Overflow or [cr] with the tag VBA that's when we get the RSS feed, correct?

Comintern

@puzzlepiece87 I gave up the business guy part for the programmer part and went back to school.

Where I learned VB.NET.

A.S.H

@ThunderFrame lol, I have to say I somehow "knew" that :D

ThunderFrame

I took a week's course on Logo for C64 when I was 12. Otherwise self - ~~experimented~~ taught

Comintern

Almost went back to business, found C#, and was like "Ahhhhh... much better."

M.Doerner

@ThunderFrame I am actually also not a programmer by trade but an actuary.

Comintern

10:11 PM

Interesting. I learned VBA when I was doing reinsurance billing.

Hosch250

I learned VBA when... Oh wait, I didn't.

Comintern

lol

Admit it. You could write VBA if you wanted to.

Hosch250

Yes, I could.

I got some VB.NET into the Roslyn repo.

Comintern

Hell, you could probably answer VBA questions on SO.

Hosch250

Umm, perhaps.

Comintern

10:14 PM

A lot of, "you need to exit the loop" or "you're missing an End If" or "you need a Nothing check".

Hosch250

With a search engine, for sure.

Comintern

^ or RTFM.

Hosch250

Meh, I post those as a comment and VTC the question.

Comintern

^ yep. Unless there's too much wrong to fit in a comment.

Those probably have genuine educational value.

Hosch250

What percent of the people who have too much wrong to fit in a comment actually read the answer?

And what percent of those actually understand the answer?

@Comintern You are in Nebraska? What's the outlook of programming jobs there? I want to get out of MN--I've seen too much of it.

The problem with being a smart genius is that you (at least, I) have tendencies to become a cynic extremely young.

Comintern

10:17 PM

Not too bad, really. You don't want to live in Nebraska though...

Hosch250

Why not?

Comintern

It's kind of backward. I'm originally from MN.

Hosch250

Too conservative for you, or something?

I love the country, and I'm leaving MN because of the liberals.

Comintern

That's only part of it. There really isn't a lot to do here.

Hosch250

Don't need a lot. Give me a computer and a bike and my two legs, and I can amuse myself all day.

Comintern

10:19 PM

The rural areas are worse - it's too developed agriculturally. Everything is in crop production.

Hosch250

Like southern MN and Iowa.

Yeah, I was thinking Utah or Montana, maybe.

Comintern

Yeah. I think of country as more like central - northern MN.

Hosch250

Yeah, I've been up there. Real pretty.

IvenBach

Move to California, where I live, it's not like there's anything liberal around here...

Duga

[rubberduck-vba/Rubberduck] bravesparrow commented on issue #2641: Test Explorer Hangs in an infinite loop

> Code explorer hangs before any of the tests output a result (see that duration is 0ms for each test).

![image](https://cloud.githubusercontent.com/assets/18690257/23484056/6c424ea6-ff29-11e6-91ed-59f164611c9f.png)

What other info could I provide to help debug this?

Hosch250

10:21 PM

LOL.

I want a change of scenery from swamps and forests, though.

I was thinking Oklahoma or Texas too.

My family has a friend in Oklahoma, maybe I could get something with his connections.

Comintern

The Michigan Upper Peninsula would probably meet the conservative\pretty standard fairly well.

Oklahoma is like living in a Steinbeck novel without the exciting moving west part.

Mat's Mug

TTGH

Hosch250

Oh, I can put up with liberals, really, and conservative extremists are just as annoying.

Most of it is I want a change of scenery.

Mat's Mug

Oh hai @A.S.H!! Welcome to our little corner of SE!

A.S.H

Hi @Mat'sMug, thanks a lot!

Comintern

10:24 PM

@Hosch250 Upstate NY - Rochester, Syracuse, that area.

I'll consider it.

bring a gun

lol

Hi, @A.S.H. Looking to contribute to RD, or just report bugs?

@Mat'sMug Why?

Comintern

I lived about 60 miles south of Syracuse for 3 years growing up. Bring a gun.

Hosch250

10:25 PM

Bears, or outlaws?

Comintern

Otherwise it's like showing up without pants.

Hosch250

Well, I'd rather not have to use a gun, but I'm certainly not afraid of them.

I'll remember your advice.

Comintern

"I had this dream last night when I showed up at work and all I was wearing were my clothes!" "Ooooooo".

Hosch250

So, are they gun-extremists, or escaped crooks?

Comintern

Neither really. More of a culture thing. That's probably changed since I lived there though.

Hosch250

10:28 PM

Oh, I see.

Well, I'd rather have that culture than one like inner-city Chicago/Detroit/Minneapolis.

puzzlepiece87

@Comintern I'm sticking with the business part for as long as I can because I have a passion for problem-solving and business strategy - automation will eat a lot of business jobs before I retire, so programming is something I do to make sure I have a viable career in 15 years.

IvenBach

^

A.S.H

@Hosch250 Hi. I wasn't considering it, but, well, if I can be helpful that will be a pleasure. Still too early though but will be involved within time :)

Hosch250

NP.

If you like copy/pasta and drudge work, you can help clean up the existing tests once I get a better framework setup.

IvenBach

A lot of pineapples can't figure out that lots of stuff that can be automated should be.

puzzlepiece87

10:30 PM

@IvenBach Of course, by the time they figure that out they probably won't need amateur programmers like you and I either: arstechnica.com/information-technology/2016/08/…

The Congressional Budget Office (US) recently exceeded industry estimates for automation. The old best-cited estimate was 40% of jobs by 2050, now it's 47% by 2040.

It will be a very tricky transition for the US. EU countries are pretty well-prepared. They basically already have UBI.

Hosch250

UBI?

I don't think the US will have a problem, except with employment issues.

ThunderFrame

@Hosch250 if you choose Texas. Bring lots of guns.

Hosch250

LOL. You guys are reminding me of an article I read once: the guy lived in the country, everybody knew each other, and everybody knew everybody else had a gun; no crime. Moved to inner-city, couldn't find someone with a gun for love or money; crime every day.

Trying to write a cover letter to a recruiter for a defense company I almost landed an internship with last summer.

Who knows, I could end up working in the UK or Australia (well, probably not, since all the jobs seem to require citizenship).

A.S.H

@Comintern is the Com part of your nickname related to "com"?

Hosch250

@A.S.H I think it is related to COM Interop stuff, not .com.

Comintern

10:42 PM

@Hosch250 That's more the "everybody knew each other" than the "everybody else had a gun".

Hosch250

@Comintern Yup.

A.S.H

yeah i mean DCOM of MS

Comintern

@A.S.H No, that's just a coincidence. Comintern as in the international communist conspiracy. It was my MUD handle back in the 90s.

It stuck because it's usually available. :-)

A.S.H

that's should be the feeling of many members I think... because you're COM expert and the first thing I thought about when I read your posts was

must be some member of the COM team or so

Comintern

It would actually be really funny if I interned somewhere in a COM field.

puzzlepiece87

10:48 PM

@Hosch250 UBI = Universal Basic Income

M.Doerner

I hate performance testing. I have been wondering why the initial parse takes 10s longer after introducing selective resolving of references. In the end, its just that my computer is slow today, Switching to another branch without the changes gave the same runtime.

Comintern

Are you running a profiler, or just using the timestamps?

ThunderFrame

@Comintern nominative predisposition?

M.Doerner

It was just the timestamps.

puzzlepiece87

@Hosch250 And yes, I think employment issues will be THE problem. I'm of the belief that of those 47% of jobs, some will reappear in new ways but many will not. The US has poor safety nets, so we'll see the poverty rate grow until we accept that not everyone needs to be employed.

M.Doerner

10:50 PM

I should start using a profiler.

Comintern

@M.Doerner Did you ever get a community R# subscription? The profiler in there is decent.

IvenBach

Based on Test Driven Design (TDD) do you write how the result will be tested or do you build and write tests as you buld?

M.Doerner

Nope

Comintern

Percentage of time in procedure and call counts are much more relevant.

ThunderFrame

@puzzlepiece87 And Microsoft Rubberduck VBA for Office™ will be used for the other 53% of tasks

Hosch250

10:51 PM

@puzzlepiece87 Meh, I think European countries will have a lot of failing businesses.

They won't be able to afford to go full automatic and pay their unneeded employees.

puzzlepiece87

@ThunderFrame Ha, I think once we start automating everything people will start using better tools :P

ThunderFrame

@puzzlepiece87 Maybe it will be Uber Rubberduck VBA for Tesla Office™

puzzlepiece87

@ThunderFrame Uber's on deathwatch, otherwise your statement could be correct lol.

ThunderFrame

you're right.... Hosch Inc. VBA for Office

Duga

[rubberduck-vba/Rubberduck] bravesparrow commented on issue #2641: Test Explorer Hangs in an infinite loop

> I cleaned and rebuilt the Rubberduck solution from the latest repo just now.

I removed the offending Test module from my Access Project.

I re-started Rubberduck via the Debug AnyCPU in Visual Studio.

I opened the Access file and opened the VBE, and clicked the "Pending Button" but it got stuck at "Resolving Declarations" after beginning the Parse. It seems I am in deeper trouble now as the new build will not parse (and there are a muber of COM exceptions being thrown).

Here is the

puzzlepiece87

10:55 PM

Google opened a very convincing lawsuit that Uber stole all their self-driving tech. Uber loses $2B a year currently, their venture capital was counting on them making self-driving taxis. Once they lose this lawsuit they'll be toast.

M.Doerner

Coming back to my question from before, if nobody can tell me a good reason why the dictionaries on the DeclarationFinder should be concurrent, I will convert them to ordinary dictionaries for performance's sake.

We only write to them in the constructor.

puzzlepiece87

A Google engineer spoke with Uber leadership, downloaded all of Google's tech to disk, resigned, founded a self-driving company, and Uber bought them 6 months later for $700M

ThunderFrame

^ Standard Google Model

Comintern

OK, WTF. Why am I getting the throws for if (!_listening)` in SubClassProc?

puzzlepiece87

@ThunderFrame Ha, except Google does it the legal way, by buying the entire company rather than by paying an employee to steal trade secrets

ThunderFrame

10:58 PM

@puzzlepiece87 I meant tech/people leaving Google

puzzlepiece87

@ThunderFrame Oh agreed, though using the ideas is fine and I don't have a problem with it. It's when you walk out with everything on disk and a pre-arrangement to get bought out that the problem starts :P

Every big company leaves some opportunities open due to their own prioritization. If people see a niche to capitalize, go for it.

ThunderFrame

@puzzlepiece87 disks???? Sync with BitBucket/GitHub is so much easier.

puzzlepiece87

@ThunderFrame Right? :P

IvenBach

Why didn't he use a floppy disk? Not like that tech is going to be obsolete.

puzzlepiece87

@ThunderFrame The one I'm most curious about is the NSA contractor who took like 18 highly-classified TB home.

@IvenBach Maybe Google wouldn't spring for the drives :P

ThunderFrame

11:04 PM

I'm guessing Google would fail to see the irony of uploading the source of Google Drive to Google-Drive

puzzlepiece87

Ha

If only anonymous free accounts had more than 15 GB

@ThunderFrame, Thanks for helping me with my earlier coding issues, we got it resolved successfully.

ThunderFrame

np - How's the performance?

puzzlepiece87

I think we got it done in 20 min

(Which to me is not bad at all for a 1 GB file)

1.5M rows and 162 columns

ThunderFrame

hmm, that seems pretty slow

This answer was reading and processing a file (including searching for duplicates) at about 6.5MB/s. For a 1TB file, that would equate to about 150 seconds.

If you're just splitting up a file, you should be able to get performance (Depending on your hardware) at 45MB/s or more. 45MB/s would equate to a 22 second run time.

Are you sure your system can load a 40 GigaByte file in 1 second? A really fast SSD can theoretically read up to 2500MB/s, but reading 40GB in 1 second would be 16 times faster than that. Do you perhaps mean a 40MB file? — ThunderFrame Sep 14 '16 at 8:11

puzzlepiece87

11:20 PM

Thanks, I will give that a look and try to improve the performance!

ThunderFrame

@Hosch250 If you're serious about Australia - skilled workers in certain professions can get Visas border.gov.au/Trav/Work/Work/…

Duga

[rubberduck-vba/Rubberduck] MDoerner created pull request #2764: Selective reference resolving part1 to merge MDoerner/Rubberduck/SelectiveReferenceResolvingPart1 into rubberduck-vba/Rubberduck/next

> This PR introduces selective resolving of references (by module). More precisely, only references of modules referencing a module that gets parsed are resolved. To achieve this, I had to pull the store the unresolved declarations outside of the DeclarationFinder; they now live on the ModuleStates in a new collection separate from the real declarations. This has the added benefit that the DeclaratioFinder seen by the inspections now really see the undeclared declarations in the...

[rubberduck-vba/Rubberduck] build for commit 2971896d on unknown branch: AppVeyor build succeeded

M.Doerner

@Duga yay!

Duga

[rubberduck-vba/Rubberduck] rubberduck203 opened issue #2765: FAQ

> Create an FAQ from the links referenced in #2762.
These questions arise frequently and the info is all available, but doesn't seem to be quite visible enough. Then make it "big & visible".

- [ ] Create FAQ in the Wiki
- [ ] Add section in the ReadMe that links to it.
- [ ] Add link to it in the website?
- [ ] Add it to the about dialog?

[rubberduck-vba/Rubberduck] rubberduck203 added label difficulty-01-duckling to issue #2765: FAQ

[rubberduck-vba/Rubberduck] rubberduck203 added label repository-wiki to issue #2765: FAQ

[rubberduck-vba/Rubberduck] rubberduck203 added label up-for-grabs to issue #2765: FAQ

Hosch250

@puzzlepiece87 About 90% of their jobs require security clearance.

Their stuff is so secret their workers can't even bring their phones into certain rooms when they work.

Given what I know about phones, why am I not surprised?

Comintern

11:36 PM

Does this pattern leak objects? new Task(() => DispatchSelectedProjectNodeDeclaration(_vbe.SelectedVBComponent)).Start();

Hosch250

Well, in a way.

The objects will exist at least until they are garbage collected when the task is garbage collected.

Comintern

Task is IDisposable, but keeping a reference around to dispose it defeats the purpose.

Hosch250

You usually don't dispose tasks.

Comintern

These methods will get a ton of throughput, but they block on the main VBE thread.

Hosch250

blogs.msdn.microsoft.com/pfxteam/2012/03/25/…

If you haven't read that you, you should.

Comintern

11:39 PM

Reading it now - I should be fine, I just need to keep the Window message pump from blocking. The Declaration lookups are just too expensive.

Wow, that is indescribably better.

Mat's Mug

@Comintern what is?

Comintern

The status bar updates. Switching to native event handling completely killed the responsiveness.

It was blocking on the VBE's UI thread.

ThunderFrame

@Comintern IKR - I could hardly tab between codepanes

Comintern

In my next PR.

Mat's Mug

"Deleted code is debugged code." - Jeff Sickel

Tweeted by CodeWisdom on March 1, 2017 at 7:39 PM

Comintern

11:50 PM

Comintern's corollary - "Commented out code is code waiting to be debugged."

A.S.H

Actual code is code with undiscovered bugs :D

it's the same as CodeWisdom's tweet actually, I didn't get it fast enough!

Comintern

:deletes _DockableWindowHost.cs: <--debugged.

Mat's Mug

lol

Comintern

Testing an idea now.

Transcript for