VBA Rubberducking

Duga

12:01 AM

RELOAD!

[retailcoder/Rubberduck] 11 commits. 4 opened issues. 3 closed issues. 6 issue comments.

[Vogel612/BloggingPlatform] 2 commits. 2 issue comments.

[Zomis/Duga] 1 opened issue. 1 issue comment.

Mat's Mug

12:15 AM

@SimonAndréForsberg slacking off? :p

FreeMan

@Mat'sMug Cool beans. Neat little tool, so I'm not complaining, just letting you know what I'm seeing.

Simon André Forsberg

12:31 AM

@Mat'sMug I have been focused on interview-preparations lately, and will go AFK on Monday for a week

Mat's Mug

Good stuff! (prepares for a zombie hunt)

Simon André Forsberg

Tired stuff! (~~prepares for a~~ TTGTB)

Mat's Mug

Good night!

Mat's Mug

3:16 AM

0

A: Excel VBA - Get Folder Names

UX Before diving into the code, I have a few points to mention about the user experience of it. You're not resetting the Application.Statusbar before the procedure exits, which means whenever the code runs to completion, the Excel status bar remains "frozen" with the last processed folder. You...

A whole VBA answer without mentioning Rubberduck!

Duga

3:29 AM

[retailcoder/Rubberduck] retailcoder commented on issue #20: Option Base 1 is being used.

> Too many complex implications. Declining.

[retailcoder/Rubberduck] retailcoder closed issue #20: Option Base 1 is being used.

[retailcoder/Rubberduck] retailcoder removed label status-deferred from issue #20: Option Base 1 is being used.

[retailcoder/Rubberduck] retailcoder added label status-declined to issue #20: Option Base 1 is being used.

[retailcoder/Rubberduck] retailcoder pushed 3 commits to GrammarIsFun

Duga

4:59 AM

[retailcoder/Rubberduck] retailcoder pushed commit 3c48a19f to GrammarIsFun

> closes #283.

[retailcoder/Rubberduck] retailcoder closed issue #283: Fixing issue by the Code Inspections tool causes Compile error in a class that implements interface

Duga

9:54 AM

[retailcoder/Rubberduck] ckuhn203 pushed commit e4c032c0 to sourcecontrol

> created separate control for branches view

Mat's Mug

1:01 PM

> CodeSnippetBlog is now following you on Twitter

RubberDuck

1:11 PM

You too?

in The 2nd Monitor, 4 mins ago, by Duga

You might want Code Review. — pnuts 55 secs ago

Mat's Mug

I think most if not all variable usage inspection false positives are now resolved. I just have a few little things to tweak with rename, and then I can move on to find all references and safe delete, and then 1.29 is ready to go

Stack Exchange

1:27 PM

0

Q: Speed up a .find Macro

I would like to speed up this piece of code. The macro get a search variable from the worksheet "RAW_DATA_ARCHIVE" and then it should find it in the worksheet "RAW DATA". When it found accordance then value of column A from WS "RAW DATA" will written in the last column of "Raw_Data_Archive". Fo...

Mat's Mug

is it just me or we're getting more and more vba CR questions? :)

ugh

You don't need a VBA macro to do this. Have you seen what VLOOKUP can do for you? — Mat's Mug 16 secs ago

RubberDuck

We are, but what we need are more reviewers.

Speaking of...

0

A: Is it too easy to post your code for review?

To answer the surface question, "Is it too easy to post your code for review?", the answer is an emphatic No. It certainly is not too easy to post code for review. I don't think it could be too easy. In fact, it should be as easy as possible to come to our community and ask a question in my opini...

sᴉɔuɐɹɥԀ

^^ New SNAP error :D

2

RubberDuck

1:43 PM

Tell ya what, I'll just re-write it using the Access runtime. Even that would be better than what you've got...

@Mat'sMug why just a comment? Hell..... that's the whole answer.

Mat's Mug

lol

sᴉɔuɐɹɥԀ

1:57 PM

(hint: the product was never replaced)

3

Mat's Mug

oh wow

Mat's Mug

2:09 PM

@RubberDuck heck, posted.

Chrismas007

Someone nominate me for SO mod

jkjk

RubberDuck

I just re-read your question Dave. The last bit, about introducing a third class, that's called the Facade pattern and is quite common when exposing .Net assemblies to com interop. You may want to look into it. — RubberDuck 9 secs ago

sᴉɔuɐɹɥԀ

