VBA Rubberducking

8:00 PM

Didn't I change that after removing the proxies?

HRNPGHH!! Don't tell me the member dropdown thingkey is dependent on where the cursor is... :grunt: I was looking in the right file, just the wrong class in the file.

Mat's Mug

it's... yeah

feel free to pull that class into its own file (should have been done a long time ago already)

IvenBach

:HangsHead: There's a middle dropdown....

Duck Check: Dropdowns in the header of files are Project | Class | Member?

Mat's Mug

click on the class identifier, Alt+ENTER > R# already knows what you want to do

something like that yeah

IvenBach

Don't have R# at work.

Mat's Mug

8:02 PM

pity

IvenBach

indeed...

WaynePhillipsEA

Guys, for late bound unit testing, have you considered using a wrapper Assert class (and Fakes), using a predeclared ID. The Assert class might look something like this:

Private LateBoundAssert As Object

Private Sub Class_Initialize()
Set LateBoundAssert = CreateObject("Rubberduck.AssertClass")
End Sub

Public Sub AreEqual(Expected As Variant, Actual As Variant, Message As String)
LateBoundAssert.AreEqual Expected, Actual, Message
End Sub

With the predeclared ID, you could then make use of intellisense, so Assert.AreEqual would work just like the early bound version

IvenBach

@Mat'sMug ^

Mat's Mug

hmm, our late-bound setup code looks pretty much exactly like that, no?

WaynePhillipsEA

You currently don't get any intellisense

Mat's Mug

8:05 PM

I guess the predeclared ID would be some attribute decorating the AssertClass interface?

WaynePhillipsEA

No, it's the VB_PredeclaredID attribute

on a class named Assert in the VBA project.

Mat's Mug

Oh (facepalm)

Vogel612

so you want to wrap the access to Rubberduck's LateBound Assert in a VBA class module to enable IntelliSense?

sounds simple and reasonable

Mat's Mug

yeah

WaynePhillipsEA

Yeah, I think it would offer the best of both worlds (late and early bound)

I'm just thinking out loud really

Mat's Mug

8:07 PM

it wasn't done because we wanted to interfere as little as possible with the users' code... but I don't think it's going overboard. I like it :)

#FunFact the original VBA implementation did use a predeclared ID Assert class

WaynePhillipsEA

It would only add two class modules

IvenBach

The benefits of Rubberduckin out loud.

Mat's Mug

and we could even have an option for it

> [x] Create Assert VBA wrapper class
> [ ] Create Fakes VBA wrapper class

WaynePhillipsEA

sounds plausible.

Mat's Mug

yeah, that way existing (user) code would still work

that said we're really starting to need per-project settings

WaynePhillipsEA

8:11 PM

What are you using to invoke the methods? Application.Run?

Mat's Mug

depends on the host application, but yeah

Outlook is... more complicated :)

WaynePhillipsEA

Yeah, vbWatchdog doesn't use Application.Run, but I don't think I can expose how I got round that ;)

Have you thought about using class modules for the unit tests instead? I guess you probably have

Mat's Mug

absolutely

except...

couldn't get .net to create instances

that'd be like the holy grail

WaynePhillipsEA

Oh, yes you should be able to do that

Mat's Mug

if we can do that then we can have an actual mocking framework

wait, how?!!?!

WaynePhillipsEA

8:15 PM

grab the ITypeInfo from any IDispatch object implemented by your VBA project, grab the parent ITypeLib from that, and then enumerate all the ITypeInfos in the project. Once you find one, you can call ITypeInfo::CreateInstance

Mat's Mug

you gotta be kidding

WaynePhillipsEA

No, that should work

Mat's Mug

@ThunderFrame needs to see this

this

Confirming - that doesn't require the obsolete tibinfo library, right?

WaynePhillipsEA

should... can't guarantee it without actually implementing it. But unless VBA screwed up the whole ITypeInfo implementation, it should work

@this no

