« first day (378 days earlier)      last day (2802 days later) » 

3:00 PM
@Hosch250 what is it?
@ticker wut?
> Is it possible to access VBA code in outlook using VBA in Excel?
 
First I do all the calls - and re-write them on one line just as I showed there.
Then, I do all the signatures, and re-write them on one line.
But, if you have a call written across multiple lines above the signature...
Then, the line number is messed up.
So, I need to figure out where to do the signature(s) so that won't happen.
Should actually be an easy fix, just wait til I've had breakfast.
 
3:21 PM
when you do the rewrite to a single line, would it be reasonable to fill in some blank lines as placeholders to keep all the other line numbers in sync?
 
[Vannevelj/VSDiagnostics] Vannevelj pushed commit 9ff67fb2 to issue63: Using null propagation where applicable, got rid of internal fields by making them private and exposing the rule as an internal property, re-arranged members and some other minor
 
@FreeMan that would be the easiest thing to do I guess
 
@FreeMan perhaps, but it would be an odd UX.
 
@Mat'sMug 'tis a bit kludgy, but if you kept track of where you inserted blanks, then you could remove them when the processing is complete.
 
and then that makes a good excuse to make a "cleanup vertical whitespace" feature
2
 
3:22 PM
@Mat'sMug great minds, yada yada yada...
 
lol
 
Yeah. There we go. Good idea.
Thanks @FreeMan.
 
@Hosch250 go with that ^^
the alternative is to maintain some lineOffset variable, and that gets messy
(I know, that's how I did it in the Extract Method refactoring)
 
Actually, what I'm doing is creating a List of signatures.
Then, I'll sort both the signatures the same way I do the references, and keep the index of where I am in the signatures collection (because it is smaller).
And loop the references and each loop see if I need to update a signature.
 
how many signatures can there be? not sure I follow here
 
3:28 PM
Several.
 
ok. why.
 
We have the getter/setter/letter signatures.
 
ah
 
And we have all the event handlers.
And we have the interfaces.
 
but these would be in separate modules
 
3:29 PM
Yeah.
 
so it's really only the goddamn VBA property syntax that's problematic
 
No, that is fine.
It's this that is the trouble:
Public Sub Moo(ByVal Var1 As Integer, ByVal Var2 As String, ByVal VarXYZ As Byte, ByVal Var4 As Object, ByVal Var5 As Date, ByVal Var6 As Long, ByVal Var7 As Double)

    Foo Var1, _
        Var2, _
        VarXYZ, _
        Var4
    Foo Var1, Var2, _
        VarXYZ, Var4, _
        Var5

End Sub

Public Sub Foo( _
               ByVal _
               Var1 _
               As _
               Integer _
               , _
               ByVal Var2 As String, ByVal Var3 As Byte, _
               Optional ByVal Var4 As Object, Optional ByVal Var5 As Date, _
Because I update all the references first, then the signatures.
 
Sub Moo? lol
2
 
I've actually got to interleave them.
So, I'm making two collections - one of signatures, and the other of references.
 
wouldn't it be simpler to just track how many lines you're removing above the signature, and adjust by that much to locate the signature?
also, that's an edge case, not a showstopper =)
but yeah needs fixed.
 
3:34 PM
Not really, I don't think.
Actually, hold on.
 
not really an edge case? not really needs fixed? or not really simpler to track removed lines?
 
Oh, I'm just going to add the extra whitespace.
 
or that ^
 
Not really an edge case, and not simpler to track removed lines.
 
as a VBA programmer I can assure you that multiline calls are an edge case ;-)
 
3:36 PM
Oh, those.
I thought you meant calls above the signature.
 
they're only a problem if they're multiline
 
Then I can get rid of that LINQ too, which will probably be slightly performance enhancing (not that my code is slow).
 
eh, let's not care about performance here - let's just make it work right :)
performance bottlenecks are more with parsing, resolving and inspecting than anything else
 
Why didn't I doc this upfront when I was working on it?
Stupid stupid duck.
 
Merge pull request #102 from Vannevelj/issue63

Using null propagation where applicable, got rid of internal fields b…
> Fixed in #102
 
3:44 PM
@RubberDuck with all the publicity that API got, ...I think you'd be writing that doc pretty much for yourself.. and with GitHub integrated in the IDE, why be bothered with writing VBA code to do that? I've been puzzled with that API for a while...
 