If Not foo Is Nothing Then 'This Syntax Is Not Makes No Sense

Mat's Mug

it's easier with braces: If Not (foo Is Nothing) Then

it's because of the Is Nothing comparison, AFAIK Is Not Nothing isn't valid, although it would read more naturally

#WhyCSharpKilledMyLoveForVBA

2

skiwi

Come on, after you learn any decent programming language I bet VBA sucks ;-)

Mat's Mug

2:20 PM

except JavaScript ;)

oh, right, decent.

2

skiwi

Sooooon this will rise to the top

39

A: A quick and easy way to join array elements with a separator (the opposite of split) in Java

Using Java 8 you can do this in a very clean way: String.join(delimiter, elements); This works in three ways: //directly specifying the elements String joined1 = String.join(",", "a", "b", "c"); //using arrays String[] array = new String[] { "a", "b", "c" }; String joined2 = String.join(",",...

#TakingOverAQuestionFrom2009

RubberDuck

To be fair, I'd rather work with js than php...

sᴉɔuɐɹɥԀ

@Mat'sMug If foo != Nothing would make the most sense...

Mat's Mug

9 mins ago, by Mat's Mug

#WhyCSharpKilledMyLoveForVBA

exactly :)

Chrismas007

A wife sent her programmer husband to the grocery story. She said, "Get a loaf of bread, if they have eggs, get a dozen." He came back with a dozen loaves of bread.

2

Mat's Mug

2:30 PM

> What? They did have eggs!

2

hi @Chrismas007!

2

Chrismas007

Three logicians walk into a bar. The bartender asks, “Do all of you want a drink?”

The first logician says, “I don’t know.”
The second logician says, “I don’t know.”
The third logician says, “Yes!”

RubberDuck

bahbhahahahah I like that one. It made me think.

How've ya been @Chrismas007?

skiwi

@RubberDuck I like it too, it made me drink.

Chrismas007

@RubberDuck Too busy for programming :-(

RubberDuck

@skiwi my job does that to me.

skiwi

2:35 PM

@RubberDuck No alcoholic beverages though :( Heading to my 2nd exam now!

RubberDuck

@skiwi see ya buddy.

Chrismas007

There is this band called "1023MB"... They haven't gotten any Gigs yet...

RubberDuck

> A COBOL programmer made a killing on the Y2K bug, so he had himself cryogenicly preserved.
He left instructions to have himself woken up when the last COBOL program had been replaced, so that he could enjoy his retirement in peace.
The old programmer woke from his artificial sleep, groggy, but joyful and ready to enjoy his retirement.
"Ahhhh... what year is it?", asked the old man.
"It's the year 9999. You know COBOL, right?"

2

Mat's Mug

in The 2nd Monitor, 49 secs ago, by Mat's Mug

I just scored +32 scanning through my own unaccepted questions and going on a checkmark-spree. Some were waaay overdue. Sorry for the delay, @Community

Chrismas007

A programmers wife is having a baby. When the baby is born, the doctor hands the baby to the programmer to announce the gender. The wife says, "Is it a boy or a girl?" The programmer responds, "True."

2

sᴉɔuɐɹɥԀ

2:44 PM

lol

3

> This is a flower pot.

Mat's Mug

3:29 PM

I'm pretty sure this fix [...] doesn't apply - but did you try it? — Mat's Mug 1 min ago

FreeMan

3:59 PM

github.com/retailcoder/Rubberduck/wiki/…

@RubberDuck, @Mat'sMug -- would it be worth changing the way the quick fix on that works. Currently it breaks the code with no hint of what it was supposed to be.

bar = TODO

Mat's Mug

yeah, that inspection is disabled

FreeMan

if that was changed to: bar = 'TODO: "foo" is never assigned into that would be a great reminder of what it was supposed to do and be much easier for the programmer to fix up

/is weird - reads documentation at lunch...

Mat's Mug

lol

@FreeMan that's a very good idea though

or NOTE: variable 'foo' is never assigned (taking the inspection result's text)

it was disabled because there were too many false positives with assignment inspections

could be worth re-enabling now

Duga

4:17 PM

[retailcoder/Rubberduck] daFreeMan commented on issue #360: Too many subfolders in the x64 install path

> This also applies to the x86 32-bit version:

`C:\Program Files\Rubberduck\Rubberduck`

Mat's Mug

#YayInstallers

sᴉɔuɐɹɥԀ

4:37 PM

@ticker Sounds like that should go on SuperUser... Not sure the gimmeh-teh-codez will be well received on SO...

sᴉɔuɐɹɥԀ

4:55 PM

@Mat'sMug Can you check my answer for accuracy?

0

A: Get a subrange that has all the cells of a range except for those on the first row?

I think you're looking for Range.Offset property. Like this: Set rng = Range("B5:H20") Set subRng = rng.Offset(RowOffset:=1, ColumnOffset:=0).Select Or shorter (but less clear): Set subRng = rng.Offset(1, 0).Select Here is a tutorial with more information: http://www.homeandlearn.org/the_of...

Lots of FGITW answers on there

Mat's Mug

I hate Offset...

Powerful but confusing as hell!

TBH it's one of the few things I have yet to fully understand in Excel...

FreeMan

yesterday, by Mat's Mug

@RubberDuck We're programmers, we can do anything

sᴉɔuɐɹɥԀ

Actually, I just updated my answer because it was wrong, a combination of Offset and Resize is needed for it to work like OP asked

FreeMan

How come this one's got a yellow start w/a black outline?

5 stars makes it special?

Mat's Mug

5:10 PM

I pinned it ;)

FreeMan

Ah. Seems.... self serving? ;)

RubberDuck

Boy IE is the best browser ever!

hides

sᴉɔuɐɹɥԀ

yesterday, by Mat's Mug

A lot of issues marked 1.3 will end up in 1.4

Feb 25 at 16:05, by RubberDuck

I say, Release Early. Release Often.

Mat's Mug

I also said "1.3 isn't shipping without source control integration"

sᴉɔuɐɹɥԀ

True :)

