« first day (1592 days earlier)      last day (1588 days later) » 
00:00 - 16:0016:00 - 19:0019:00 - 22:0022:00 - 00:00

Duga
Duga
12:01 AM
RELOAD!
[bruglesco/fleet-command] 2 commits. 1 opened issue. 178 additions. 22 deletions.
[rubberduck-vba/Battleship] 2 closed issues. 1 issue comment.
[rubberduck-vba/Rubberduck] 5 commits. 2 opened issues. 4 closed issues. 23 issue comments. 91 additions. 33 deletions.
[skiwi2/GeneticBrainfuck] 6 commits. 126 additions. 122 deletions.
> I just tried this in VB6 with the current build (my only 32-bit host ATM), and it was able to process this library and gave me what appears to be a complete serialization. Note that I do not have SAP installed, and it is possible that this library uses some forward references from an SAP library.

There were a couple minor tweaks in the library reader (although I can't imagine any of the changes would have resolved this). Would it be possible for you to test this again with the current build
 
Duga
Duga
12:22 AM
> I can't replicate the false positives for the listed inspections on next. There are 2 other false positives in the example code above:

```
Public Sub foo()
Dim bars() As Range
bars = GetBars 'Implicit default member assignment.
End Sub

Private Function GetBars() As Range()
Dim foo() As Range
ReDim foo(1 To 2)
Set foo(1) = Sheet2.Range("A1") 'Requires set keyword.
Set foo(2) = Sheet3.Range("A1") 'Requires set keyword.

GetBars = foo
 
Duga
Duga
12:49 AM
[rubberduck-vba/Rubberduck] comintern is checking what fun stuff @Duga can say about project cards
> Closing this as a duplicate of #2592. This specific example will no longer be reported after the incoming fix for #3532.
 
 
1 hour later…
Duga
Duga
2:12 AM
> Duplicates are an easy fix (PR inbound). The BTW "quickfix for all occurrences in module throws an error" appears to be due to the selection logic in the `GetQualifiedMemberName` ([here](https://github.com/rubberduck-vba/Rubberduck/blob/next/Rubberduck.CodeAnalysis/Inspections/Results/IdentifierReferenceInspectionResult.cs#L25)).

This asserts on the "in procedure" quick-fix also. The problem seems to simply be that there are multiple matching enclosures.
 
Duga
Duga
2:26 AM
[rubberduck-vba/Rubberduck] build for commit 7a59b5c1 on unknown branch: AppVeyor build failed
BUILD FAILURE!
 
Duga
Duga
2:51 AM
 
Comintern
Comintern
3:02 AM
@Duga OK, WTF? Other that the one I forgot to re-ignore, those tests pass individually...
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit 9448424e on unknown branch: AppVeyor build failed
BUILD FAILURE!
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.2%`.
> The diff coverage is `39.51%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
==========================
> Failing tests only fail in Release, and only fail if they are not run individually. Is the test setup borked again?
[rubberduck-vba/Rubberduck] build for commit 92801229 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.22%`.
> The diff coverage is `53.8%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
========================
[rubberduck-vba/Rubberduck] build for commit 92801229 on unknown branch: 60.14% (target 0%)
 
Mathieu Guindon
Mathieu Guindon
3:27 AM
@Duga unless I'm way behind, doesn't look like it
 
Comintern
Comintern
Concurrency issue? That's the only reason I can come up with why they would pass when run individually and not when running all tests.
 
Mathieu Guindon
Mathieu Guindon
what's failing?
weird
re-building
 
Comintern
Comintern
The 3 failing tests all use serialized declarations.
 
Mathieu Guindon
Mathieu Guindon
yeah I noticed... but.. they're not the only ones...
 
Comintern
Comintern
I think there's a problem with the mock parser.
Everything's cool when they're "live", and all of them pass when run singly.
if (CachedReferences.TryGetValue(reference, out var cachedDeclarations))
^ Looking at you, StaticCachingReferencedDeclarationsCollectorDecorator
 
Mathieu Guindon
Mathieu Guindon
3:39 AM
hmm
 
Comintern
Comintern
That only gets called from the mock parser. I wonder if that should block until the TryGetValue succeeds.
 
Duga
Duga
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.22%`.
> The diff coverage is `53.8%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
========================
[rubberduck-vba/Rubberduck] build for commit 9448424e on unknown branch: AppVeyor build failed
BUILD FAILURE!
[rubberduck-vba/Rubberduck] build for commit 5bceeb75 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.23%`.
> The diff coverage is `55.29%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
=======================
[rubberduck-vba/Rubberduck] build for commit 5bceeb75 on unknown branch: 60.15% (target 0%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.23%`.
> The diff coverage is `55.29%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
=======================
 
IvenBach
IvenBach
4:16 AM
@BloggingDuck Rough weekend mug? Taking your irritations out on bad code you've seen?
 
Mathieu Guindon
Mathieu Guindon
lol nah just publishing a semi-old draft :)
 
IvenBach
IvenBach
I feel the Get of my lawn wanted to creep into that post. How hard did you fight to keep it out?
 
Mathieu Guindon
Mathieu Guindon
IKR! very
 
IvenBach
IvenBach
@MathieuGuindon RE:4217 I think that is good to go. Lemme take a look again.
Rubberduck (Chris) Recommended amazon.com/gp/offer-listing/0735619670. Does anyone have any experience with that book? It's not like I already have 5 other books I need to read already.
 
Duga
Duga
4:33 AM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=h1) Report
> Merging [#4217](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/91cf0d8779bb8d20304cba670f30ca988f92e163?src=pr&el=desc) will **decrease** coverage by `3.86%`.
> The diff coverage is `7.79%`.


```diff
@@ Coverage Diff @@
## next #4217 +/- ##
========================
> F5 and unit tested. AFAICT this is up to date.
 
IvenBach
IvenBach
@Duga @MathieuGuindon
 
Mathieu Guindon
Mathieu Guindon
:+1:
 
IvenBach
IvenBach
I tried a while back to ping you on it. I'll get GH and actual collaboration figured out, just give me another 2 years.
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit 8ee0a233 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=h1) Report
> Merging [#4217](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/91cf0d8779bb8d20304cba670f30ca988f92e163?src=pr&el=desc) will **decrease** coverage by `0.08%`.
> The diff coverage is `8.82%`.


```diff
@@ Coverage Diff @@
## next #4217 +/- ##
========================
[rubberduck-vba/Rubberduck] build for commit 8ee0a233 on unknown branch: 63.63% (target 0%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=h1) Report
> Merging [#4217](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4217?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/91cf0d8779bb8d20304cba670f30ca988f92e163?src=pr&el=desc) will **decrease** coverage by `0.08%`.
> The diff coverage is `8.82%`.


```diff
@@ Coverage Diff @@
## next #4217 +/- ##
========================
 
IvenBach
IvenBach
4:55 AM
@Self Figure out how to properly prune with git. Have it auto-remove merged and subsequently deleted branches. IE #UpMyGitGame
 
Duga
Duga
 
IvenBach
IvenBach
Tackling Import module folder annotations shouldn't be that hard right?
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit e74ba3c9 on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit e74ba3c9 on next: 63.63% (target 0%)
 
Mathieu Guindon
Mathieu Guindon
5:14 AM
@IvenBach you can do it!
I'm down to 1 failing test 
Expected: <foo = "" & _
      "|" & _
      "">
  But was:  <foo = "" & _
      "" & _
      | & _
      "">
actually two 
Expected: <foo = "" _
      & "|" _
      & "">
  But was:  <foo = "" _
      & "" _
      & | _
      & "">
ok, I give up. it's not a showstopper thing, and it's one hell of a whack-a-mole feature
 
Duga
Duga
5:30 AM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.23%`.
> The diff coverage is `55.29%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
=======================
 
Duga
Duga
5:40 AM
[rubberduck-vba/Rubberduck] build for commit 0a52a913 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.15%`.
> The diff coverage is `56.03%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
=======================
[rubberduck-vba/Rubberduck] build for commit 0a52a913 on unknown branch: 60.08% (target 0%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=h1) Report
> Merging [#4418](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4418?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/f38ebf776c9de99883f2e3aca5e2291f0569bded?src=pr&el=desc) will **increase** coverage by `0.15%`.
> The diff coverage is `56.03%`.


```diff
@@ Coverage Diff @@
## next #4418 +/- ##
=======================
 
IvenBach
IvenBach
Now I know I can do it. Just had to learn how to learn first from ya`ll at the pond and from Docs.
@MathieuGuindon Sleep and get a fresh look at it tomorrow.
Night pond.
</ivenbach>
 
Duga
Duga
> @rubberduck-vba/dev there is code in here intended to handle the two failing/ignored test cases, but so far every attempt at fixing these broke more important test cases.

The two cases not working as intended are when backspacing on a line-continuated concatenated string; the intent would be to remove the entire line *when that line isn't the last line in the logical line of code*:

```
Expected: <foo = "" & _
"|" & _
"">
But was: <foo = "" & _
"" & _
| &
> @rubberduck-vba/dev there is code in here intended to handle the two failing/ignored test cases, but so far every attempt at fixing these broke more important test cases.

The two cases not working as intended are when backspacing on a line-continuated concatenated string; the intent would be to remove the entire line *when that line isn't the last line in the logical line of code*:

```
Expected: <foo = "" & _
"|" & _
"">
But was: <foo = "" & _
"" & _
| &
 
Mathieu Guindon
Mathieu Guindon
</mug>
@IvenBach if you have some spare time, let me know if you can find any weird behavior with the SCP PR that needs to be fixed prior to merge.
(I've handled the previous round of feedback)
 
 
1 hour later…
Son Goku ssj4
Son Goku ssj4
7:04 AM
Hi folks. Pulled and Rebuilded newest version from Git and my Settings won't open. Anybody with the same problem?
 
M.Doerner
M.Doerner
7:26 AM
Do you have a settings file from version 1.4.3? If yes, delete it.
 
this
this
I'm syncing the 4027, and I got those files as conflicted. They are apparently deleted from the next branch. This does not seem right. Does anyone know why those needs to be deleted?
 
M.Doerner
M.Doerner
@Comintern Part of the problem is that we are caching at the wrong point in the tests now.
Since the declarations are no longer loaded from file, they should not be cached.
Instead the COMProjects loaded from file should be cached to avoid loading them over and over from file.
Thus, the serialized projects serializer needs an interface (for the loading part) and a caching decorator.
Btw, I explicitly stated that the cache is not thread safe and that tests should not be run in parallel as a consequence.
It could be made thread-safe by wrapping everything in a lock though (or at least the try get and the insert into the dictionary).
 
this
this
7:56 AM
@this worked it out - it's actually me who deleted the file - I misread thinking someone else had deleted file but in reality it was modified which caused the conflict with this branch.
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit e7cbfe5f on unknown branch: AppVeyor build failed
BUILD FAILURE!
 
 
3 hours later…
Duga
Duga
10:31 AM
> The duck gotta be preened, look all nice & spiffy.
 
Duga
Duga
10:41 AM
[rubberduck-vba/Rubberduck] build for commit ac270895 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4439?src=pr&el=h1) Report
> Merging [#4439](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4439?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/e74ba3c960f9a794360512d8bd76d7080f08a294?src=pr&el=desc) will **decrease** coverage by `3.78%`.
> The diff coverage is `n/a`.


```diff
@@ Coverage Diff @@
## next #4439 +/- ##
==========================
[rubberduck-vba/Rubberduck] build for commit ac270895 on unknown branch: 59.84% (target 0%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4439?src=pr&el=h1) Report
> Merging [#4439](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/4439?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/e74ba3c960f9a794360512d8bd76d7080f08a294?src=pr&el=desc) will **decrease** coverage by `3.78%`.
> The diff coverage is `n/a`.


```diff
@@ Coverage Diff @@
## next #4439 +/- ##
==========================
 
Son Goku ssj4
Son Goku ssj4
10:55 AM
@M.Doerner What settings file where? Should I delete something and rebuild?
 
Mathieu Guindon
Mathieu Guindon
%appdata%\Rubberduck\rubberduck.config
 
Son Goku ssj4
Son Goku ssj4
Ah, this one. Okay. :)
Very nice RD log BTW :) Really yellow :D
So, deleting config file didn't help. Settings menu still not opening.
It didn't create one either
 
M.Doerner
M.Doerner
11:11 AM
I cannot verify atm whether this is a general problem.
 
 
2 hours later…
this
this
1:31 PM
@Duga this curiously lists the same files that came up conflicted in the 4072 branch. Those are listed as new files even though they aren't "new" in the next; they were merely modified by other PRs....
@M.Doerner for me at least, I can confirm that it works. @SonGokussj4 can you check what the Rubberduck.log says?
 
Hosch250
Hosch250
Back from vacation--took me an hour to get caught up on admin stuff like computer updates, emails...
 
FreeMan
FreeMan
1:47 PM
@Hosch250 welcome back!
Stored Procedure design question: I have a query in mind that will be used a lot to return data for a report. Sometimes I need the data as an aggregate of locations over a time period, sometimes I need the same data for a specific location (or locations) over a time period.
Would it make sense to write 1 SP that has a WHERE Location like @LocationParm and pass in % when I need all locations, (or A% when I need locations matching that), or should I write 2 different SPs, 1 without the Location check, one with?
 
Duga
Duga
 
this
this
@FreeMan #ItDepends - both might work fine and in the case where I need to know, it's best to experiment first.
 
Duga
Duga
Rename the installer script
Add new graphic files/folders
Modify the installer script to use the new graphics
Move Ducky.ico to the new Graphics folder
Merge pull request #4439 from bclothier/InstallerUpdate

Update the installer with new graphics
 
FreeMan
FreeMan
@this That's your answer to everything, isn't it... :/
 
this
this
You see, this isn't really 100% science. It's 50% science and 50% crafts.
 
FreeMan
FreeMan
1:53 PM
and 82.37% black magiks...
 
this
this
since it depends much more on your tables' shape, the data volume, etc. so one strategy that works well for one database might turn out to blow on the other database with different schema.
 
FreeMan
FreeMan
I guess they call it "Computer Science" because "Computer Science and Crafts" wouldn't attract as many matriculating freshmen to the program...
 
this
this
The engine tries to be smart as it can be but either it can't be or ends up being too clever (c.f. parameter sniffing), so you need to try them out if performance is that critical to you.
 
Duga
Duga
 
this
this
I bet you nobody would show up to "Computer Arts and Crafts"
 
Duga
Duga
1:55 PM
[rubberduck-vba/Rubberduck] comintern pushed commit fae5c6bd to next: Adds IsMissingOnInappropriateArgumentInspection
[rubberduck-vba/Rubberduck] comintern pushed commit 8c3f788d to next: Finish IsMissingOnInappropriateArgumentQuickFix.
[rubberduck-vba/Rubberduck] comintern pushed commit 6d3e51ef to next: Fix handling of optional Variant arrays, add tests.
[rubberduck-vba/Rubberduck] comintern pushed commit 44b3debf to next: Add second part of IsMissing inpections. Closes #4212
 
FreeMan
FreeMan
Not too worried about performance (though my use of the phrase "used a lot" would lead you to believe otherwise)...
 
Duga
Duga
[rubberduck-vba/Rubberduck] comintern pushed commit d974cb5d to next: Merge next
[rubberduck-vba/Rubberduck] comintern pushed commit ade00996 to next: Add qualified name for VB6 support.
[rubberduck-vba/Rubberduck] comintern pushed commit 60ce8076 to next: Remove merge artifacts, redundant tests.
 
FreeMan
FreeMan
@this the scrapbooking set might.
 
Duga
Duga
Merge pull request #4434 from comintern/ismissing

Add inspections for bad calls to IsMissing
 
FreeMan
FreeMan
trundles off to conjugate and test...
 
Mathieu Guindon
Mathieu Guindon
1:59 PM
@all any thoughts on SCP backspace not working as intended (though not in a completely borked kind of way) when deleting an empty string in the middle of a logical code line?
 
this
this
@MathieuGuindon what I'm not clear is what was the original input code?
 
Mathieu Guindon
Mathieu Guindon
I'll play with SCP some more tonight and see if anything annoying sticks out, but if the only itchy spot is that edge case then I'm ok with it 
        var original = "foo = \"\" & _\r\n      \"\" & _\r\n      \"|\" & _\r\n      \"\"".ToCodeString();
        var expected = "foo = \"\" & _\r\n      \"|\" & _\r\n      \"\"".ToCodeString();
and: 
        var original = "foo = \"\" _\r\n      & \"\" _\r\n      & \"|\" _\r\n      & \"\"".ToCodeString();
        var expected = "foo = \"\" _\r\n      & \"|\" _\r\n      & \"\"".ToCodeString();
 
this
this
Holy escapes, batman! What about string literals?
 
Mathieu Guindon
Mathieu Guindon
lol, I guess
 
FreeMan
FreeMan
Just fired up the VBE and the new splash screen made me smile.
 
this
this
2:02 PM
foo = "" & _
      "" & _
      "|" & _
      ""
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit a87f3c0f on next: AppVeyor build failed
BUILD FAILURE!
 
Mathieu Guindon
Mathieu Guindon
yeah
@Duga oh, now what
 
this
this
hm. I'm thinking this is actually preferable if all I did was backspace. What if I intended to delete a literal and put in an expression?
 
FreeMan
FreeMan
@MathieuGuindon make some string constants!
^inspection idea!!
 
Mathieu Guindon
Mathieu Guindon
@this well the idea is to have backspace work symetrically with smart-concat
 
this
this
2:05 PM
well, I shift+enter to smart-concat, right?
 
Mathieu Guindon
Mathieu Guindon
no
just enter while inside the string literal
ctrl+enter to concat & vbNewLine & _
 
this
this
hmm
@Duga isn't that an error that used to be usually ignored?
 
Mathieu Guindon
Mathieu Guindon
it works flawlessly if you're backspacing in the last line, just not if you're in the middle of the logical line
 
this
this
cos it looks like it successfully built
 
Mathieu Guindon
Mathieu Guindon
yeah but there's no GH tag for it
 
this
this
2:07 PM
ah
 
Mathieu Guindon
Mathieu Guindon
@FreeMan I'll take that over "made me cringe" ;-)
 
this
this
so just to clarify - with the scenario where I'm editing 3rd logical line to be an expression instead of literal, my backspacing the opening " would have put me on the 2nd logical line, still inside the string literal. I then have to arrow right, type space + _, then enter the expression adn then space?
 
FreeMan
FreeMan
Interesting... If you can't read a calendar and attempt to pass 9/31/2018 to a function expecting a date parameter, you get a Type Mismatch error. Thanks for that helpful pointer, VBE...
 
Mathieu Guindon
Mathieu Guindon
@this wouldn't you just go where you want to insert the expression and just type it instead?
 
Duga
Duga
 
this
this
2:14 PM
but I have to delete the original literal that I want to replace as an expression?
 
Mathieu Guindon
Mathieu Guindon
@Duga wth? make up your mind!
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit de6f9941 on next: AppVeyor build succeeded
 
FreeMan
FreeMan
lol
 
Duga
Duga
[rubberduck-vba/Rubberduck] build for commit de6f9941 on next: 63.76% (target 0%)
 
this
this
@MathieuGuindon might have been a timeout
99.9% of timeouts => panic alarms over nothing
 
Mathieu Guindon
Mathieu Guindon
2:15 PM
@this to be clear... this is only when the backspacing results in the SCP getting removed and leaving only the concatenation on the current line
 
this
this
eventually
 
Mathieu Guindon
Mathieu Guindon
i.e. it's pretty much an edge case
 
this
this
so if I'm ` "foo"| & _`
and I backspace this 5 times (?), I will be left with ` | & _` ?
 
Mathieu Guindon
Mathieu Guindon
yes
that's vanilla behavior though
 
this
this
but if I'm ` "foo|" & _ `, and I backspace 4 times, I will be on the previous logical line, right?
 
Mathieu Guindon
Mathieu Guindon
2:17 PM
physical line, assuming the previous line is in the same logical line
 
this
this
sorry, yes
Ok, now that I typed that out, it makes sense for backspacing ` "|" & _` to end up on the previous line
 
Mathieu Guindon
Mathieu Guindon
actually, since you're ending with & _, then there's another line underneath, and that's the edge case - your backspacing gets you | & _
 
this
this
I guess it'd be only a bit annoying. Not very often that I will edit several lines in the middle.
 
KySoto
KySoto
morning
 
Mathieu Guindon
Mathieu Guindon
hi!
@this yeah. that said you do get the full "" removal, so there's that
it's annoying in the sense that it's clearly not what the feature intends to do, but IMO not something to block the PR over
 
this
this
2:22 PM
Actually that got me thinking - how many of us do edit the multiple lines to fit the horizontal size? I do this for block comments but not really for string literals. Does anyone else?
 
KySoto
KySoto
im not exactly the best example, but i never use line continuations
 
Mathieu Guindon
Mathieu Guindon
I don't. picked it up when testing/playing with smart-concat deletion
(which is going to happen - people will fiddle around and see how it behaves)
 
FreeMan
FreeMan
@this I do, particularly for formatting SQL queries. BUT I'm being a good boy and starting to refactor those out, so I probably won't for very much longer.
 
Hosch250
Hosch250
@FreeMan Good boy rumples head
:P
 
Mathieu Guindon
Mathieu Guindon
SQL strings are definitely a use-case here
@this actually, I sometimes do, esp. when there are line breaks in the string.
there could even be an option to "ensure string literal line ends with a space"
 
this
this
2:30 PM
even for my SQL strings, I don't worry about horizontal size, typically.
 
Mathieu Guindon
Mathieu Guindon
my OCD can hardly stand SELECT, FROM, WHERE not being in the same column :)
then again, depends on the length of the thing
"select * from thing where abc=1234", I wouldn't bother
 
