« first day (337 days earlier)      last day (2843 days later) » 

Duga
Duga
00:00
RELOAD!
[Cardshifter/Cardshifter] 3 commits. 3 opened issues. 3 closed issues. 7 issue comments.
[retailcoder/Rubberduck] 2 commits. 1 issue comment.
 
4 hours later…
Duga
Duga
03:43
[retailcoder/Rubberduck] retailcoder pushed commit b13cf7ac to GrammarIsFun: fixed concurrency/async parsing issues
2
Mat's Mug
Mat's Mug 
private bool _hasStaleParseResults; // todo: use this value to give a UI cue about stale parse results
 
1 hour later…
Duga
Duga
05:09
[retailcoder/Rubberduck] retailcoder pushed commit d0395d82 to GrammarIsFun: renamed _hasStaleParseResult to _needsResync
 
9 hours later…
Stack Exchange
Stack Exchange
14:06
Running code inspections on a background thread: inspection results could be stale before they're even shown. Thoughts? @Rubberduck203
Tweeted by rubberduckvba on May 9, 2015 at 2:06 PM
Mat's Mug
Mat's Mug
14:21
Well, given the VBE doesn't tell us when code changes...
I wonder what happens if I launch code inspections and then close the toolwindow
Probably not something pretty...
skiwi
skiwi
15:14
@StackExchange Cut the thread once there's a chance it is becoming stale?
Have inspections "link" themselves into parts of the AST, and once the AST gets updated (through modification), then recalculate those inspections?
Mat's Mug
Mat's Mug
15:53
@skiwi if the VBE would notify when code changes, if could be done..
skiwi
skiwi
@Mat'sMug Watch the filesystem itself maybe?
(I don't know about how much power a plug-in has)
Mat's Mug
Mat's Mug
Code files are embedded in the excel workbook, I'd only pick up changes when file is saved ;)
skiwi
skiwi
Grmblglg, ugh
that sux
Mat's Mug
Mat's Mug
16:17
Yeah
skiwi
skiwi
Can't you send a PR to Excel to do that? :p
 
1 hour later…
RubberDuck
RubberDuck
17:27
So. I know I'm supposed to be finishing the Git Integration, but this has been bugging the shit out of me for a while, so I just did it.
There will be a pull request so we can talk about it, but..
It bugs me that every version of RD requires COM clients to relink to the assembly, so I'm breaking the project into several dlls.
It also sufficiently breaks the source control code away from the rest of the code so that I can easily license it under MIT instead of that virus license.
skiwi
skiwi
virus license?
RubberDuck
RubberDuck
17:43
GPL
skiwi
skiwi
I really don't know enough about licenses
RubberDuck
RubberDuck
Basically, if you use a GPL library, your code also has to be GPL.
skiwi
skiwi
It's like it infects other codebases
RubberDuck
RubberDuck
18:09
Yeah. That's why I call it a virus.
Which is the opposite of code being free (as in speech) in my opinion.
skiwi
skiwi
That makes sense yes
The duck makes sense
(and ducklings)
RubberDuck
RubberDuck
I have my moments. Now if I could just figure out why it's not resolving this extension method...
internal class...... ugh.
 
1 hour later…
Duga
Duga
19:38
[retailcoder/Rubberduck] ckuhn203 created branch architectureChange
[retailcoder/Rubberduck] ckuhn203 pushed commit 94365fbe to architectureChange: Broke project into several dlls
[retailcoder/Rubberduck] ckuhn203 pushed commit ecaef989 to architectureChange: Create license.md
[retailcoder/Rubberduck] ckuhn203 pushed commit 17e125b1 to architectureChange: Create license.md
 
1 hour later…
Duga
Duga
20:45
[retailcoder/Rubberduck] ckuhn203 pushed 53 commits to architectureChange
[retailcoder/Rubberduck] ckuhn203 pushed commit 4df2117b to architectureChange: resolved merge conflicts
[retailcoder/Rubberduck] ckuhn203 pushed commit 444e6a63 to architectureChange: put branch in correct project
[retailcoder/Rubberduck] ckuhn203 pushed commit 39ee5fb0 to architectureChange: fixed missing files
[retailcoder/Rubberduck] ckuhn203 created pull request #447: Architecture Change to merge architectureChange into master
[retailcoder/Rubberduck] ckuhn203 commented on pull request #447: Architecture Change
> Damn it. Wrong branch.
[retailcoder/Rubberduck] ckuhn203 rejected pull request #447: Architecture Change
Hosch250
Hosch250
Just got that email from GitHub.
Pretty big commit.
Duga
Duga
[retailcoder/Rubberduck] ckuhn203 created pull request #448: Architecture Change to merge architectureChange into next
Hosch250
Hosch250
More still!
RubberDuck
RubberDuck
21:00
Yeah. Big commit.
Hosch250
Hosch250
Walk time for the doggie, see you.
RubberDuck
RubberDuck
Oh yeah! Cc: @Mat'sMug
Mat's Mug
Mat's Mug
21:25
I'm not sure I understand the implications of this. Which license agreement does the user need to agree to?
At the end of the day, Rubberduck*.dll is Rubberduck...
Does this breach the GPL?
RubberDuck
RubberDuck
The GPL.
No. It's an aggregate.
The parts that have nothing to do with that grammar file don't have to be GPL.
Mat's Mug
Mat's Mug
Which license is Rubberduck.dll?
RubberDuck
RubberDuck
It has to be GPL because it uses code that is generated from the grammar.
Mat's Mug
Mat's Mug
Ah! Now I get it!
The PR didn't mention it ;)
RubberDuck
RubberDuck
Oh. Sorry. I got excited because I think I fixed the references breaking each release.
Mat's Mug
Mat's Mug
21:30
So we can add Rubberduck.SmartIndenter under MIT!
@RubberDuck yup, that should do it
RubberDuck
RubberDuck
Uhhh. If you contact them and ask if they'll license it that way. They already agreed to GPL.
Mat's Mug
Mat's Mug
Let's keep it GPL then
RubberDuck
RubberDuck
The catch is, we'll have to remember to manually change the assembly version when we need to.
Mat's Mug
Mat's Mug
Fine by me
RubberDuck
RubberDuck
Me too. Poke around the new structure when you get a chance. Let me know if you think everything is grouped logically.
I moved some things because I had to. So, your parser project might need some adjustment.
Mat's Mug
Mat's Mug
21:58
@RubberDuck why does Declaration need a using Rubberduck.VBEEditor;?
How does that build without circular references? I'm seeing Rubberduck.Parsing in Rubberduck.VBEEditor... or are there namespaces that don't match the dll they're in?
I need my laptop, phone doesn't handle it ;)
RubberDuck
RubberDuck
22:15
I might have missed some unused using directives.
I'm thinking about it too and 'Rubberduck.Core` might be more appropriate for that project.
Hosch250
Hosch250
@RubberDuck You can check that in vanilla VS.
Right-click them and remove and sort and stuff.
Mat's Mug
Mat's Mug
22:40
@Hosch250 you can remove them all at once across the entire solution with R#
Hosch250
Hosch250
@Mat'sMug Nice.
If you can find a version that doesn't slow VS down and doesn't shove popups in my face until I ask for them, please send me a link.
Mat's Mug
Mat's Mug
After the assemblies are cached, performance is very decent.
Hosch250
Hosch250
How can I tell it to cache the assemblies and get it done with?
Mat's Mug
Mat's Mug
Only the demo gives you popups
Hosch250
Hosch250
And what about those popups?
Mat's Mug
Mat's Mug
22:43
It caches at startup
Hosch250
Hosch250
Oh.
Well, how about this.
Mat's Mug
Mat's Mug
@Hosch250 Did you know popups you can turn off whenever you want
Hosch250
Hosch250
I want to be able to run VS with or without it by right-clicking VS and choosing whether to run it.
Mat's Mug
Mat's Mug
Well then you're out of luck
Hosch250
Hosch250
Pity :(
Mat's Mug
Mat's Mug
22:44
You're the first person I ever hear complaining about R# by the way
Hosch250
Hosch250
I mean the popup when I put my cursor in a method and scroll down to look at other code.
Mat's Mug
Mat's Mug
VS loads all installed addins at startup. Not just R#
Hosch250
Hosch250
It makes a popup that covers 2-3 LoC telling me which method my cursor is in, like I really care about that.
@Mat'sMug I don't have any addins in.
I suppose when I get a job, I'll put R# in.
Then, I can chat here for a couple minutes while VS opens.
Mat's Mug
Mat's Mug
@Hosch250 I don't know what that is
Hosch250
Hosch250
I'd post a screenshot if I wanted to bother installing it again, but I'm working on my Unity thing now.
Maybe later.
Mat's Mug
Mat's Mug
22:49
Oh about that, you'll have to take control of your startup code if you want IoC, and in MVVM the view doesn't know about the VM ;)
Hosch250
Hosch250
OK.
But, how can it bind to anything?
Don't I have to set this.DataContext = VMInstance;?
I am doing my startup code, but App.xaml.cs isn't blank, like in normal WPF, and you navigate to a page by navigating to the View, so I have no idea of where to turn.
Well, this isn't VBA, so I guess we are off-topic.
I wonder if I can find a small, working example on the web I can study
Mat's Mug
Mat's Mug
@Hosch250 you can set the datacontext from the outside...
Hosch250
Hosch250
How can I set a page's data context if I don't have the page?
Mat's Mug
Mat's Mug
@Hosch250 lol
Hosch250
Hosch250
Do I pass the page to the VM, and initialize it there?
Mat's Mug
Mat's Mug
22:54
Yes!
Hosch250
Hosch250
I'm trying to do that now.
I figured that out from you answer.
But, Windows Store is different than WPF.
I have to handle OnStart and OnSuspending in App.xaml.
Oh, NVM.
Hosch250
Hosch250
23:30
Ohh, I'm initializing my page from my VM in my simplified app!
However, I still have to call my VM with the constructor manually.
Hmm, it is passing the parameters alright, but they are null!
What have we here...
Ooh, Unity just did it for me!
2
@Mat'sMug Quick question:
MainPage is defined so: MainPage : Page
In my MainPageVM, this does not work: 
    public MainPageVM(Frame frame, Page page)
But this does: 
    public MainPageVM(Frame frame, MainPage page)
(This is when I am calling it with Unity.)
Should I be doing the first, somehow?
Somehow, I need to bind MainPage to MainPageVM and do it the first way, right?

« first day (337 days earlier)      last day (2843 days later) »