Mat's Mug

5:21 PM

So if we release anything without SC, it's going to be <1.3

sᴉɔuɐɹɥԀ

But isn't SC mostly a "convenience" feature rather than a "make your code better" feature?

RubberDuck

I disagree with that.

I think that what you've done would be validly labeled as 1.3 even without SC.

Even though I've made lots of progress on it, it's going slow due to other obligations.

Mat's Mug

I'll help you finish it, it's pretty much 80% there, right?

RubberDuck

@Mat'sMug 65%?

The GUI is coming together well, but we still need a way to (de)serialize repo info, store it, and transmit it into the SC presenter.

I've been averaging one commit every other day. sigh

sᴉɔuɐɹɥԀ

@RubberDuck @Mat'sMug Might I suggest something? Perhaps try polling your Twitter followers to see if they'd rather have the other 1.3 things earlier and keep SC for 1.4?

Mat's Mug

5:25 PM

And I did almost nothing throughout February

RubberDuck

And even once it's done, it's going to be buggy.

@sᴉɔuɐɹɥԀ has a point @Mat'sMug.

Maybe put a "poll" post up on RD News and share it on twitter.

sᴉɔuɐɹɥԀ

My feeling is that most people that use SC either use the command line, or a specialized program like SourceTree, rather than the IDE's built-in SC stuff. I almost never use Eclipse's SC stuff, other than to change repository

Mat's Mug

@RubberDuck Oops.

sᴉɔuɐɹɥԀ

(of course I'm not a reference as to "most programmers" since I'm not really one, at least just yet)

RubberDuck

@sᴉɔuɐɹɥԀ I use the built in stuff for committing, switching branches, etc. The routine stuff, but I default to the command line anytime I need to merge anything, re-write history, etc.

Mat's Mug

5:30 PM

I hardly ever use cmd line

Only when the team explorer borks

Stack Exchange

There's quite a bit of work left to do on the #SourceControl feature. Should we ship v1.3 without it? Inspection fixes, refactor/rename, ...

Tweeted by rubberduckvba on April 10, 2015 at 5:28 PM

FreeMan

@StackExchange I don't twit much, so I'll vote YES here. Lots of little niggley things have been fixed in the last 2 days that I'd be interested in having before a major minor release comes out.

Mat's Mug

*tweet ;)

@FreeMan under the hood, there's enough changes in that release to call it 2.0

RubberDuck

@Mat'sMug RT

FreeMan

OK, whatever you want to call it, the fixes for the few things I've found would be great!

Mat's Mug

5:36 PM

Hehe.. I'll stick to 1.3, and there's a number of other things I want to release together with the rename refactoring, too

The code explorer needs to properly support the rename refactoring, and then having all identifier references handy makes find all references and safe delete almost trivial to implement.