this
this
same here
 
FreeMan
FreeMan
@Hosch250 Almost as big a smile as the new yellar ducky!
 
Mathieu Guindon
Mathieu Guindon
anything involving joins though...
 
FreeMan
FreeMan
@MathieuGuindon I like that one
 
Mathieu Guindon
Mathieu Guindon
2:33 PM
SELECT foo, barFROM table has bitten everyone at least once, right?
 
FreeMan
FreeMan
Who, me? No, nevar...
 
Hosch250
Hosch250
@MathieuGuindon No.
Because I don't do my SQL like that.
 
FreeMan
FreeMan
@Hosch250 Good boy rumples head
:)
 
Mathieu Guindon
Mathieu Guindon
yeah I use views all the time too
 
Hosch250
Hosch250
I mean, I don't concatenate SQL strings in my code :P
 
Mathieu Guindon
Mathieu Guindon
2:35 PM
#depends 
        var sql = $"select * from mrp.[{table}]";
^ that hardly counts as concatenating though 
        var sql = $"select * from mrp.[{table}] where [Id]=@id";
Note, this is one of the 75+ (maybe 80 at this point) code inspections in Rubberduck (disclaimer: I manage this OSS project), which would have caught (and fixed) this problem. — Mathieu Guindon 30 secs ago
 