The IDispatch interface itself exposes members to grab the raw ITypeInfo

From there you get the parent (giving you an ITypeLib). And from there you enumerate the coclasses

this

8:20 PM

Assuming that works, then we would be able to do what we need from a class and implement all tests as a class which also avoid the host-specific implementation of Application.Run

Mat's Mug

@this it doesn't fix that ;-)

WaynePhillipsEA

The only issue is getting a hold of the initial object from the VBA project.

Mat's Mug

I like how you make it sound trivial

well, it is

WaynePhillipsEA

You'd have to have a stub routine that passes an object to your managed side so that you can then grab the initial ITypeInfo

Mat's Mug

you just need to get the VBA code to pass the instance, right?

yeah

WaynePhillipsEA

8:23 PM

And so you'd still need Application.Run or equivalent to get it started. But it can be any generic VBA created object that you pass

Mat's Mug

a bit like we grab the VBE instance with parser.Initialize Application.VBE in the (experimental) reflection API

WaynePhillipsEA

There is a cheat for getting the ITypeInfo actually

Mat's Mug

or equivalent - e.g. get ahold of the VBA runtime's CallByName method, and invoke it

WaynePhillipsEA

No that needs an object too

Mat's Mug

something tells me there's black magic going on :)

29 mins ago, by Vogel612

relevant: Case 211: "No Magic"

Quack Exchange

8:27 PM

0

Q: Handling lazy-loading webpages using vba

After a long try I've been able to create a script in vba which can successfully handle webpages with lazy-load. It can reach the bottom of a slow loading webpage if the hardcoded number of the loop is set accurately. I tried with few such pages and found it working flawlessly. The one I'm pastin...

WaynePhillipsEA

Yeah, and I'm not sure if you'll go with it.

Mat's Mug

if it involves injecting precompiled assembler into user code, ...yeah not sure we want to go there ;-)

WaynePhillipsEA

not yet anyway ;)

no, it's not that bad

If you grab the VBE.ActiveProject.References COM object, and grab the 5th pointer from the object pointer, that's the ITypeInfo

but I didn't tell you that

Mat's Mug

huh

lol

just curious about how you'd possibly find out these kinds of things

WaynePhillipsEA

don't ask :)

Mat's Mug

8:30 PM

I didn't!

IvenBach

@Mat'sMug I never thought I'd hear this come from Mug.

Mat's Mug

@IvenBach lol, I know nothing

WaynePhillipsEA

That's an easy way of getting the ITypeInfo without having to get any live object instance

IvenBach

Nothing is relative...

WaynePhillipsEA

And that has worked since the first version of VBA6 (and VB6 as well I think)

Vogel612

8:33 PM

@WaynePhillipsEA you do know that this is "public record"?

WaynePhillipsEA

Actually sorry it's obviously not the ITypeInfo( which is instance specific), it's the project ITypeLib

Mat's Mug

yeah was going to say :)

WaynePhillipsEA

Well I could be anyone pretending to be anyone

;)

Mat's Mug

I could nuke it from orbit if it's too sensitive

IvenBach

@Mat'sMug Breadcrumb got me there. Not sure what it should correctly display though.

WaynePhillipsEA

8:35 PM

No, it's fine. It's self obtained info, so I'm with it.

If it possibly helps you out, I don't see why I can't share it.

Mat's Mug

:D

thanks a bunch. FWIW vbWatchDog is quite possibly the only VBIDE add-in we couldn't eat lunch if we tried

WaynePhillipsEA

Some stuff I wouldn't be willing to divulge, but tidbits like above are fine

well I'm honored to hear that :) And FWIW Rubberduck is probably the only addin I would openly recommend to all VBA developers

9

apart from my own of course haha

Mat's Mug

that's... wow.

WaynePhillipsEA

You've made great improvements since when I last looked at it

IvenBach

Mug will now print that out and hang it on his wall, next to wedding and children's photos.

Mat's Mug

8:40 PM