Or release early/often means ship what you have, release the rest later...

4

I guess these things can go for 1.31, they shouldn't take more than 2 weeks to implement

I'm about to.convince myself that we're about ready to ship 1.3

4

sᴉɔuɐɹɥԀ

6:02 PM

RT as well

Mat's Mug

Thanks!

sᴉɔuɐɹɥԀ

@Mat'sMug You're welcome ;)

FreeMan

in VBA, is there something that's inherently wrong with using Class_Initialize to populate members of the class?

I'm getting error '91' Object variable or With block variable not set when I try to set x = classobject(key), even though I've just populated it (in Class_Initalize) with a known, existing key.

Mat's Mug

What's the code?

classobject(key) is a function returning an object?

FreeMan

6:19 PM

Guess I'll post a full-blown SO question... This is a near identical copy of some other code I wrote earlier in the week. the only difference I can identify is that this one pushes values in the _Initialize(), whereas the other one gathered info 'during execution'.

I'm trying to store some predefined colors to be used later.

RubberDuck

Code plz. =;)-

FreeMan

6:32 PM

0

Q: Error 91 in class function

Using Access 2010, I'm gathering information and dropping it on an Excel spreadsheet. When I run the code below, I'm getting Run-time error '91':Object variable or With block not set in my class on this line Set Cci = ChartColorItems(ColorID) in Public Function GetRGB(ByRef ColorID As Strin...

excel excel-vba access-vba ms-access-2010

Chrismas007

what do you guys know about number formats

?

RubberDuck

@Chrismas007 enough. What's up?

Chrismas007

So if I put in a ten digit number and custom format the field as (###) ###-#### I will get a nice phone number

But if I need to add an extension, I would use something like (###) ###-#### x####

RubberDuck

That looks right to me.

Chrismas007

However, when applying that to a number without an extension, it dumps the last four digits of the 10 digit phone number into the extension (gives preference to the extension over starting in the parenthesis)

sᴉɔuɐɹɥԀ

6:35 PM

@Chrismas007 Why not just use a String type?

RubberDuck

Try (###)###-#### x0000

Chrismas007

Non VBA

RubberDuck

Oh....

sᴉɔuɐɹɥԀ

What platform/language then?

Chrismas007

Excel

RubberDuck

6:36 PM

Pretty sure it goes right to left, so you would need to right pad it to 14

Chrismas007

Format Cells dialogue box

Mat's Mug

gotta work - bbl

Chrismas007

so when i use (###) ###-#### x#### OR (###) ###-#### x0000 it takes an input of 1234567890 and makes it () 12-3456 x7890

RubberDuck

Yup. You're gonna need to right pad that number somehow.

Just trying to think how.

Chrismas007

Found out how to left pad

doesn't help lol

=IF(LEN(H3)=10, TEXT(H3, "(###) ###-####"), CONCATENATE(TEXT(LEFT(H3, 10), "(###) ###-####"), " x", MID(H3, 11, 99)))

RubberDuck

6:43 PM

@Chrismas007 hold up. I've almost got it

sᴉɔuɐɹɥԀ

Whoa, you can do IF conditions in Excel?

Chrismas007

@sᴉɔuɐɹɥԀ Sometimes I can't tell if you are being serious or sarcastic

sᴉɔuɐɹɥԀ

Serious :)

I've used Excel commands for calculations and formatting, I seriously did not know you could use it for logical operations too!

RubberDuck

@Chrismas007 this assumes the value you want to right pad is in E1.

=E1*POWER(10,(14-LEN(E1)))

must be a number

ignore the garbage, just B & E are relevant

Chrismas007

Now I just need to drop the zeroes if no extension

:-D

=IF(LEN(H4)=10, TEXT(H4, "(###) ###-####"), IF(LEN(H4)>10, CONCATENATE(TEXT(LEFT(H4, 10), "(###) ###-####"), " x", MID(H4, 11, 99)), ""))

FreeMan

6:51 PM

@sᴉɔuɐɹɥԀ It's not fun until you've got the IF statements nested 5-7 deep. Excel's little editor bar is so not useful for sorting that out...

RubberDuck

@FreeMan what should I pass into ColorId

Where's the client code?

sᴉɔuɐɹɥԀ

@FreeMan Well, you could write it in Notepad, using regular code formatting, then just paste in the bar, no?

FreeMan

        Set ColorWheel = New ChartColors
        i = 1
        For Each pnt In CChart.Chart.SeriesCollection(1).Points
          pnt.Format.Fill.ForeColor.RGB = ColorWheel.GetRGB("Pie" & i)
        Next

@sᴉɔuɐɹɥԀ quite often!

RubberDuck

I don't think the key exists in the collection.

FreeMan

NOTE: there are 10 colors defined, I cut my sample off after 2 for brevity...

Set Cci = New ChartColorItem
Cci.Red = 149
Cci.Green = 55
Cci.Blue = 53
Cci.ColorID = "Pie1"
Colors.Add Cci, Key:=Cci.ColorID
Set Cci = Nothing

RubberDuck

6:54 PM

Ohhhhhhhhhh

sᴉɔuɐɹɥԀ

Does the Excel formula ignore white space completely?

RubberDuck

LOL.

FreeMan

doesn't Colors.Add Cci, Key:=Cci.ColorID add it with the key?

Does @Duga auto-star every "LOL" only comment?

4

RubberDuck

Nope. Wasn't the key. You're gonna kick yourself @FreeMan.

0

A: Error 91 in class function

You defined the private collection at the module level as pCharColorItems, but you never initialize it in the class' intialize method. Instead, you use a locally scoped Colors collection variable. Private Sub Class_Initialize() Dim Cci As ChartColorItem Dim Colors As Collection Set Colo...

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #360: Too many subfolders in the x64 install path

> Guys, this is a nice idea and all, but we already have a bunch of people out there with it installed in this location. Do we have any idea how changing this will affect an upgrade? I don't. It needs to be tested if we do this.

RubberDuck

7:08 PM

@FreeMan @SimonAndréForsberg I have a feature request....

sᴉɔuɐɹɥԀ

18

A: What's a Zombie? And what are the many other memes of Code Review?

Meme: LOL - AutoStar Originator: Mat's Mug / Jamal Cultural Height: The 2nd Monitor Background: "lol" somehow got targeted at this point here (the first starred lol): retailcoder: lol (darn, again) Followed shortly by: Jamal: lol (don't star that) And then the tradition was cement...

FreeMan

@RubberDuck D'oh! And, I understand the difference between what I wrote earlier and this. Thanks!

RubberDuck

@FreeMan You're welcome!

FreeMan

@RubberDuck mkay....?

RubberDuck

Oh. Sorry. Didn't mean to ping you exactly.

Simon wrote Duga. She doesn't auto-star LOLs right now, but.... ahem... feature-request

FreeMan

7:12 PM

@RubberDuck Ah. I was wondering since the star was about 15 seconds after you posted...

RubberDuck

Mug's probably still lurking.

2

sigh

_GN_          '!!!!!!! for some reason placing the "not a coffin" validations before the other ones results in SA being unable to GetValue()s. It returns "#ERROR#" instead.
_GN_          ' and now the the same issue is happening whenever the POG Status1 is Pending or live, but not WIP

compiler-bug status-wont-fix

_GN_               'MerchZPlacement must be set to edge
_PS_               For planograms
_PS_                    '     I have no idea why this would fix GetValue(), but it does. See '!!!!!!! comment
_PS_               End Planograms

This makes me very afraid to touch this code... but there's a bug of mine in here sometwhere too....

Simon André Forsberg

@RubberDuck status-planned

2

RubberDuck

Nice!

Glad to know we all know which features are the really important ones.

FreeMan

7:30 PM

@sᴉɔuɐɹɥԀ Thank you. Now I feel like I'm in

FreeMan

7:44 PM

so how come there aren't options like this anymore?

skiwi

@Chrismas007 lol

2

Duga

[retailcoder/Rubberduck] PetLahev commented on issue #332: Snippet for Excel: ToggleWaitMode

> +1
Would be awesome to see it in some future version of ducky. I'd even go further like complex snippets for progress bar, database connection, XML, custom parts.. etc.
I'd let user adding her specific snippets, maybe even maintain some NuGet-like site.. hmm I've already started dreaming about it!

Mat's Mug

8:11 PM

> ExcelVBA is now following you on Twitter

> @rubberduckvba yes, you can always do v1.4 in next week :)

PetLahev

@Mat'sMug it's me

Mat's Mug

oh hai!

:)

@FreeMan huh?

RubberDuck

@Mat'sMug LOL. I'm glad people think we're that awesome...

Mat's Mug

oh, "pass parameter by value" was removed because it broke arrays... I guess it can be reinstated

RubberDuck

Hey @PetLahev. How goes it.

FreeMan

8:22 PM

@Mat'sMug But it's there on the Parameter x can be passed by value item. Seems to make sense to me that if neither is specified in the code, the 'ByRef implicitly' pops up, then me, the user, can choose for each issue what resolution I want.

Before I set the by value to DoNoShow, I would get both suggestions appearing.

PetLahev

@RubberDuck not bad, sunny day, Friday. Yeah pretty good. Going to suggest a feature for RubberDuck 2.0 :)