Because it's cool. No other reason.
Honestly, it started as a proof of concept, but now I feel a responsibility to maintain it.
It'a a nice exercise in COM interop if nothing else.
It's also about to get some more publicity too.
I didn't want to talk about it publicly until it was stable. Now it's stable.
 
And offers some functionality the GUI doesn't quite yet.
 
like remote hookup?
 
Yeah.
The API can do it. GUI can't.
 
3:47 PM
hmm
 
Yet.
 
The GUI is the easy part.
 
mmmm hmmmm.....
It's way easier to have a repo that's in a state where you can't commit, merge, do anything than it is to keep it in a valid state. The GUI has actually been very hard for this.
Because, once it's invalid, off to the command line you go.
Oh! @Mat'sMug, the GUI can't "Undo" yet either.
 
oh.. that's pretty major
 
Yeah.
So, I'll be back to it soon enough. I just really needed to step away from it for a while. I was starting to go crazy.
 
3:51 PM
I understand. I haven't gone back to identifier reference resolution either.
and that shit has a fatal StackOverflowException waiting to crash the VBE and Excel without warning.
there will certainly be a 1.4.1 bugfix release
 
Unless it becomes 1.5
Didn't the 1.3.1 bugfix release become the biggest release yet?
 
yeah but it was released as a "pre-release" and never really picked up. 92 downloads so far
 
And, you guys said you want to save the old config file?
 
@Hosch250 No. Not this time. I'd like to keep things in a state where we can actually release bug fixes. We boxed ourselves in between 1.22 and 1.4.
 
3:56 PM
OK.
Yeah, not pushing anything to Next until it is done will help.
 
Yeah... something like Rubberduck.Config.[DateTime].bak
That should be unique enough, right?
 
In my app, before I went to source control, I would keep 2+ versions.
One "master" ready for release at all times.
One development release.
 
Yeah. That's what I'd like to see us get back to.
 
And up to several for testing features.
 
@RubberDuck oh good idea. I was thinking Rubberduck.old.Config but then went "meh" over what should happen when that file already exists
 
3:58 PM
OK.
 
Being able to hotfix into master and merge fixes into next too.
 
It depends how far ahead of master next gets.
 
@Mat'sMug lol. Yeah. My first thought was Rubberduck.Config.bak but then I thought the same thing.
@Hosch250 What do you mean?
 
We refactored everything so much that it would take two patches to fix a bug in master and next.
 
@Mat'sMug Which API is that?
 
3:59 PM
There should probably be more releases (after every major feature?).
 
@FreeMan the source control COM API
 
Right. I'd like to see less drift.
 
@Hosch250 yes
 
Only trouble there is, we could end up releasing once a week/two days in a row.
 
4:00 PM
So what?
If we have a good build going on, who cares?
 
Lol, it might be a little pesky for people to keep up.
 
Nobody has to update if they don't want to.
Personally, I never update to a x.0 version, but I also don't update to each minor release either.
 
they'll follow us on Twitter and see what we're releasing, and at one point they'll see a feature that triggers that "oh shit, I WANT THAT!" and they'll update.
 
I check the release notes and decide if I want to.
 
so far we've been releasing "oh shit, I WANT THAT!" every release, that's the difference ;-)
 
4:04 PM
@RubberDuck @Mat'sMug Ah, didn't realize it had gotten a lot of publicity. That's good, right?
 
@FreeMan it... didn't.
 
Yeah. I was keeping it quiet until I got it stable.
I didn't know if what I did was going to be right until I used it for a bit.
<lunch>
 
hrm, I must be lost then. I'll go back to eating lunch and lurking. as you were
 
4:24 PM
[Vannevelj/RoslynTester] Vannevelj pushed commit 2b611bdc to master: AppVeyor configuration
 
When you have a unit test and you want to verify that a certain diagnostic is triggered, what would you use for that?
Up until now I've had this:
var expectedDiagnostic = new DiagnosticResult
{
	Id = TestAnalyzer.DiagnosticId,
	Message = string.Format(TestAnalyzer.Message, "Method"),
	Severity = TestAnalyzer.Severity,
	Locations =
	new[]
	{
		new DiagnosticResultLocation("Test0.cs", 10, 24)
	}
};
That's a lot of code, the test filename is completely random (it's hardcoded like that in the separate test project) and the line numbers are annoying -- I just fail the test and adjust the line numbers after every fail anyway
Personally I'm thinking I would just accept a params string[] where each string signifies the diagnostic's message that is shown
No more ID, severity or location hassle
 