so.. wait a minute.. if we can get the ITypeLib for the active VBA project, then we don't need to parse anything to grab module & member attributes, do we?

no mug, no we don't

WaynePhillipsEA

what type of attributes do you mean?

Mat's Mug

e.g. VB_Description and VB_PredeclaredId/VB_Exposed

which we're already picking up in the type libs we're COM-reflecting

WaynePhillipsEA

Yeah, they should be accessible

Mat's Mug

i.e. that code is pretty much already written

this simplifies so many things

without needing to implement a code pane from scratch

WaynePhillipsEA

ouch you're not planning that are you?

Mat's Mug

8:43 PM

well....

WaynePhillipsEA

you've got your work cut out!!

Mat's Mug

the idea is to have custom syntax highlighting, code folding, tearable tabs, and put the "intelli" back into "intellisense"

WaynePhillipsEA

wow. that's going to be quite involved.

IvenBach

@Mat'sMug For those that don't have a description and error out I want to log them. I'm stumbling a little bit. Once you're done.

Mat's Mug

@WaynePhillipsEA yeah. hopefully we'll be done before Microsoft figures out a way to put the VBIDE out of service

:)

WaynePhillipsEA

8:46 PM

@Mat'sMug you're ambitious, I'll give you that

Mat's Mug

"Bring the VBE into the 21st Century"

^ mission statement :)

WaynePhillipsEA

Well, yes, a new code pane would certainly help with that

IvenBach

Some would call us duckbrained.

Mat's Mug

the only problem is that it would have to be design-time only

'cause no way I'm going to get my hands into the debugger

WaynePhillipsEA

But.... perhaps vbWatchdog could expose some stuff to help you out

Mat's Mug

8:49 PM

you'd give us some compiled blackbox API to play with?

WaynePhillipsEA

just thinking out loud. obviously, we'd have to be careful with regards to the non OSS nature of vbWatchdog

But if I were to expose a few things, and Rubberduck could optionally hook onto them, perhaps that could work.

@Mat'sMug, yeah, something like that. Much like the VBE itself

Mat's Mug

except with a better API :D

WaynePhillipsEA

naturally

Mat's Mug

one problem is that having a .dll like that in RD's repository would make it very easy for anyone to grab and play with

WaynePhillipsEA

true, but we might be able to come up with some sort of solution.

Vogel612

8:54 PM

that does sound wonderful, but maybe we should go one step at a time :)

it's not like we have ~400 feature requests on our hands that also want their share of attention :)

Mat's Mug

lol

WaynePhillipsEA

nobody will notice if you give them a shiny new code pane and debugger

nobody will care :)

Mat's Mug

Sep 21 at 16:41, by Mat's Mug

Mar 3 '15 at 19:11, by skiwi

So when will you drop the IDE in which you are building the plugin, and writ eyour own IDE?

WaynePhillipsEA

Yes, that might be an idea

Mat's Mug

Aug 21 '16 at 20:44, by ThunderFrame

Aug 14 at 11:58, by RubberDuck

Hey guys. You know how we're always joking about writing our own IDE? I bet (if we could ever find time to write code for fun again) we could fork VS Code. http://opensource.stackexchange.com/a/4292/775

so many instances of "own IDE" in the transcript of this chat..

my favorite:

Apr 17 '15 at 15:41, by RubberDuck

Yeah. Given that, you can't @RichardFoxton. Not unless you want to build your own IDE from scratch. Even the Mug and I aren't that crazy. — RubberDuck 9 secs ago

WaynePhillipsEA

8:58 PM

I'd be up for that. I wrote my own frontend C++ compiler, so I've got experience.

Mat's Mug

ouch

WaynePhillipsEA

using the superb LLVM optimizing backend compiler.

I guess we're now a little off track. sorry about that.

Mat's Mug

eh, happens all the time, no worries :)

WaynePhillipsEA

how many core devs on this team?

IvenBach

4-5ish?