RubberDuck

@PetLahev Friday. Amen cuz.

Mat's Mug

@FreeMan it's there, commented-out:

        return new Dictionary<string, Action<VBE>>
            {
                // this inspection doesn't know if parameter is assigned; suggest to pass ByRef explicitly
                // and then let ParameterCanBeByVal inspection do its job.
                //{"Pass parameter by value", PassParameterByVal},
                {"Pass parameter by reference explicitly", PassParameterByRef}
            };

I need to fix these annoyances before next release

PetLahev

@Mat'sMug Looking at the time you usually send the most commits... Man, you work hard and late!

Mat's Mug

I forgot that was tracked....

FreeMan

8:28 PM

@Mat'sMug uncomment! UNCOMMENT!!!!!

If I select ByVal and break my code, it's my fault...

Mat's Mug

well if the parameter is an array, it won't let you pick it ;)

            if (Context.LPAREN() != null && Context.RPAREN() != null)
            {
                // array parameters must be passed by reference
                return new Dictionary<string, Action<VBE>>
                {
                    {"Pass parameter by reference explicitly", PassParameterByRef}
                };
            }

            return new Dictionary<string, Action<VBE>>
                {
                    // this inspection doesn't know if parameter is assigned; suggest to pass ByRef explicitly

I just need to add ParamArray to that

RubberDuck

> Has anyone ever run across an instance where moving one section of code in front of another magically made entire blocks of code be "skipped" like they weren't even there.

I know this sounds crazy, but hear me out. No. It's not a logic issue. It's two simple, unrelated, For Planograms Where blocks. I feel like I'm losing my mind here. Can anyone confirm that they've seen this bizarre behavior?

You know it's a bad day when you post stuff like this on one of your linkedin groups.

Why does this fix it?!

1 hour ago, by RubberDuck

_GN_               'MerchZPlacement must be set to edge
_PS_               For planograms
_PS_                    '     I have no idea why this would fix GetValue(), but it does. See '!!!!!!! comment
_PS_               End Planograms

Okay... Print the code.

Get it all in front you @ducky.

It's 727 lines of BASIC. There has to be a logic issue.

It's not black magic. There's a reason for this.

sᴉɔuɐɹɥԀ

8:47 PM

Ew.

RubberDuck

Yeah. I know.

Proprietary BASIC.

Which means, even if there is a bug in the compiler, I can't find it or fix it.

But I already know there is..

So what do I do? What do you do @ducky?

You spent all after noon trying to repro.

And you can, in the actual script, but never in a simplified example.

What if you copy/paste it into a new one duck?

Try that.

sᴉɔuɐɹɥԀ

Poor ducky. At least it's almost Beer-Thirty on Friday!

Duga

[retailcoder/Rubberduck] PetLahev opened issue #374: Compatibility check 32bit/64bit

> Some sort of compatibility checker between 32bit/64bit version will be handy. Consider this code Private Declare PtrSafe Function GetWindowLong Lib "user32" Alias "GetWindowLongA" _
(ByVal hwnd As LongPtr, ByVal nIndex As LongPtr) As LongPtr
This is 32/64bit safe but **won't work in 2007 or lower** Obviously this won't work with 64bit Office Private Declare Function GetWindowLong Lib "user32" Alias "GetWindowLongA"...

_
(ByVal hwnd As Long, ByVal nIndex As Long) As Long

Suggestion could be like this

#If VBA7 Then
Private Declare PtrSafe Function GetWindowLong Lib "user32" Alias "GetWindowLongA" _ (ByVal hwnd As LongPtr, ByVal nIndex As LongPtr) As LongPtr #Else Private Declare Function GetWindowLong Lib "user32" Alias "GetWindowLongA" _ (ByVal hwnd As Long, ByVal nIndex As Long) As Long #End If

sᴉɔuɐɹɥԀ

@RubberDuck You ever been to Thunder Over Louisville?

RubberDuck

Never been to Kentucky. Been begging the wife to take me for a year now. =P

Mat's Mug

8:52 PM

@PetLahev that's a very very good idea!

note, that the PtrSafe syntax isn't part of the grammar in 1.22

I added it though, so Declare statements correctly get picked up in next release

sᴉɔuɐɹɥԀ

It's this coming weekend, Sat April 18. Maybe you ought to drive over to Southern Indiana if you want to take a short trip and see the most amazing fireworks show in the country!

PetLahev

@Mat'sMug yep, sometimes it will save me hours. My wife is running 64bit office just because of this :)

Duga

[retailcoder/Rubberduck] retailcoder added label feature-request to issue #374: Compatibility check 32bit/64bit

skiwi

Holy cow, issue number 374 already?

FreeMan

ruh-roh George! again...

Mat's Mug

8:55 PM

@skiwi 97 open | 237 closed

FreeMan

Just got

Unhandled exception has occurred in a component in your application: Object reference not set to an instance of an object

it was reparsing the todo list

then Access crashed.

Will any of the debugging info Windows gathered help?

it's the stuff Dr. Watson gathers to send to MS.

the MS .NET Framework error box is unavailable to get details from

Mat's Mug

it can't hurt, for sure

oh

FreeMan

want me to spew them in here?

Mat's Mug

spew them on the issue, @Duga will spew them in here ;)