M.Doerner
M.Doerner
2:51 PM
Hm, if your DB is stupid, the resulting query will not work.
Square brackets are legal in table names and have to be escaped in a specific way.
 
Hosch250
Hosch250
@MathieuGuindon I don't put SQL strings in my code, period :)
 
Mathieu Guindon
Mathieu Guindon
@M.Doerner my DBs are never stupid though :)
(I like to think so)
 
this
this
@MathieuGuindon not including Sage? :D
 
Mathieu Guindon
Mathieu Guindon
my DBs 
    [MapsTo("Categories")]
    public class Category : EntityBase
    {
        [MapsTo("CollectionId")]
        public Collection Collection { get; set; }
        public string Code { get; set; }
        public string Description { get; set; }
 
this
this
technically, Sage DB is also yours, if we are talking about DB under your management as opposed to who created the schema. ;)
 
M.Doerner
M.Doerner
2:56 PM
I have a lot of sql strings in my code. I have no creation rights on our DB. So, I have to fake stored procedures.
 
Mathieu Guindon
Mathieu Guindon
@Hosch250 I don't put SQL strings in my EF CRUD apps either :)
 
M.Doerner
M.Doerner
And it has to be in VBA. So, there are no fancy ORM frameworks available.
 
Mathieu Guindon
Mathieu Guindon
        [RemoteKey("ICCATG", "CATEGORY")]
        public string SageCode { get; set; }

        [RemoteKey("GXCOLH", "COLHKEY")]
        public string SageSizeCode { get; set; }
    }