@SimonAndréForsberg @Mat'sMug Sorry for creating more work, but you need to update the translation for "PromptLoadDefaultConfig".
@RubberDuck File saved, and user notified in prompt.
Got proofreader.
Or rather, I earned it. It hasn't been delivered yet.
Now its been delivered.
 
5:06 PM
@JeroenVannevel Yeah. That's a pain, but the location is important, isn't it?
Otherwise, how do you know it triggered on the right thing?
 
^ point
 
@SimonAndréForsberg I added a prompt to let the user know their old config would be saved.
Just a few words at the end.
 
@Hosch250 ...the value is "{0}".
 
Not merged yet.
 
5:10 PM
@Hosch250 ping me when it is merged into this: github.com/rubberduck-vba/Rubberduck/blob/next/RetailCoder.VBE/…
or show me where the new is
 
Here.
If you commit, there will be a merge conflict, but if you send me the translation, I can just stick it in and push.
 
FYI
0
Q: The multilingual duck

Mat's MugSo, we have this "Rubberduck Settings" window, with a number of "tabs" dedicated to each configurable feature. The "root" tab contains the application's "general settings" and, in the next release, we've found something to put there: The display language! Credits to @SimonAndréForsberg and @Vo...

 
Your old file will be saved in '{3}' ---> Din gamla fil kommer att bli sparad i '{3}'
 
@RubberDuck True, but I'm not convinced that the chances of triggering at the wrong thing are worth it
 
> The first thing I tried when I merged Simon's pull request, was of course trying to switch the display language to Swedish... and that blew up with a CultureNotFoundException.
 
5:15 PM
@JeroenVannevel You're doing the fixes by input and expected output right?
I perhaps wouldn't bother with testing the location unless there wasn't a related fix.
 
now that I think of it... I think @RubberDuck actually did the merge.
 
What merge? Wasn't me. =;)-
Seriously though, what merge?
 
2 mins ago, by Mat's Mug
> The first thing I tried when I merged Simon's pull request, was of course trying to switch the display language to Swedish... and that blew up with a CultureNotFoundException.
ninja-reworded
> The first thing I tried after Simon's pull request was merged, was of course trying to switch the display language to Swedish... and that blew up with a CultureNotFoundException.
 
xml settings? json format is all the rage now
 
@SimonAndréForsberg Thanks.
 
5:17 PM
Bah. I like xml
 
2
Q: The multilingual duck

Mat's MugSo, we have this "Rubberduck Settings" window, with a number of "tabs" dedicated to each configurable feature. The "root" tab contains the application's "general settings" and, in the next release, we've found something to put there: The display language! Credits to @SimonAndréForsberg and @Vo...

 
public override bool Equals(object obj)
{
    var other = (DisplayLanguageSetting) obj;
    return Code.Equals(other.Code);
}
 
I'm not sending this over the web.
 
@JeroenVannevel wtf would I need json for a desktop ... that ^^
 
That's an awefully optimistic Equals check
Even locally.. I like the new project.json config files for ASP.NET 5
 
Oh yeah. That can blow up with a cast exception and that should never happen in an equals.
 
#AndThisIsWhyWePostOnCodeReview
2
 
@RubberDuck Just fixed an Equals I wrote, thanks.
 
Don't thank me. Thanks @JeroenVannevel.
 
5:22 PM
@RubberDuck that's Jeroen Vannevel dos Sànchez di Castello du Aragon de Pompidou
3
 
@Mat'sMug gesundheit
 
CultureNotFoundException - the jokes just write themselves, don't they?
 
CultureNotFoundException was directed at you, or at your i18n setup? — rolfl ♦ 2 mins ago
 
5:41 PM
unlurking ^^^
 
hey @rolfl
 
Boo ;-)
Yeah, really, free-flags for obsolete/not-constructive comments....
 
@rolfl and I just flagged 2 moderator comments...
thanks :)
 
And I just processed two comment flags.
> The exception that is thrown when a method is invoked which attempts to construct a culture that is not available on the machine.
what happens if you try to construct it without first calling a method?
 
CultureInfo.GetCultureInfo does that...
 
5:54 PM
1
Q: Taking more than 10 minutes to execute 20 rows