FreeMan

erm... I guess you want me to open an issue... otay...

actually, they're .cvr files. Notepad chokes on them. Have something that will read 'em?

well, one's an xml

Mat's Mug

8:59 PM

no idea

RubberDuck

and I'm printing code...... I need a beer.

Y'all have a good weekend if I don't see ya.

FreeMan

crap. how do I post the dump files on an issue in Git?

Mat's Mug

Meh, nevermind. Can you repro?

FreeMan

bah. I'll have to post on pastebin when I get home.

sᴉɔuɐɹɥԀ

@FreeMan You can't attach files to an issue... May need to upload the file somewhere, or if it's all text you can probably make files with the content on Github

FreeMan

9:04 PM

I dunno. I'm just hoping that I can recover the changes I'd made to code before it puked

Mat's Mug

Any luck with Notepad++?

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #374: Compatibility check 32bit/64bit

> :+1:

FreeMan

work machine. can't install too many tools.

files are zipped & sent home. I'll see what I can do with it from there

Yay! code's still there.

will see if it reproduces on Monday.

TTFN

sᴉɔuɐɹɥԀ

TTFN?

Chrismas007

Ta ta for now

Duga

10:02 PM

[retailcoder/Rubberduck] rossknudsen forked us into rossknudsen/Rubberduck