made myself an "ORM" here
 
FreeMan
FreeMan
0
Q: How can pilots minimize parasite drag?

AbdullWhat can pilots do to minimize parasite drag? As far as I know the effect of parasite drag really depends on the manufacturer of the airplane, but can pilots do anything about it?

drag pilot-technique
I want to say "fumigate the plane to kill the parasites, then keep to doors closed to keep the parasites out", but they're not quite as into the humor over there as we are here...
@MathieuGuindon Overly Rounded Mathematics?
 
Mathieu Guindon
Mathieu Guindon
Object/Relational Mapper
 
QHarr
QHarr
3:08 PM
\o
 
Mathieu Guindon
Mathieu Guindon
Got an IMappedEntity implementation for every table; most derived from EntityBase. Then a base repository class that handles 99% of everything I need to do with these entities, including fetching rows from a different database (aka "remote keys" to the Sage db) so that my view knows what valid values to load into dropdowns
long live reflection and custom attributes
 
KySoto
KySoto
does RD do 64 bit? im being lazy and not looking
 
Mathieu Guindon
Mathieu Guindon
of course it does =)
 
KySoto
KySoto
thats good
so ive come to the conclusion i have a lot of function prototypes
and its probably a good idea to 64-bit proof them
sigh
 
Hosch250
Hosch250
@KySoto Definitely a good idea.
Since there's a not too unreasonable chance of them being used in 2032 still.
Just think. 13.5 more years, and x86 will be dead.
 