user76130I am writing a code to check no of months passed when a sales agent has not sold a single item. Sub Latency() ' this sub-routine is used to calculate the no of recent idle months by an agent Application.ScreenUpdating = False For Each rng In Range("BL1:BL13000") j = 1 Do Until ActiveCell.Offset(...

 
uh, 20 rows?
For Each rng In Range("BL1:BL13000")
 
Yeah 20 rows.
Each row has 13.000 columns
but still, 20 rows
;-)
 
0
Q: Vba excel taking a lot of time to go through loops

Karl-Éric QuirionHi everyone and thanks for reading ! I've looked for answers to my problem but didn't found enough... Here is the idea of my code : In a table there is a number of column (chambre) and a number of row (alveole). If these matches in a list in an other sheet then it checks if it the date is betwe...

 
wut, 2 questions in < 2 minutes?!?!!
 
Damn it! Lippert just stole my checkmark!
 
6:07 PM
is Lippert related to Dogbert?
 
Closer related to Catbert.
He was one of the C# language designers once upon a time.
 
C#, JScript, VB.NET and a bunch of compilers too
 
@StackExchange I guess I'm reviewing today. Maybe I can rep-cap....
 
I got the sales reps one under way
 
Does this look like spam to anyone else?
> NAME: Gary Mitchell

EMAIL: REDACTED@gmail.com

SUBJECT: Marketing suggestion for your website

MESSAGE: Want more clients and customers? We will help them find you by putting you on the 1st page of Google. Email us back to get a full proposal.

PAGE: http://psychoproductions.net/contact-us/
REFERER:
 
6:19 PM
MoveTo > TrashCan
or
nah, don't bother replying
uh
my data slooks some thing like this — user76130 1 min ago
did you think you could paste a picture in a comment??
 
@Phrancis yes
I get that sort of thing a few times a month as well (those that pass through the spam filter)
 
When are we planning on releasing?
I've got releases on my mind.
 
if we got everything translated and we're ready... should be this weekend
if all goes well with the installer
 
Oh, and do you realize we've overrun the deadline for 1.4 again, and we have 9 days to get 1.5 out?
 
lol
 
6:29 PM
It's a good thing I don't miss deadlines for college, or this would look bad...
No hire - can't meet deadlines. The new features are nice, but if we can't release...
 
That's the great thing about RD. They're self imposed deadlines.
 
@RubberDuck you've... linked to the same SO post I did lol
and you missed j = 1 in your "proper" indentation ;-)
> I like seeing short comments like this explaining what the code does. It's a good comment.
wut?
 
Like a doc comment.
I'll elaborate on it. Thanks.
> I like seeing short comments like this explaining what the purpose of the procedure is. It's a good comment. The problem is though, I can't figure out how the Sub's name (Latency) has anything to do with this. Try to come up with a better name for it.
 
I think your answer is better than mine now :)
 
I got the change log for the api done, I can spend the rest of the day reviewing me thinkgs.
 
6:38 PM
I'll work on release notes tonight
are we good to release tomorrow-ish?
 
@JeroenVannevel What surprised me is that whatever it was, it managed to post to my contact form on my website, that wasn't a direct email
 
@Mat'sMug AFAIK, yeah.
> Release Note:
Shit tons of changes. Just look at the closed issues.
2
 
LOL
 
7:15 PM
> @WXLSinfo quoted your tweet and said "Nice"
 
0
A: VBA Excel taking a lot of time to go through loops

RubberDuckLet's think about what you're doing for a second. For ligne = 2 To lastrow DoEvents For chambre = 1 To 38 DoEvents For alveole = 1 To 6 6 * 38 = 228 iterations for every row. Wow. Yes. There's no wonder you had to give the OS time to operate. All of this just to set th...

TL;DR: Don't use VBA at all.
 
Rubberduck 1.4 should release this weekend. Working on the release notes. Lots, lots, LOTS of good stuff coming your way! #NextReleaseRocks
 
7:44 PM
LOL. @Mat'sMug did someone go download RD and expect all those features to be there?
 
looks like it!
the features page says "as of v1.4"
 
Quick draw McGraw over here... lol
Okay. Hit the cap. I guess I'll get back to updating the docs.
 
^^ Woohoo!
 
@Duga damn it! Out of stars for the stargazers.
 
8:02 PM
@RubberDuck he just offered to make a Portuguese translation!
 
Seriously? OMG..... lol
Someone should ping the Japanese guys on Twitter...
 
