VBA Rubberducking

> Note that this isn't any different than any other function call - IIf isn't being special at all here: it's just another function call that's given arguments: the arguments need to be evaluated before the call can happen, is all.

SlowLearner

2:45 AM

Just updated RD and saw the splash-screen, shocked and saddened. Didn't know ThunderFrame beyond the chat room but I know he was, like many here, incredibly generous and talented; and will be missed.

http://tributes.theage.com.au/obituaries/theage-au/obituary.aspx?n=andrew-jackson&pid=190953261

Mathieu Guindon

2:59 AM

hey @SlowLearner

curious about the functions involved in #4846

why would function calls be problematic if they're not side-effecting?

oh

oh well

Duga

[rubberduck-vba/Rubberduck] retailcoder created comment on issue #4846: Request for code Inspection: Flag functions in IIf

> Actually it is different, in that IIf itself is presumably being confused with a keyword (it's a function) - even though the problem is with side-effecting functions being used in the argument list, it wouldn't make sense to generalize this inspection and warn about any function being used in any argument list, because there is no common assumption that other functions might conditionally consume their arguments.

Mathieu Guindon

landing in 30 minutes, shutting down

this

4:50 AM

Funny how Iif keeps coming up. We had a discussion about it in another mailing list

In Access expressions, the IIf is short-circuited unlike the VBA.IIf.

It was suggested that the MS-VBAL specs on IIf is misleading because it implies it is how it is implemented when in fact it's constrainted by the requirement to evaluate all arguments before invoking function which is documented somewhere else in the specs.

@Duga confused. Are we saying that IIf is also a keyword as well as a function?

IvenBach

6:56 AM

@this so if the condition checked evaluates to true and there’s a RTE in false it barfs. But access isn’t that way?

Duga

7:34 AM

[rubberduck-vba/Rubberduck] retailcoder created pull request #4859: ObsoleteWhileWendStatementInspection to merge retailcoder/Rubberduck/WhileWendInspection into rubberduck-vba/Rubberduck/next

> Merging will close the linked issue, but the quickfix doesn't deal with replacing GoTo jumps with Exit Do (upon reflection, not sure it should anyway), only changes While...Wend to Do While...Loop. Further enhancement could be to analyze the conditional and, if negatively phrased, invert its logic and change the loop to Do Until...Loop - but that's for another PR I think.

Duga

7:49 AM

[rubberduck-vba/Rubberduck] retailcoder closed issue #4440: Range.Find may return Nothing

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

BUILD FAILURE!

[rubberduck-vba/Rubberduck] retailcoder synchronized pull request #4859: ObsoleteWhileWendStatementInspection

Duga

8:16 AM

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

[rubberduck-vba/Rubberduck] codecov[bot] created comment on pull request #4859: ObsoleteWhileWendStatementInspection

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4859?src=pr&el=h1) Report
> Merging [#4859](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4859?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f042077a5dd3d0cbba6a66307ff2bae5efb22ee9?src=pr&el=desc) will **increase** coverage by `0.01%`.
> The diff coverage is `83.33%`.

```diff
@@ Coverage Diff @@
## next #4859 +/- ##
=======================

[rubberduck-vba/Rubberduck] build for commit 9bf522b7 on unknown branch: 64.55% (target 0%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #4859: ObsoleteWhileWendStatementInspection

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4859?src=pr&el=h1) Report
> Merging [#4859](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4859?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f042077a5dd3d0cbba6a66307ff2bae5efb22ee9?src=pr&el=desc) will **increase** coverage by `0.01%`.
> The diff coverage is `83.33%`.

```diff
@@ Coverage Diff @@
## next #4859 +/- ##
=======================

Duga

8:46 AM

[rubberduck-vba/Rubberduck] retailcoder assigned retailcoder to issue #3970: Inspection for external access of forms'/documents' controls -- treat them as private

Duga

12:17 PM

[rubberduck-vba/Rubberduck] LazySquirrel888 starred us

this

12:40 PM

@IvenBach correct but it's SQL not VBA.SELECT IIf(1, 1, 1 / 0) works peachy. The equivalent fails in VBA because of the division by zero.

M.Doerner

Now that I am doing the German translations, I seem to miss a setting for the language of code templates.

I really think that the user should be able to switch back to English regardless of the language setting for the UI.

Duga

1:57 PM

[rubberduck-vba/Rubberduck] MDoerner created pull request #4860: German translations for 2.4.1 to merge MDoerner/Rubberduck/GermanTranslationsFor241 into rubberduck-vba/Rubberduck/next

[rubberduck-vba/Rubberduck] MDoerner review_requested pull request #4860: German translations for 2.4.1

Duga

2:16 PM

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

[rubberduck-vba/Rubberduck] build for commit dff81fb5 on unknown branch: 64.54% (target 0%)

Duga

3:44 PM

[rubberduck-vba/Rubberduck] mandober starred us

2

mansellan

4:09 PM

@Comintern @this artifacts uploaded to dropbox.com/s/njmpltno6c6vy0h/ODE.zip?dl=0. I put a readme in there too with what I tried and the linker output.

Duga

4:37 PM

[rubberduck-vba/Rubberduck] Vogel612 commented on Rubberduck.Resources/Inspections/InspectionInfo.de.resx of pull request #4860: German translations for 2.4.1

Duga

4:49 PM

[rubberduck-vba/Rubberduck] MDoerner created comment on pull request #4860: German translations for 2.4.1

> Hm, I did not look up the whether we use honorific or non-honorific form. I'll change it to honorific.

I avoided the umlauts because they can actually break the project when viewed with the wrong system language.

[rubberduck-vba/Rubberduck] MDoerner synchronized pull request #4860: German translations for 2.4.1

Duga

5:18 PM

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

[rubberduck-vba/Rubberduck] codecov[bot] created comment on pull request #4860: German translations for 2.4.1

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4860?src=pr&el=h1) Report
> Merging [#4860](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4860?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f042077a5dd3d0cbba6a66307ff2bae5efb22ee9?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.

```diff
@@ Coverage Diff @@
## next #4860 +/- ##
==========================

[rubberduck-vba/Rubberduck] build for commit 94420253 on unknown branch: 64.53% (target 0%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #4860: German translations for 2.4.1

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4860?src=pr&el=h1) Report
> Merging [#4860](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4860?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f042077a5dd3d0cbba6a66307ff2bae5efb22ee9?src=pr&el=desc) will **decrease** coverage by `<.01%`.
> The diff coverage is `n/a`.

```diff
@@ Coverage Diff @@
## next #4860 +/- ##
==========================

[rubberduck-vba/Rubberduck] ilkerhalil starred us

2

Vogel612

6:04 PM

It seems like I've finally been able to finish fixing my PC

Vogel612

6:16 PM

unrelated sidenote: @MathieuGuindon I love that branch name

this

@Vogel612 great! Now you don't have to worry about offending anyone! ;-)

Vogel612

~snrk

currently updating my local for Rubberduck ...

git is kinda stuck counting objects...

I'm at "104564, done"

with an intermediary repository compression in the background...

and that's only the changes from mugs repo

Delta compression using up to 4 threads.
Compressing objects: 100% (22930/22930), done.
Writing objects: 100% (104564/104564), done.
Total 104564 (delta 85697), reused 99096 (delta 80856)
Removing duplicate objects: 100% (256/256), done.
Checking connectivity: 104564, done.

Duga

7:06 PM

[rubberduck-vba/Rubberduck] SmileyFtW created comment on issue #4858: RD misbehaving with VBIDE on opening VBIDE

> I *think* it may be some window docking that is the root cause. I disabled MZT, Disabled all add-ins. Installed the latest pre-release. No joy; still the same - a complete lockup of Excel. I also did a Repair on MS Office which didn't help either.
I attempted to drag the VBE to resize and once I got a little resizing everything locked up and I had to restart Excel. Same thing when clicking the maximize button on the VBE window. Sometimes the window text would get really big and pixellated lik

Quack Exchange

7:25 PM

-1

Q: Standardize and reduce repeating same procedures in my coding Excel VBA

I am aware that my VBA code can be improved.I would like to standardize and reduce repeating same procedures. Can I have some guide lines for this? Where to start? I would be glad to have some examples from you! I would like to learn to write code more efficiently. Also does this will improve ef...

Duga

8:52 PM

[rubberduck-vba/Rubberduck] emansom starred us

3

IvenBach

11:22 PM

@QuackExchange Included a plug for Rubberduck as part of my answer.

Transcript for