KySoto
KySoto
3:28 PM
i dont honestly believe you
:P
the hard part in all of this is knowing if i need to alter the prototype declaration
the ones that use hwnd are easy, bucause i know that partiular parameter is supposed to be a longptr
 
Hosch250
Hosch250
@KySoto Yeah, well, those people will be living in the 1970s...
 
Comintern
Comintern
@M.Doerner Makes sense. I'll give that a go when I get off work.
 
Hosch250
Hosch250
(I think that's one of my best puns ever.)
 
KySoto
KySoto
lol
 
this
this
@KySoto that's what MSDN Docs is for
 
IvenBach
IvenBach
3:31 PM
#TIL Ctrl+' is much simpler than Ctrl+\, Ctrl+M to get to TeamExplorer in VS.
 
KySoto
KySoto
@this sigh
 
IvenBach
IvenBach
Glad I didn't have a tree on my car this morning. Windy today and a few small/med trees blew over onto cars.
 
this
this
3:49 PM
@IvenBach but what about all the air on your car?!?
 
IvenBach
IvenBach
It's blowing over it?
 
this
this
and pressing down on it!
 
IvenBach
IvenBach
That pressure is designed for. An abrupt tree fall not so much.
Sematics check: FileUploadPath | FileUploadLocation | FileUploadFolder. Which is preferred?
 
this
this
I always find myself needing to disambiguate between a path and a name. So I name my variable to reflect that fact.
"C:\foo\bar\` => FileUploadPath; bar\` => 'FileUploadDirectoryName`
 
IvenBach
IvenBach
Do you consider Name to just be the file name of Foo.xls?
I've seen some have FullName include Path & Name
 
this
this
3:56 PM
Yeah. Inconsistent conventions. :\
Foo.xls is the file's name
 
IvenBach
IvenBach
@MathieuGuindon Me.Range(| and typing double quote produces Me.Range ( ""|") when used on a worksheets code behind.
 
KySoto
KySoto
interesting
i had code that worked till i moved the enum to a different module (consolidating all of that stuff)
 
Mathieu Guindon
Mathieu Guindon
@IvenBach consistently?
 
IvenBach
IvenBach
Every time.
 
Tweeting Duck
Tweeting Duck
Our designers made these fancy slide templates and it just seems like all the stock photos ones shouldn't go to waste.
Tweeted by Nick_Craver on October 15, 2018 at 2:13 PM
 
IvenBach
IvenBach
3:59 PM
@MathieuGuindon Me.Range(|) and typing " --> Me.Range (""")
 
00:00 - 16:0016:00 - 19:0019:00 - 22:0022:00 - 00:00

« first day (1592 days earlier)      last day (1588 days later) »