Mat's Mug

10:20 PM

@Duga Forked but not starred? :(

Mat's Mug

10:33 PM

> 55 downloads

RubberDuck

11:10 PM

He tried to build his own 2 years ago.

github.com/rossknudsen/VbaAddin/commits/master

Duga

11:23 PM

[retailcoder/Rubberduck] daFreeMan opened issue #375: Object reference not set to an instance of an object

> Received this error while refreshing the TODO: list in Access 2010:

>Unhandled exception has occurred in a component in your application: Object reference not set to an instance of an object

Access fully crashed, so I wasn't able to get any info from it. The Dr. Watson files are located on my Dropbox. You should be able to access it [here](https://www.dropbox.com/sh/sbvok738s8ujjwv/AABpnS1Ahrin0RQpMCVInPe1a?dl=0)

FreeMan

@Chrismas007 thanks for covering for me... :)

Mat's Mug

@RubberDuck it just occurred to me, we can measure test coverage - we know what methods are being called from test methods, and what methods are called by these public methods...

RubberDuck

OMG! That's awesome!

Mat's Mug

11:40 PM

2.0 ;)

RubberDuck

Wait.. No. That's not quite how test coverage analysis works though.

Say I have this function.

If this Then
    Bar
Else
    Baz
End If

Code coverage would need to analyze how many of those execution paths get tested.

Mat's Mug

Oh. Right.

Not quite there yet then

RubberDuck

We could pretty easily do a simple, "Hey, you've not tested this at all" alert though.

But I'd hate for that to kill anyone into a false sense of security..

s/kill/lull

stupid-phone

Mat's Mug

11:56 PM

Great! Looks much better, but there's still room for improvement, feel free to make another post anytime! — Mat's Mug 1 min ago

Transcript for