WaynePhillipsEA

9:00 PM

cool

Mat's Mug

a handful... there's @M.Doerner, @Hosch250, myself (haven't committed anything meaningful in a long while though), @this, ...lots of recent newcomers, and even more fly-bys

IvenBach

All at varying levels.

WaynePhillipsEA

I don't think Rubberduck likes parsing my vbMAPI class modules

Mat's Mug

hmm, parser bugs are down to near-zero by now... what's it complaining about?

WaynePhillipsEA

It's taking a long time for about 20 classes

Mat's Mug

9:03 PM

just parsing?

I bet SLL mode is failing

(there would be a warning in the logs for it)

WaynePhillipsEA

It's sitting there eating CPU%

Mat's Mug

that's embarrassing

no parser error reported?

WaynePhillipsEA

nope

M.Doerner

Are there long chains of boolean expressions?

We had one bug report where this was causing an incredibly long parse time.

Mat's Mug

I wonder how/whether Antlr4.7 is going to improve that

Duga

9:06 PM

[rubberduck-vba/Rubberduck] MDoerner synchronized pull request #3615: COM Release for menu items and command bar

M.Doerner

Actually, it might, or rather the necessary grammar changes.

I think it was constantly backtracking because of the large amount of Not operators.

Duga

[rubberduck-vba/Rubberduck] build for commit 25ba0642 on unknown branch: AppVeyor build failed

BUILD FAILURE!

Mat's Mug

@Hosch250 was making progress on the Antlr upgrade, I wonder how close to a PR he's at

@Duga why are you trying to copy to the debug folder ffs

WaynePhillipsEA

I'll be amazed if this eventually completes. I would guess an infinite loop, since CPU is using one core, and RAM use is static.

M.Doerner

My yml looks exactly the same as the one on next.

The word Debug does not even appear in it.

this

9:11 PM

@WaynePhillipsEA doesn't MAPI generally have cyclic properties?

Mat's Mug

@M.Doerner I know - fwiw that comment was directed at appveyor, not you ;-)

this

maybe that's what sending the parser down a rabbit hole

M.Doerner

I tested with the code. At about half the number of conditions, it completed in 2 minutes.

Mat's Mug

@WaynePhillipsEA and I thought @ThunderFrame's code was evil!

WaynePhillipsEA

@Mat'sMug do you want me to leave it running

Nelson Vides

9:13 PM

@Mat'sMug did I hear Antlr4.7? :D

this

@Mat'sMug just to clarify - that's MAPI OM, not his classes that exposes it. The MAPI itself is evil, IMHO. :)

Mat's Mug

oh

M.Doerner

Btw, each module is parsed on only one thread; the paralellism is in the modules.

WaynePhillipsEA

I honestly don't think there's much evil code.

Mat's Mug

so it's the COM Collector that's crawling then

WaynePhillipsEA

9:13 PM

Not by my standards

Mat's Mug

'Project Name = MyProject
'Module Name = MyModule
Option Explicit

Private MyVar As MyProject
Private MyVar1 As MyModule

Private Type MyProject
  MyModule As String
  MySub As String
End Type

Private Type MyModule
  MyVar As String
  String As MyProject
End Type

Private Type MySub
  MyVar As MyModule
  MyVar1 As MyProject
End Type

Private Type MyVar
  MyProject As MyModule
End Type

Sub MySub()

  Dim MyProject As MyProject
  Dim MyModule As MyProject.MyModule.MyProject
  Dim MySub As MySub

^ resolves fine

WaynePhillipsEA

Yeah, I've got a project with similar stuff to that. Loads of edge cases for my decompiler

this

@Mat'sMug to be fair, while that's evil, it's not really cyclic. I seem to remember that with MAPI, they have list of properties that can be themselves properties to another properties that is already self-referenced.... At least that's how I remembered it.

WaynePhillipsEA

but vbMAPI is nothing like that.

Mat's Mug

it's a type library?

WaynePhillipsEA