expecting a fork shortly ....
> RUBBERDUCKVBA SAYS: Great! Fork us on GitHub, and make a copy of Rubberduck.UI.RubberduckUI.resx with the appropriate language suffix, and submit a pull request – we’ll merge it and ship it along with the German translation!
 
I have a commit updating the version numbers waiting for me to merge in too.
I figured it's time.
 
oh crap, the guy's an MVP!!
 
Really?
 
8:08 PM
look at your inbox
oh wait
lol
he thinks he's talking to Christopher
 
Nothing in my inbox....
 
sent
his signature says Microsoft Excel MVP
 
> Microsoft Excel MVP
Well.... I'll be damned.
Sooner or later, someone's gonna nominate us.
 
you
if they all think I'm you, I'll be shrouded in your shadow :)
lol
 
8:18 PM
there's the fork!
 
@Duga ......and I'm still out of stars.
 
8:35 PM
@Mat'sMug Done.
 
awesome, thanks!
 
@JeroenVannevel I put the value there so they would know what to do, and added a "z " to the beginning to keep it straight.
I wish we had more features done.
I feel guilty pestering people for translations.
Oh well, by 2.0, most of the initial features will be done.
Who knows what the future holds.
 
@Hosch250 worst case, just stop putting that "z" prefix, and leave the English version there. the worst thing that will happen is that we'll get a GitHub issue about a missing translation :)
@RubberDuck we might have another localization issue
 
Huh?
What's up?
 
8:40 PM
the 2-letter suffix might not be enough
in The 2nd Monitor, 11 mins ago, by Ismael Miguel
ARGH. 'Brazillian' hurts my soul when I read it. It's even worst than EN-us and EN-uk
Portuguese-Portugal and Portuguese-Brazil would need two different files
 
Yuck.
 
we're currently only handling the language, not the localization
 
ISO-Codes Inc
 
Why doesn't the whole world just speak English?
 
lol
 
8:42 PM
'Merica
 
#LocalizationIsFun
...for some values of "fun"
 
Helly guys. You know - this chat is public, right? :)
 
Yup. Well aware. =)
How are you @FelipeCostaGualberto?
 
Hi!
 
@FelipeCostaGualberto public and googleable, as well as indexed and flaggable
 
8:49 PM
Fine, thanks, and you?
About the translation, I've never used GitHub. I just know Excel and VSTO, so I'll integrate my VS later to my GitHub account and find out how pulls work.
 
No worries. We appreciate the translation.
You can always find us here if you have questions.
 
feel invited to drop a ping to any of us if you're having difficulties ;)
 
Okay, so maybe we're not always here, but probably more than we should be.
 
Never collaborated in a project before.
And thanks, If I have some difficulty, I'll ask you.
 
okay now back to one round of gaming and then I'll get to cleaning out my first milestone on the helper
 
8:50 PM
This is an awesome project.
6
 
And here I am all out of stars!
Thanks. We've been working hard on it.
 
@RubberDuck Don't worry I got you covered
 
@Vogel612 ty
 
I can't wait for the day my fans start to show up
anytime now
 
back. ....aaaand too late :(
 
@JeroenVannevel I know the feeling.
I'm sure we all have fans though, we're just blinded right now by the many fans the ducks have.
 
@JeroenVannevel right about the time I talk Mug into switching over to C# 6... you can count on me to be a future fan.
@SimonAndréForsberg And you sir! You have fans in this chat room.
3
Anyway, beer time fellas. Cya.
 
@RubberDuck Naw, thanks!
 
I'll make sure most of the bugs that blow up your computer are gone by then
 
@RubberDuck :hearnoevil: lalalalala
 
9:25 PM
[Vannevelj/RoslynTester] Vannevelj pushed commit a9220d67 to master: Create an overload that accepts a simple diagnostic message instead of the entire DiagnosticResult shamawazizzle. Cleaned up the abstract overload madness and fixed a copy-paste bug tha
 
9:36 PM
> Fixed in #a9220d6
> RoslynTester v1.3.0 introduces an overload which can simply take a diagnostic's message and verify using that. This makes sure we don't have to specify ID, Severity and the exact location anymore.

These elements are cumbersome to do each time, are unintuitive in the case of the location's filename and very unlikely to be wrong anyway. Besides: who explicitly entered the exact location of the diagnostic rather than just adapting what the error message told it expected it to be?
 
9:57 PM
@RubberDuck are you online?
 

« first day (378 days earlier)      last day (2802 days later) »