« first day (1315 days earlier)      last day (1865 days later) » 
00:00 - 19:0019:00 - 00:00

Duga
Duga
12:01 AM
RELOAD!
[rubberduck-vba/Rubberduck] 3 opened issues. 11 issue comments.
[Zomis/FactorioMods] 18 commits. 403 additions. 132 deletions.
 
IvenBach
IvenBach
I've now reached a new level of dislike for commented out code...
 
Duga
Duga
12:55 AM
[rubberduck-vba/Rubberduck] build for commit ea904287 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=h1) Report
> Merging [#3621](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f4cf930cd361dd4d67f17a97d739500ba2775b97?src=pr&el=desc) will **increase** coverage by `0.94%`.
> The diff coverage is `93.28%`.

[![Impacted file tree graph](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621/graphs/tree.svg?width=650&src=p
[rubberduck-vba/Rubberduck] build for commit ea904287 on unknown branch: 72.19% (+0.94%) compared to f4cf930
[rubberduck-vba/Rubberduck] build for commit ea904287 on unknown branch: 93.28% of diff hit (target 71.25%)
 
Tweeting Duck
Tweeting Duck
This is beyond awesome! Thank you @Microsoft for the recognition! #Excel #MVP
Tweeted by rubberduckvba on January 11, 2018 at 12:57 AM
 
IvenBach
IvenBach
I think I got the majority of the rebase done correctly.
#TimeToRewriteHistory
 
Duga
Duga
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=h1) Report
> Merging [#3621](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f4cf930cd361dd4d67f17a97d739500ba2775b97?src=pr&el=desc) will **increase** coverage by `0.94%`.
> The diff coverage is `93.28%`.

[![Impacted file tree graph](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621/graphs/tree.svg?width=650&heigh
[rubberduck-vba/Rubberduck] build for commit c643afd2 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=h1) Report
> Merging [#3621](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f4cf930cd361dd4d67f17a97d739500ba2775b97?src=pr&el=desc) will **increase** coverage by `0.92%`.
> The diff coverage is `90.9%`.

[![Impacted file tree graph](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3621/graphs/tree.svg?width=650&height
[rubberduck-vba/Rubberduck] build for commit c643afd2 on unknown branch: 90.9% of diff hit (target 71.25%)
[rubberduck-vba/Rubberduck] build for commit c643afd2 on unknown branch: 72.18% (+0.92%) compared to f4cf930
 
 
1 hour later…
this
this
2:42 AM
@ThunderFrame @Mat'sMug Indeed. It's the #1184. FWIW, if RD supports switching, that'd be awesome. The current process is that one has to remove/add references then change the compilation flag in correct order or otherwise, you get stuck in a state where VBE is complaining about references it doesn't have. Bonus points if it is still doable without RD (albeit manually).
 
Mat's Mug
Mat's Mug
@this a RD feature that works without RD?
 
this
this
IOW, RD automates steps without use of any black magic
no no, the effects that can be applied/reversed without having to have RD
 
 
1 hour later…
Mat's Mug
Mat's Mug
4:06 AM
#meta-meta
 
this
this
@Mat'sMug it's turtles all way up!
Also, @Mat'sMug any chance of reviewing my unanswered CR posts. No bites so far....
 
Mat's Mug
Mat's Mug
I shouldn't even be sitting at my desktop ATM :-/
 
this
this
sorry. other time, then.
 
Mat's Mug
Mat's Mug
is it normal that my HDD flatlines at 40% with one VM running?
...and at 100% with 3
I'm so frying my hard drive with this
 
this
this
that does seem expected, yes.
keep in mind that it won't be exactly linear
2 VM is more like 2.5 the burden
3 VM = 4... roughly.
 
Mat's Mug
Mat's Mug
4:17 AM
I only have the WinXP VM up now, and.. oh. 1% now.
 
this
this
it also depends on type of hardware you have as well. Consumer hardware aren't generally optimized for VMs.
 
Mat's Mug
Mat's Mug
IOW shut them down before it goes to sleep, or fry your hard drive in a week
 
this
this
Hm. Something's off
Sleeping should be sufficient to ease back the thorttling
if it pegs even when sleeping, then probably need to check VM configuration for possible suspects
GTG - GN!
 
Mat's Mug
Mat's Mug
IDK, I probably just panicked when I woke it up and saw a pretty constant 100% HDD usage still, and thought "damn, it's been spinning like this all day?!"
'night!
 
Duga
Duga
4:32 AM
> This is such a great idea @ThunderFrame. Also, thanks for teaching me I could just New up FSO, I should have connected the dots there but hadn't seen it that way on SO yet. Much approval here.
 
Duga
Duga
4:43 AM
> When I click it, nothing happens. Could be due to the "if user = @puzzlepiece87" clause hahaha
> Code Inspection Result: “Parameter SourceReportsFolderPath is never used”

classModel Class Module

Option Explicit

Private pSourceReportsFolderPath As String

'pSourceReportsFolderPath Properties
Public Property Get SourceReportsFolderPath() As String
SourceReportsFolderPath = pSourceReportsFolderPath
End Property
Public Property Let SourceReportsFolderPath(ByVal SourceReportsFolderPath As String)
If Right(SourceReportsFolderPath, 1) <> "\" Then SourceReportsFolderPath
 
Duga
Duga
5:39 AM
Reverted Code Explorer hotkey default to Ctrl+R

closes #3686
[rubberduck-vba/Rubberduck] build for commit a54e4c78 on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit a54e4c78 on next: 71.25% remains the same compared to f4cf930
[rubberduck-vba/Rubberduck] build for commit a54e4c78 on next: Coverage not affected when comparing f4cf930...a54e4c7
 
Tweeting Duck
Tweeting Duck
6:21 AM
It looks like the next release will put an end to the risk of crashing the host on exit (and *maybe* on unload/reload too) - thanks to the scrutiny of Wayne Phillips (you have to see http://www.everythingaccess.com/vbwatchdog.asp) and the hard work of Max Doerner. Anyone crashing with the latest build?
Tweeted by rubberduckvba on January 11, 2018 at 5:25 AM
 
IvenBach
IvenBach
7:03 AM
@TweetingDuck Cool Beans!
 
 
3 hours later…
Duga
Duga
9:57 AM
> I am trying to run rubberduck Windows 7 Enterprise, I am using big test program , when i am trying to rubberduck,Getting empty parse error
#System Details
Hardware i7-5600U @ 2.60 GHz
RAM 8 GB
64-bit OS
I am not Administrator
MS-Office -2010

why small program can able validate and rubber duck will open
if i start excel contails more than one tab and data with macros , rubber duck is not able load, giving empty parse error


Do you need any admin priviliges to run rubberduck
 
Duga
Duga
10:17 AM
> I am trying to run rubberduck Windows 7 Enterprise, I am using big test program , when i am trying to rubberduck,Getting empty parse error
#System Details
Hardware i7-5600U @ 2.60 GHz
RAM 8 GB
64-bit OS
I am not Administrator
MS-Office -2010
IG-XL Data tool also integrated with Excel
normally Excel ,i opened from Teradyn IG-XL Data Tool

why small program can able validate and rubber duck will open
if i start excel contails more than one tab and data with macros , rubber duck is
> I am trying to run rubberduck Windows 7 Enterprise, I am using big test program , when i am trying to rubberduck,Getting empty parse error
#System Details
Hardware i7-5600U @ 2.60 GHz
RAM 8 GB
64-bit OS
I am not Administrator
MS-Office -2010
IG-XL Data tool also integrated with Excel
normally Excel ,i opened from Teradyn IG-XL Data Tool

why small program can able validate and rubber duck will open
if i start excel contails more than one tab and data with macros , rubber duck is
 
Duga
Duga
10:31 AM
 
Duga
Duga
10:51 AM
> I am trying to run rubberduck Windows 7 Enterprise, I am using big test program , when i am trying to rubberduck,Getting empty parse error
#System Details
Hardware i7-5600U @ 2.60 GHz
RAM 8 GB
64-bit OS
I am not Administrator
MS-Office -2010
IG-XL Data tool also integrated with Excel
normally Excel ,i opened from Teradyn IG-XL Data Tool

why small program can able validate and rubber duck will open
if i start excel containes more than one tab and data with macros , rubber duck is
> @manjugb would you please open a separate issue for this?
It's not really a good place to discuss this in a separate (and probably unrelated) issue, because the conversation gets messy.

FWIW: You do not need administrator privileges to *run* Rubberduck, but you need them for **installation**.
> Could you please add which version of Rubberduck you're running? Thanks
> Could you please add the version information for your installation? Thanks
> The big question is: which folder are we going to put everything in by default?
 
ThunderFrame
ThunderFrame
@Duga TODO folder ;-p
 
Duga
Duga
> We already present anything that is unannotated as being in a folder with the same name as the project.
 
Duga
Duga
11:15 AM
> I thought we had an issue for this already, but I can't find one....

This is an excerpt from SO's [Why does my for loop not step up?](https://stackoverflow.com/questions/48205249/why-does-my-for-loop-not-step-up/48205439#48205249)

```vb
Sub test()

Dim h As Long
Dim BiezerStepInterval As Double

BiezerStepInterval = 0.167

For h = 0 To 1 Step BiezerStepInterval
Debug.Print h
Next h

End Sub
```

RD should be able to spot the precision mismatch
 
Vogel612
Vogel612
11:27 AM
@this you cannot start a rebase with a dirty working directory
 
 
2 hours later…
Tweeting Duck
Tweeting Duck
1:18 PM
Can you answer this? Excel VBA Multiplayer Tetris Game Loop Repaint Rate https://codereview.stackexchange.com/q/184727?atw=1 #game
Tweeted by StackCodeReview on January 11, 2018 at 1:07 PM
 
this
this
1:50 PM
@Vogel612 GTK. As I said, I never had an occasion to rebase so didn't know what to expect. Had to think about it a bit listening to Iven's observations.
 
Hosch250
Hosch250
2:02 PM
Anyone here have experience with Helix/TestTrack ticketing system?
It's truly horrible.
 
 
2 hours later…
Duga
Duga
3:48 PM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664?src=pr&el=h1) Report
> Merging [#3664](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/a54e4c781bbddcd3ca0a9d5af9198bc88738f3af?src=pr&el=desc) will **increase** coverage by `0.47%`.
> The diff coverage is `91.94%`.

[![Impacted file tree graph](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664/graphs/tree.svg?token=0zIVyLTga
[rubberduck-vba/Rubberduck] build for commit 0d703e87 on unknown branch: 71.72% (+0.47%) compared to a54e4c7
[rubberduck-vba/Rubberduck] build for commit 0a771a52 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664?src=pr&el=h1) Report
> Merging [#3664](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/a54e4c781bbddcd3ca0a9d5af9198bc88738f3af?src=pr&el=desc) will **increase** coverage by `0.47%`.
> The diff coverage is `91.93%`.

[![Impacted file tree graph](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3664/graphs/tree.svg?height=150&widt
[rubberduck-vba/Rubberduck] build for commit 0a771a52 on unknown branch: 91.93% of diff hit (target 71.25%)
[rubberduck-vba/Rubberduck] build for commit 0a771a52 on unknown branch: 71.72% (+0.47%) compared to a54e4c7
 
Quack Exchange
Quack Exchange
4:15 PM
0
Q: Print Excel Autofilter settings as VBA code to Immediate Window

peter.domanicoI had been searching for a way to print an Excel sheet's Autofilter settings to the Immediate window and I came up with this. This allows a developer to quickly convert an end user's filter settings to VBA code. I would appreciate feedback from anyone who cares to test the code. Option Explici...

vba excel
 
Nelson Vides
Nelson Vides
4:28 PM
What a day I've had today, people changing layouts and me fixing their macros... those gigantic God procedures with offsets and activesheets...
funny thing is that I had to fix one of my own macros from just three months ago and I didn't know if hitting myself because of how stupid I was or being happy for how much have I learnt
anyway, now I'm back to closing-tabs time...
gonna connect from home in a little while, hope somebody wants to come back to design time with me :)
 
IvenBach
IvenBach
4:47 PM
@NelsonVides #GottaStartSomewhere
At least you're progressing and can see that you've learned.
 
IvenBach
IvenBach
4:59 PM
Duck check. Does it make sense to have a copy-to-clipboard option for Find All References?
 
this
this
I suppose.
I personally would place more value in being able to reload the list again after doing other operations because of the click-to-jump
 
IvenBach
IvenBach
I typically do too. Right now I have grunt work that where that option would be nice to build a list.
 
puzzlepiece87
puzzlepiece87
@Hosch250 Is this frustrating design? 
Private Function InitialErrorsFound() As Boolean

    InitialErrorsFound = True

    If AllReportsInSetAreNotTheSameType = True Then Exit Sub
    If MaximumSourceReportsAtOneTimeExceeded = True Then Exit Sub

    InitialErrorsFound = False

End Function
 
Pinlop
Pinlop
glad to see most people who were here a few months ago are still here :)
Iven and this
 
puzzlepiece87
puzzlepiece87
It works well, but would you want to inherit it?
Hey Pinlop :)
 
Pinlop
Pinlop
5:05 PM
hello!
 
this
this
@IvenBach have you seen MZ Tools 8.0's approach?
 
IvenBach
IvenBach
If AllReportsInSetArNotTheSameType or MaximumSourceReportsAtOneTimeExceeded Then
        InitialErrorsFound  = True
    Else
        InitialErrorsFound  = False
    End If
@puzzlepiece87 ^
@this Nope. I have no other exposure to any VBA IDE tools/addins/whatnots
@Pinlop Making progress on studying?
 
this
this
@IvenBach I assume the intent is to allow for short-circuiting -- the Or isn't short-circuited. In this case, I'd use Select Case True
@IvenBach They use tabs to allow multiple results....
 
Hosch250
Hosch250
@puzzlepiece87 Yes.
@IvenBach That.
 
IvenBach
IvenBach
I'm pretty confident that it wasn't for short-circuiting.
 
Hosch250
Hosch250
5:10 PM
VBA has returns, doesn't it?
 
IvenBach
IvenBach
I used to write code like that as well. Never had a clue about short-circuiting either.
 
Vogel612
Vogel612
@Hosch250 no it doesn't
 
Hosch250
Hosch250
Shoot.
 
this
this
In the original, it would have. In your revision, it no longer short-circuits.
 
Vogel612
Vogel612
doesn't stop you from moving the expression out of the if-condition into the assignment ...
 
Hosch250
Hosch250
5:10 PM
Bigger question--are those subs?
 
puzzlepiece87
puzzlepiece87
@IvenBach Thank you for that feedback, but I would not want to inherit your suggested code because every additional test I add would make your conditional run even longer.
 
this
this
@puzzlepiece87 then you do want a Select Case True
 
Hosch250
Hosch250
Or are they fields, or calculated properties, or normal properties, or what?
 
IvenBach
IvenBach
@puzzlepiece87 Welp... The 3rd time I've ever been wrong it seems.
 
Vogel612
Vogel612
InitialErrorsFound = AllReportsInSetAreNotTheSameType or MaximumSourceReportsAtOneTimeExceeded
 
this
this
5:11 PM
@Vogel612 still not short-ciruciting, tho.
 
puzzlepiece87
puzzlepiece87
@IvenBach It would become If Condition 1 Or Condition 2 Or Condition 3 Or ... etc
 
Vogel612
Vogel612
but equivalent to the previously presented versio n...
 
this
this
Judging from the name, I assume those 2 are very expensive operations.
No. In previous version, it exited
 
puzzlepiece87
puzzlepiece87
@this Both short-circuiting and readability
 
Duga
Duga
> @retailcoder Hey, Mat's Mug! I finally got around to installing :) . I have checked the issues, but can't find this one. I have also read the contributing guidelines.

Environment: Win 8.1 Enterprise x64; Office 2013 x86.

Steps:
- Install [v2.1.1](https://github.com/rubberduck-vba/Rubberduck/releases/download/v2.1.1/Rubberduck.Setup.2.1.1.exe) as local admin per [the instructions](https://github.com/rubberduck-vba/Rubberduck/wiki/Installing). **However**, instead of `C:\ProgramData`
 
puzzlepiece87
puzzlepiece87
5:12 PM
@IvenBach I really do appreciate your attempt, thanks :)
 
this
this
@puzzlepiece87 Yeah, Select Case True is your best bet.
 
IvenBach
IvenBach
#AForEffort
 
Hosch250
Hosch250
Whatever. Use a real language.
:P
 
puzzlepiece87
puzzlepiece87
@Hosch250 Yup, those are subs. Each one checks for the error stated.
 
Duga
Duga
 
Hosch250
Hosch250
5:14 PM
Seems to me you might want to consider the design further than this function.
 
Duga
Duga
 
puzzlepiece87
puzzlepiece87
@this Not very expensive, but I short-circuit as a habit since VBA won't do it for me.
 
Duga
Duga
 
Hosch250
Hosch250
Maybe you should use an event to fire as soon as the error happens rather than re-calculating if it happens every time?
 
Duga
Duga
 
Hosch250
Hosch250
5:15 PM
Basically, passively listen for errors instead of actively doing the calculation, or whatever.
 
puzzlepiece87
puzzlepiece87
@Hosch250 Best advice so far lol
 
Vogel612
Vogel612
meh. .. someone needs to actively validate that stuff anyways. ..
 
this
this
@puzzlepiece87 Waitaminute - they can't be subs.
Has to be either a function or a property
 
puzzlepiece87
puzzlepiece87
@this Sorry, functions.
 
Hosch250
Hosch250
Yeah, but checking a bool is fast, and you don't even need to worry about short-circuiting.
 
this
this
5:16 PM
otherwise the code wouldn't be legal.
checking a boolean property in a Or, nobody'll notice the extra cycle wasted.
 
Hosch250
Hosch250
That ^
 
this
this
checking a function that does stuff => ????
 
puzzlepiece87
puzzlepiece87
@Hosch250 Right, but if I am in the habit of short-circuiting I won't forget to do it when it's expensive. Plus I will be able to read it because I won't have 7 conditions on the same line.
 
Hosch250
Hosch250
It won't even be an extra cycle in C...
 
puzzlepiece87
puzzlepiece87
@this Exactly, the function might do expensive stuff, who knows.
 
this
this
5:17 PM
so the answer is still Select Case True. :)
 
puzzlepiece87
puzzlepiece87
This is the initial design I was trying to avoid, btw: 
Private Function InitialErrorsFound() As Boolean

    InitialErrorsFound = False

    If AllReportsInSetAreNotTheSameType = True Then
        InitialErrorsFound = True
        Exit Sub
    End If
    If MaximumSourceReportsAtOneTimeExceeded = True Then
        InitialErrorsFound = True
        Exit Sub
    End If

End Function
 
this
this
yeah. lot of noise.
 
puzzlepiece87
puzzlepiece87
I guess the reason I was worried the code smelled in this: 
Private Function InitialErrorsFound() As Boolean

    InitialErrorsFound = True

    If AllReportsInSetAreNotTheSameType = True Then Exit Sub
    If MaximumSourceReportsAtOneTimeExceeded = True Then Exit Sub

    InitialErrorsFound = False

End Function
Was because it "assumes" an error is found before an error is actually found, then reverses the assumption if one isn't found.
I didn't know if a professional programming team would find that amateurish
I'm still trying to professionalize as much as I can.
 
this
this
Private Function InitialErrorsFound() As Boolean
  Select Case True
    Case AllReportsInSetAreNotTheSameType, _
         MaximumSourceReportsAtOneTimeExceeded
      InitialErrorsFound = True
  End Select
End Function
 
IvenBach
IvenBach
The SO flame wars & @Hosch250 consensus is VBA == amateurish.
 
puzzlepiece87
puzzlepiece87
5:21 PM
@Hosch250 Yeah, I was sort of doing that?
 
Hosch250
Hosch250
Good.
 
puzzlepiece87
puzzlepiece87
But I was trying to follow Mat's advice more. He hates when I blow the call stack to exit.
 
Hosch250
Hosch250
@IvenBach Well, it's old, and it's not like Assembly where it is still underlying everything useful.
 
puzzlepiece87
puzzlepiece87
So I was trying to pass errors up the call stack more. This time was easy since it was only one or two levels down.
@this This is beautiful - sorry, I didn't mean to make you write this.
 
this
this
@puzzlepiece87 Yeah, I prefer to use defaults and avoid adding to the noise because each line is going to prompt "why did the original author put it there?" questions.
 
puzzlepiece87
puzzlepiece87
5:23 PM
@this I liked your Select Case suggestion all along and was going to write it myself, but I was busy responding to chat and thanking people.
 
Hosch250
Hosch250
It's only useful where everything else fails :P
 
this
this
@puzzlepiece87 No worries - just wasn't sure if it was making sense so thought example best to illustrate.
 
puzzlepiece87
puzzlepiece87
@this It was very clear, thanks! I use Select Case a lot for my logic-heavy claim resolution programs.
 
IvenBach
IvenBach
@this So Select Case will allow short-circuiting whereas If ... OR ... Then doesn't?
 
this
this
Correct. Here's a proof.
 
IvenBach
IvenBach
5:26 PM
Make 2 simple functions that Debug.print
 
puzzlepiece87
puzzlepiece87
@IvenBach Good question - looking at it more closely, I wouldn't have guessed that that Select Case usage allows for short-circuiting either.
 
this
this
If 1=1 Or 1/0 Then
  Debug.Print "Yea!"
End If

Select Case True
  Case 1=1, 1/0
    Debug.Print "Nay!"
End Select
the first will fail with a runtime error
whereas the second 1/0 is simply unreachable.
 
IvenBach
IvenBach
Second won't because sc
 
this
this
#FastestTypistInTheWest :p
 
IvenBach
IvenBach
My mind works much better than my word usage.
 
puzzlepiece87
puzzlepiece87
5:28 PM
@IvenBach What do you have that's going to be breaking btw?
Thanks again @IvenBach @Hosch250 @this
 
IvenBach
IvenBach
Workbook full of engineering calcs. The person I've been waiting on the updates from has moved static cell references I've inherited and I'm going to have to fix them as I update it for Excel HPC (High performance computing) workbook offloading.
TL/DR = much unfun for me.
 
puzzlepiece87
puzzlepiece87
@this This was an enjoyable, concise example, btw. Well done.
 
Duga
Duga
> @Vogel612 thanks for the quick triage and feedback!
 
puzzlepiece87
puzzlepiece87
@IvenBach Oh no. Sorry, that sounds like a real slog.
@IvenBach Get a ton of these kind of search and error handle functions ready for your work now:
 
IvenBach
IvenBach
It's what I was hired for. I've been real active in RD chat because I've been waiting.
 
puzzlepiece87
puzzlepiece87
5:32 PM
Function GetColumnNumberOfCell(ByVal rngToSearch As Range, ByVal strEntireValueToFind As String) As Long

    Dim rngFound As Range
    Set rngFound = rngToSearch.Find(What:=strEntireValueToFind, LookAt:=xlWhole)
    If rngFound Is Nothing Then
        MsgBox "Can't find " & strEntireValueToFind & " in " & rngToSearch.Address & " on " & rngToSearch.Parent.Name & " sheet. This program cannot continue until it is present."
        Cancel
    End If

    GetColumnNumberOfCell = rngFound.Column

End Function
That way you can hopefully cut down on some of the work.
 
IvenBach
IvenBach
Smells a tad Hungarian to me.
 
puzzlepiece87
puzzlepiece87
I encourage you to write your own hahahahaha
 
IvenBach
IvenBach
I have. Various things I've needed I've built along the way.
 
puzzlepiece87
puzzlepiece87
Just an example. That kind of function is what finally made me start using a personal.xlsb. I was tired of writing idiot-proofing code over and over, or even copy-pasting it from elsewhere.
Idiot-proofing is tedious
 
IvenBach
IvenBach
That's the most obvious examples. I'll have plenty more by the time this is over.
 
puzzlepiece87
puzzlepiece87
5:35 PM
imgur's blocked at my work, what's that image? a code block?
 
IvenBach
IvenBach
CreateTableOfErrorsInActiveSheet
CreateTableOfErrorsInActiveWorkbook
CreateTableOfMacrosCalledByButtonInActiveSheet
CreateTableOfMacrosCalledByButtonInActiveWorkbook
CreateTableOfNamedRanges
CreateTableOfSubproceduresInActiveWorkbook
CreateTableOfWorksheetCodenameAndTabName
CreateTableThatListsWorksheetVisibility
That's a portion of what I have created to help mitigate the difficulties.
 
Hosch250
Hosch250
@puzzlepiece87 I know.
It's always, always better to define what they can do instead of what they can't do.
 
IvenBach
IvenBach
I need to knuckle down and create one that lists all the static cell references by going through all the CodeModules.
 
puzzlepiece87
puzzlepiece87
@IvenBach Awesome, you seem very prepared.
 
IvenBach
IvenBach
I've found the assumption is that they assume the capability is always there, even when they know better.
@puzzlepiece87 Not for this onslaught that's coming my way.
 
Hosch250
Hosch250
5:39 PM
@IvenBach Make them file a formal ticket for it if they want it :P
 
puzzlepiece87
puzzlepiece87
@Hosch250 By locking them out of spreadsheets and using userforms? That kind of define what they can do?
 
Hosch250
Hosch250
If that's what it takes :D
 
puzzlepiece87
puzzlepiece87
@Hosch250 xD
We need to photoshop you into a picture of a cowboy and sheep dogs rounding sheep into a pen or something.
 
IvenBach
IvenBach
I had a Dev file that I'd cleaned up, that one wasn't used... So now I get to migrate everything over for a second time :sigh:
 
puzzlepiece87
puzzlepiece87
@IvenBach :(
 
Hosch250
Hosch250
5:41 PM
@puzzlepiece87 Cowboys don't do sheep. They despise them.
 
IvenBach
IvenBach
I get to practice my refactoring skills?
 
Hosch250
Hosch250
In case you haven't read enough O'Henry.
 
this
this
Of course. They'd be shepherds were they otherwise.
 
IvenBach
IvenBach
I'll be semi-afk till this gets done over the next few days.
 
puzzlepiece87
puzzlepiece87
@Hosch250 Oops, yes :P My mistake.
@IvenBach Always more fun to get to refactor your own junk lol
 
IvenBach
IvenBach
5:44 PM
I've had great ideas that I refactored. Only to realize it wasn't such a great idea... That's how it goes.
 
Duga
Duga
5:57 PM
> I'm observing that some inspections' usability are being harmed due to noises generated by common patterns. Let's use inspection `Not found on compile-time interface` as an example.

In an Access VBA project, it is very common to reference `Access.Control` as the iterator in `For Each ctl In SomeForm.Controls` and within the body of loop, perhaps assign to `.Value`. Here's the funny thing - `Value` is not a member that's found on the control because it's not a member that exists on all deriv
 
Pinlop
Pinlop
@IvenBach I haven't had any time to study further since we last spoke. Answering your questions based on the code you reviewed for me is still on my to do list.
I'll get it done once I have some more time
just wnated to check in here today :)
 
Duga
Duga
> Indeed, the installer should be creating that folder - currently it's created by Rubberduck at run-time. This would be the correct fix.
> I can see a benefit with this fine-tuning, however it should be noted that the "member not on interface" false positives are a direct consequence of the resolver (or COM collector?) not being able to correctly identify the `Control` subtypes.

How should this be implemented? I think the simplest would be to introduce finer-grained annotations, e.g. `@IgnoreType Access.Control, MemberNotOnInterface`:

@IgnoreType [QualifiedTypeName], [IgnoredInspection1], [IgnoredInspection2], ...

An
> Note that even if we 1+ the collector/resolver, it still doesn't help in this case:

```
Dim ctl As Access.Control
For Each ctl In Me.Controls
If ctl.Type = acTextBox Then
ctl.Value = Null
End If
Next
```

In this case, the type of `ctl` is `Access.Control` which never will have the `Value` -- AFAICT, it simply doesn't exist. I'd have to add more variable and casting just to verify that there's legitimately a `Value` member on the type. It gets worse if the code is meant to
 
Duga
Duga
6:48 PM
> Indeed, it doesn't exist. Then the inspection result is "noise" warning you about the late-bound nature of that call, which is accurate ;-)
> I think the real solution is to somehow introduce per-inspection settings, one inspection at a time. Perhaps we could start with the ones with known false positives, like "member not on interface".
[rubberduck-vba/Rubberduck] retailcoder pushed 14 commits to next (only showing some of them below)
[rubberduck-vba/Rubberduck] Hosch250 pushed commit 5582c1ca to next: Merge Main and rename one more variable
[rubberduck-vba/Rubberduck] Hosch250 pushed commit ae447223 to next: Fix tests and add resx resource
[rubberduck-vba/Rubberduck] Hosch250 pushed commit a3e88e87 to next: Merge with main
[rubberduck-vba/Rubberduck] Hosch250 pushed commit 06bb967a to next: Upgrade project to C# 7.2
[rubberduck-vba/Rubberduck] Hosch250 pushed commit d3228bb7 to next: Merge with conflicts
[rubberduck-vba/Rubberduck] Hosch250 pushed commit 5e3e6469 to next: Add support for enabling individual experimental features
[rubberduck-vba/Rubberduck] Hosch250 pushed commit 868048fa to next: Fix test
[rubberduck-vba/Rubberduck] Hosch250 pushed commit ea904287 to next: Merge with conflicts
[rubberduck-vba/Rubberduck] Hosch250 pushed commit c643afd2 to next: Extract duplicated code into its own method
Merge pull request #3621 from Hosch250/FixResx

"Experimental Features" settings enhancements.
> No doubt RE: its accuracy - it is accurate. The issue has to do more with being able to ignore 100s of them for real outliers that needs closer look.

Yes, I'm fine with getting it set up on a per-inspection. The way I figure it, if we can have it use a XML file (let's not worry about UI just yet) and thus load via the new settings file, we've a flexible way for inspection to collect information about what it should ignore in a way that it can understand.
 
00:00 - 19:0019:00 - 00:00

« first day (1315 days earlier)      last day (1865 days later) »