9:17 PM

No, it's similar implementation to vbWatchdog. But that resolves fine now in the latest release.

this

@WaynePhillipsEA what about Max's question? If the library has to make use of expressions that requires lot of left-recursions that can wreck the parser.

or at least make it go really slow

WaynePhillipsEA

Yeah, maybe. I'll leave it running to see if it completes

Mat's Mug

either it does, or some call stack ends up blowing

WaynePhillipsEA

If it doesn't complete, I'll try to reduce it down. But that might be a few days before I get to it.

this

@Mat'sMug so the parser will do recursion, not loops?

WaynePhillipsEA

9:22 PM

Just did a quick test with my personal 'evil' project. only 3 parser errors, which ain't bad really.

Mat's Mug

@this IDK. pretty sure at least some of the parser logic is recursive though

Nelson Vides

@this as far as I got into antlr, it is recursive almost by definition, isn't it?

Mat's Mug

@WaynePhillipsEA different errors? or 3 instances of the same grammar bug?

WaynePhillipsEA

3 different errors

Mat's Mug

interesting

I can smell the colons from here :)

(right?)

this

9:24 PM

@NelsonVides I'm generally the guy who try to implement recursions as a loop... generally speaking. Sometime you really can't avoid recursions but whenever you are able, it's better to use a loop so that you never have to worry about stack overflow, only the hard drive capacity of your PC. :)

Mat's Mug

and line continuations?

this

But TBH, I don't think I have the brainpower to improve what ANTLR guys did soo.. there's that.

WaynePhillipsEA

One looks like a colon bug, yes

Mat's Mug

@this most likely the problem is with our grammar, not with Antlr itself

Nelson Vides

@this I'm aware of the issues with recursion and stack overflows. But that's how grammars work for the most part, with heavily nested recursion.

WaynePhillipsEA

9:26 PM

Dim AObject As Object
AObject.Line -(3, 4)

Line is a special method

and the above is valid, at least at compilation level

Nelson Vides

Rule A have some branching, decides to take Rule B, which has some more branching and it might decide to take Rule A again (and it might then take a different path C the second time it enters A, or the third, or the fourth...

this

@WaynePhillipsEA @ThunderFrame would love that.

Mat's Mug

@WaynePhillipsEA hmm I clearly recall seeing a parser rule specifically for that weird-syntax special instruction

Nelson Vides

The idea of recursion is later very well improved by compilers, sometimes. Haskell and family for example, has a very awesome implementation of tail recursion that virtually solves the stack overflow problem, making the stack frame reuse itself every time, therefore, you have a binary loop but your code is organised as recursion, which is very elegant

Mat's Mug

i.e. it's supposed to be handled

this

9:30 PM

@NelsonVides that's cool. I didn't know that about Haskell.

WaynePhillipsEA

If True Then:
End Sub

this

@WaynePhillipsEA inside a procedure?

Mat's Mug

Huh

WaynePhillipsEA

Sorry, yes, I didn't copy in the Sub

Sub Blah()

AObject.Line (1, 2)-(3, 4), , BF

Nelson Vides

@WaynePhillipsEA That's evil code. I'll slap a colleague who writes that in production. But don't take my words wrong, evil is motivating.

Mat's Mug

9:34 PM

hmm, it is legal

this

has to be- since that's a single line If statement, followed by an empty line then End Sub. Earlier @M.Doerner pointed out that something like If 1 = 1 Then: Debug.Print "true": Else Debug.Print "false" would be actually a "single-line If statement", in spite of the colons supposedly separating the instruction.

WaynePhillipsEA

Both of these:

AObject.PSet (1, 2)
AObject.PSet (1, 2), 555 ' FLAG 2 means Color has been specified

And this: AObject.Scale

Mat's Mug

the good news is that they're all quite obscure edge cases :)

WaynePhillipsEA

AObject.Print AVariant; Spc(5); Tab; Spc(6); AVariant, Spc(0); Tab(55)

Oh yes, absolutely.

M.Doerner

@Mat'sMug The post build action is in the project file of Rubberduck.Inspections, and it has been there forever.

Mat's Mug

9:37 PM

@M.Doerner I know.. I don't understand why AppVeyor is trying to copy to the debug folder when it's building a release config

M.Doerner

It is literaly written in the project file.

Duga

[rubberduck-vba/Rubberduck] BitSprocket opened issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> Hello all. I am getting Procedure 'ButtonX_Click' is not used in the Code Inspections window on the latest release. I've only started developing the UI for the app I've been working on for the last few months and it seems all the control events are being flagged this way. I can easily stop inspecting for them but I'll lose visibility on dead code. I *think* I saw a related issue here on github but now I can't find it. Is this a regression or am I doing something wrong?

Access 2016 64 b

Mat's Mug

from an earlier passing build:

> RubberduckTests -> C:\projects\rubberduck\RubberduckTests\bin\Release\RubberduckTests.dll
copy Rubberduck.Inspections\bin\Release\Rubberduck.Inspections.dll RetailCoder.VBE\bin\Release\Rubberduck.Inspections.dll
1 file(s) copied

M.Doerner

Well, that is probably from the action in the yml.

WaynePhillipsEA

anyway, gtg. 'night all

Mat's Mug

9:41 PM

'night!

and thanks again!

Nelson Vides

thanks a lot! :)

Duga

[rubberduck-vba/Rubberduck] Vogel612 commented on issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> #2016 seems to be related. This could be a regression...

[rubberduck-vba/Rubberduck] retailcoder commented on issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> As a work-around you can add `'@Ignore ProcedureNotUsed` at the top of each handler, or `'@IgnoreModule ProcedureNotUsed` at the top of that module.

Is this code-behind for a UserForm, or an Access form?

[rubberduck-vba/Rubberduck] Vogel612 added label bug to issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

[rubberduck-vba/Rubberduck] Vogel612 added label feature-inspections to issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

[rubberduck-vba/Rubberduck] Vogel612 added label inspection-false-positive to issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

[rubberduck-vba/Rubberduck] BitSprocket commented on issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> Access forms.

Mat's Mug

@Duga dupe then

Vogel612

issue #2016 has been closed

Duga

[rubberduck-vba/Rubberduck] retailcoder commented on issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> Not sure if fixing #389 (damn, that's an old one!) would help.

[rubberduck-vba/Rubberduck] BitSprocket commented on issue #3626: Code inspections identifying Procedure 'X' not used regarding Form event handlers

> Okay, I do think that I am experiencing #2016 so I'll just take your advice and '@IgnoreModuleProcdureNotUsed on my forms. Thanks!

IvenBach

9:54 PM

You still here Mug?

Mat's Mug

yeah

4:54PM and not much done workwise today :(

IvenBach

Could you help with a few breadcrumbs on a logger for when a resource file doesn't have an entry?

Mat's Mug

IIRC we have unit tests covering that already

IOW adding an inspection without its FooInspectionName resource key should make that test fail

also one for FooInspectionMeta

IvenBach

ObsoleteErrorSyntaxInspectionName didn't exist.

I wasn't getting any failing tests from it either.

Mat's Mug

ok, so we need two tests that do just that

ugh

        [TestMethod]
        [TestCategory("Inspections")]
        public void InspectionName()
        {
            const string inspectionName = "OptionExplicitInspection";
            var inspection = new OptionExplicitInspection(null);

            Assert.AreEqual(inspectionName, inspection.Name);
        }

ok, we need better tests for that

Vogel612

9:58 PM

that test is horrendous, btw...

Mat's Mug

'tis

and there's basically one of such for [most] inspections

Vogel612

It should be dead-simple to run that on all inspection types

Mat's Mug

@IvenBach if you're up for some more technical debt payment, that would be two very good tests to write

and fairly easy too

(and then you can nuke all these shitty InspectionName() tests)

IvenBach

I just need code writing, period.

ThunderFrame

@WaynePhillipsEA interesting way of finding the ITypeLib. I was getting close to finding it through the Project Explorer.

Mat's Mug

10:03 PM

bookmarked:

Ideas from Wayne

2 hours ago, 1 hour 38 minutes total – 244 messages, 9 users, 6 stars

Bookmarked 27 secs ago by Mat's Mug

Nelson Vides

by the way, coming back to the libgit native binary, I just found this: github.com/libgit2/libgit2sharp/issues/…

what I want to say: I have no idea why VS is not copying this .dll along with all the others.

IvenBach

@Vogel612 What about it makes it horrendous?

Vogel612

because it's ad 1: mostly noise and ad 2: shouldn't need to be typed out for every damn inspection

Mat's Mug

mostly 2

IvenBach

@Vogel612 Would the preferred manner be something of a foreach loop to check names?

Mat's Mug

10:10 PM

use reflection to discover all IInspection implementations, loop over them

Vogel612

^^ that

Duga

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #3622: Issue3485 inspection sorting order

IvenBach

I've only read about reflection, never used it.

Duga

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

IvenBach

@Mat'sMug I see there's a lot of these tests for the Name and Type. Is the technical debt to consolidate them all into 1 location?

Mat's Mug

10:17 PM

yes

17 mins ago, by Mat's Mug

(and then you can nuke all these shitty InspectionName() tests)

IvenBach

Working on understanding how to consolidate them.

Nelson Vides

10:33 PM

does anybody knows why VS is not copying Rubberduck.Inspections.dll into the build folder when building? :(

Mat's Mug

@IvenBach you'll want to get the Rubberduck.Inspections.dll Assembly. once you have the Assembly object, use GetTypes to get all the types in it. when use LINQ to filter

@NelsonVides Ctrl+Shift+B?

if you're only building Rubberduck.dll, then Rubberduck.Inspections.dll isn't being built

i.e. build the solution, not the project

Nelson Vides

@Mat'sMug yeap

@Mat'sMug that's what I'm doing, and with a successful build

Mat's Mug

post-build events in Rubberduck.Inspections.dll should include a copy operation

Nelson Vides

@Mat'sMug oh, now I see... github.com/rubberduck-vba/Rubberduck/blob/…

only to Debug :(

Mat's Mug

you're running release config??

Nelson Vides

10:41 PM

yeap

Duga

[rubberduck-vba/Rubberduck] retailcoder merged pull request #3615: COM Release for menu items and command bar from MDoerner/Rubberduck/ShutdownIsuueAgain into rubberduck-vba/Rubberduck/next

[rubberduck-vba/Rubberduck] MDoerner pushed commit 2021073b to next: Added COM release on shutdown to menu items and the command bar.

[rubberduck-vba/Rubberduck] MDoerner pushed commit 2289637b to next: Tweaks to DockableWindowHost to ensure event unsubscription.

[rubberduck-vba/Rubberduck] MDoerner pushed commit 4b899074 to next: Reverted to having the release on the SafeComWrapper and added logging.

[rubberduck-vba/Rubberduck] MDoerner pushed commit 403a9e5e to next: Writing failure to unsubscribe from DockableWindowHost's parent subclassing windows CallBackEvent to debug.

[rubberduck-vba/Rubberduck] MDoerner pushed commit a398ebbc to next: Merge branch 'next' into ShutdownIsuueAgain

[rubberduck-vba/Rubberduck] MDoerner pushed commit 25ba0642 to next: Tweaked COM release logging.

[rubberduck-vba/Rubberduck] web-flow pushed commit 008da370 to next

Merge pull request #3615 from MDoerner/ShutdownIsuueAgain

COM Release for menu items and command bar

[rubberduck-vba/Rubberduck] retailcoder merged pull request #3622: Issue3485 inspection sorting order from IvenBach/Rubberduck/Issue3485_Inspection_Sorting_Order into rubberduck-vba/Rubberduck/next

[rubberduck-vba/Rubberduck] IvenBach pushed commit 919dde7b to next: Address sorting order issues

[rubberduck-vba/Rubberduck] IvenBach pushed commit eea22686 to next: Reorder to match new sorting order

[rubberduck-vba/Rubberduck] IvenBach pushed commit 36925d7d to next: Revert ordering and add explicit sort

[rubberduck-vba/Rubberduck] IvenBach pushed commit 43e75ddc to next: Merge branch 'next' of github.com/rubberduck-vba/rubberduck into Issue3485_Inspection_Sorting_Order

[rubberduck-vba/Rubberduck] IvenBach pushed commit 5f97d91b to next: Filter capabilities

[rubberduck-vba/Rubberduck] web-flow pushed commit 65b93c00 to next

Merge pull request #3622 from IvenBach/Issue3485_Inspection_Sorting_Order

Sort inspections in settings dialog

IvenBach

@Duga But but.... I'm not done with it...

Duga

[rubberduck-vba/Rubberduck] build for commit 008da370 on next: AppVeyor build failed

BUILD FAILURE!

Vogel612

@Mat'sMug that's what you get for merging PRs with failing build...

Duga

[rubberduck-vba/Rubberduck] build for commit 65b93c00 on next: AppVeyor build failed

BUILD FAILURE!

Nelson Vides

10:50 PM

@IvenBach Iven, if you're using git CLI, put git remote -v on your RD repo

it should show something like

IvenBach

@NelsonVides I'm not. I use VS push and sync to do it.

Nelson Vides

origin  github.com/NelsonVides/Rubberduck.git (fetch)
origin  github.com/NelsonVides/Rubberduck.git (push)
upstream        github.com/rubberduck-vba/Rubberduck.git (fetch)
upstream        github.com/rubberduck-vba/Rubberduck.git (push)

IvenBach

@NelsonVides -v stands for verbose? It just tells you what's going on?

Nelson Vides

yeap, it just give you more details

IvenBach

:+1: Thanks.

Nelson Vides

10:52 PM

git remote will list all the remotes connected to your repo

IvenBach

These simple things take me a long time to get familiar with.

Vogel612

to be more correct: -v is a common POSIX shorthand for the long-opt --verbose

Nelson Vides

as you can see in my example, there's an origin pointing to my account's repo, and an upstream pointing to the common RD repo

basically, you pull from upstream, and push to origin

Mat's Mug

ok where the F is that stupid command coming from

IvenBach

@NelsonVides That, so far, I've been able to understand and am comfortable with.

Vogel612

10:53 PM

one of the reasons I hate g++/gcc is that long-options have only a single dash before them... but some don't...

Nelson Vides

like, git pull upstream and git push origin

Vogel612

g++ --std=c++17 -lstdc++

Nelson Vides

git push upstream should give you urticaria as soon as you read it :)

Vogel612

and that's why I call the "central" remote blessed

Nelson Vides

I have no idea about VS's github's addin, so I cannot help you there, but the CLI is surely a piece of art if you want to try :)

Vogel612

10:56 PM

@NelsonVides VS's addin is a load of bullcrap

but it works enough for "sync", commit, push and actually can even switch remotes

IvenBach

@Mat'sMug I've gotten to this point. I'm mucked up on the typeof(IInspectionModel).Assembly. Trying to debug my test to see what's going on but getting a System.Io.FileNotFoundException

Vogel612

not that you could add a second remote from the UI ...

@IvenBach what about Assembly.GetCurrentAssembly()?

Mat's Mug

@Vogel612 wouldn't that be RubberdduckTests.dll?

Vogel612

sure, but we should be referencing into the loaded assemblies anyways

that makes the test a tad more resilient

IvenBach

@Vogel612 Doesn't contain a definition for that member.

Vogel612

10:59 PM