Skip to content

Commit 7d0b65f

Browse files
retailcoderretailcoder
authored
Merge pull request #2585 from comintern/next
Finish removing Project from QMN, cache settings to cut down on file IO.
2 parents bf3dd31 + c624690 commit 7d0b65f

File tree

18 files changed

+191
-118
lines changed

18 files changed

+191
-118
lines changed

‎RetailCoder.VBE/Inspections/QuickFixes/RenameProjectQuickFix.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public RenameProjectQuickFix(ParserRuleContext context, QualifiedSelection selec
2828

2929
public override void Fix()
3030
{
31-
var vbe = Selection.QualifiedName.Project.VBE;
31+
var vbe = _target.Project.VBE;
3232

3333
using (var view = new RenameDialog())
3434
{

‎RetailCoder.VBE/Root/RubberduckModule.cs

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
using Ninject.Extensions.Interception.Infrastructure.Language;
3131
using Ninject.Extensions.NamedScope;
3232
using Rubberduck.Inspections.Abstract;
33-
using Rubberduck.Parsing.Symbols;
3433
using Rubberduck.UI.CodeExplorer.Commands;
3534
using Rubberduck.UI.Command.MenuItems.CommandBars;
3635
using Rubberduck.VBEditor.Application;
@@ -191,17 +190,17 @@ private void ApplyConfigurationConvention(IEnumerable<Assembly> assemblies)
191190

192191
Bind<IPersistable<SerializableProject>>().To<XmlPersistableDeclarations>().InCallScope();
193192

194-
Bind<IPersistanceService<CodeInspectionSettings>>().To<XmlPersistanceService<CodeInspectionSettings>>().InCallScope();
195-
Bind<IPersistanceService<GeneralSettings>>().To<XmlPersistanceService<GeneralSettings>>().InCallScope();
196-
Bind<IPersistanceService<HotkeySettings>>().To<XmlPersistanceService<HotkeySettings>>().InCallScope();
197-
Bind<IPersistanceService<ToDoListSettings>>().To<XmlPersistanceService<ToDoListSettings>>().InCallScope();
198-
Bind<IPersistanceService<UnitTestSettings>>().To<XmlPersistanceService<UnitTestSettings>>().InCallScope();
199-
Bind<IPersistanceService<WindowSettings>>().To<XmlPersistanceService<WindowSettings>>().InCallScope();
200-
Bind<IPersistanceService<IndenterSettings>>().To<XmlPersistanceService<IndenterSettings>>().InCallScope();
201-
Bind<IFilePersistanceService<SourceControlSettings>>().To<XmlPersistanceService<SourceControlSettings>>().InCallScope();
193+
Bind<IPersistanceService<CodeInspectionSettings>>().To<XmlPersistanceService<CodeInspectionSettings>>().InSingletonScope();
194+
Bind<IPersistanceService<GeneralSettings>>().To<XmlPersistanceService<GeneralSettings>>().InSingletonScope();
195+
Bind<IPersistanceService<HotkeySettings>>().To<XmlPersistanceService<HotkeySettings>>().InSingletonScope();
196+
Bind<IPersistanceService<ToDoListSettings>>().To<XmlPersistanceService<ToDoListSettings>>().InSingletonScope();
197+
Bind<IPersistanceService<UnitTestSettings>>().To<XmlPersistanceService<UnitTestSettings>>().InSingletonScope();
198+
Bind<IPersistanceService<WindowSettings>>().To<XmlPersistanceService<WindowSettings>>().InSingletonScope();
199+
Bind<IPersistanceService<IndenterSettings>>().To<XmlPersistanceService<IndenterSettings>>().InSingletonScope();
200+
Bind<IFilePersistanceService<SourceControlSettings>>().To<XmlPersistanceService<SourceControlSettings>>().InSingletonScope();
202201

203-
Bind<IConfigProvider<IndenterSettings>>().To<IndenterConfigProvider>().InCallScope();
204-
Bind<IConfigProvider<SourceControlSettings>>().To<SourceControlConfigProvider>().InCallScope();
202+
Bind<IConfigProvider<IndenterSettings>>().To<IndenterConfigProvider>().InSingletonScope();
203+
Bind<IConfigProvider<SourceControlSettings>>().To<SourceControlConfigProvider>().InSingletonScope();
205204

206205
Bind<ICodeInspectionSettings>().To<CodeInspectionSettings>().InCallScope();
207206
Bind<IGeneralSettings>().To<GeneralSettings>().InCallScope();
@@ -434,7 +433,7 @@ private IEnumerable<IMenuItem> GetRubberduckMenuItems()
434433
GetSmartIndenterParentMenu(),
435434
GetToolsParentMenu(),
436435
GetRefactoringsParentMenu(),
437-
GetNavigateParentMenu(),
436+
GetNavigateParentMenu()
438437
};
439438
}
440439

@@ -446,7 +445,7 @@ private IMenuItem GetUnitTestingParentMenu()
446445
KernelInstance.Get<TestExplorerCommandMenuItem>(),
447446
KernelInstance.Get<AddTestModuleCommandMenuItem>(),
448447
KernelInstance.Get<AddTestMethodCommandMenuItem>(),
449-
KernelInstance.Get<AddTestMethodExpectedErrorCommandMenuItem>(),
448+
KernelInstance.Get<AddTestMethodExpectedErrorCommandMenuItem>()
450449
};
451450
return new UnitTestingParentMenu(items);
452451
}
@@ -477,7 +476,7 @@ private IMenuItem GetNavigateParentMenu()
477476
//KernelInstance.Get<RegexSearchReplaceCommandMenuItem>(),
478477
KernelInstance.Get<FindSymbolCommandMenuItem>(),
479478
KernelInstance.Get<FindAllReferencesCommandMenuItem>(),
480-
KernelInstance.Get<FindAllImplementationsCommandMenuItem>(),
479+
KernelInstance.Get<FindAllImplementationsCommandMenuItem>()
481480
};
482481
return new NavigateParentMenu(items);
483482
}
@@ -502,7 +501,7 @@ private IEnumerable<IMenuItem> GetCodePaneContextMenuItems()
502501
GetSmartIndenterParentMenu(),
503502
KernelInstance.Get<FindSymbolCommandMenuItem>(),
504503
KernelInstance.Get<FindAllReferencesCommandMenuItem>(),
505-
KernelInstance.Get<FindAllImplementationsCommandMenuItem>(),
504+
KernelInstance.Get<FindAllImplementationsCommandMenuItem>()
506505
};
507506
}
508507

@@ -512,7 +511,7 @@ private IMenuItem GetToolsParentMenu()
512511
{
513512
KernelInstance.Get<ShowSourceControlPanelCommandMenuItem>(),
514513
KernelInstance.Get<RegexAssistantCommandMenuItem>(),
515-
KernelInstance.Get<ToDoExplorerCommandMenuItem>(),
514+
KernelInstance.Get<ToDoExplorerCommandMenuItem>()
516515
};
517516

518517
return new ToolsParentMenu(items);
@@ -522,7 +521,7 @@ private IEnumerable<IMenuItem> GetFormDesignerContextMenuItems()
522521
{
523522
return new IMenuItem[]
524523
{
525-
KernelInstance.Get<FormDesignerRefactorRenameCommandMenuItem>(),
524+
KernelInstance.Get<FormDesignerRefactorRenameCommandMenuItem>()
526525
};
527526
}
528527

@@ -533,7 +532,7 @@ private IEnumerable<IMenuItem> GetProjectWindowContextMenuItems()
533532
KernelInstance.Get<ProjectExplorerRefactorRenameCommandMenuItem>(),
534533
KernelInstance.Get<FindSymbolCommandMenuItem>(),
535534
KernelInstance.Get<FindAllReferencesCommandMenuItem>(),
536-
KernelInstance.Get<FindAllImplementationsCommandMenuItem>(),
535+
KernelInstance.Get<FindAllImplementationsCommandMenuItem>()
537536
};
538537
}
539538
}

‎RetailCoder.VBE/Settings/CodeInspectionSettings.cs

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
using System;
22
using System.Collections.Generic;
3+
using System.Globalization;
34
using System.Linq;
45
using System.Xml.Serialization;
56
using Rubberduck.Inspections.Abstract;
@@ -16,7 +17,7 @@ public interface ICodeInspectionSettings
1617
}
1718

1819
[XmlType(AnonymousType = true)]
19-
public class CodeInspectionSettings : ICodeInspectionSettings
20+
public class CodeInspectionSettings : ICodeInspectionSettings, IEquatable<CodeInspectionSettings>
2021
{
2122
[XmlArrayItem("CodeInspection", IsNullable = false)]
2223
public HashSet<CodeInspectionSetting> CodeInspections { get; set; }
@@ -39,7 +40,7 @@ public CodeInspectionSettings(HashSet<CodeInspectionSetting> inspections, Whitel
3940

4041
public CodeInspectionSetting GetSetting<TInspection>() where TInspection : IInspection
4142
{
42-
return CodeInspections.FirstOrDefault(s => typeof(TInspection).Name.ToString().Equals(s.Name))
43+
return CodeInspections.FirstOrDefault(s => typeof(TInspection).Name.ToString(CultureInfo.InvariantCulture).Equals(s.Name))
4344
?? GetSetting(typeof(TInspection));
4445
}
4546

@@ -59,6 +60,14 @@ public CodeInspectionSetting GetSetting(Type inspection)
5960
return null;
6061
}
6162
}
63+
64+
public bool Equals(CodeInspectionSettings other)
65+
{
66+
return other != null &&
67+
CodeInspections.SequenceEqual(other.CodeInspections) &&
68+
WhitelistedIdentifiers.SequenceEqual(other.WhitelistedIdentifiers) &&
69+
RunInspectionsOnSuccessfulParse == other.RunInspectionsOnSuccessfulParse;
70+
}
6271
}
6372

6473
[XmlType(AnonymousType = true)]

‎RetailCoder.VBE/Settings/GeneralSettings.cs

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
namespace Rubberduck.Settings
77
{
8-
public interface IGeneralSettings
8+
public interface IGeneralSettings
99
{
1010
DisplayLanguageSetting Language { get; set; }
1111
bool ShowSplash { get; set; }
@@ -17,7 +17,7 @@ public interface IGeneralSettings
1717
}
1818

1919
[XmlType(AnonymousType = true)]
20-
public class GeneralSettings : IGeneralSettings
20+
public class GeneralSettings : IGeneralSettings, IEquatable<GeneralSettings>
2121
{
2222
public DisplayLanguageSetting Language { get; set; }
2323
public bool ShowSplash { get; set; }
@@ -57,5 +57,17 @@ public GeneralSettings()
5757
Delimiter = '.';
5858
MinimumLogLevel = LogLevel.Off.Ordinal;
5959
}
60+
61+
public bool Equals(GeneralSettings other)
62+
{
63+
return other != null &&
64+
Language.Equals(other.Language) &&
65+
ShowSplash == other.ShowSplash &&
66+
SmartIndenterPrompted == other.SmartIndenterPrompted &&
67+
AutoSaveEnabled == other.AutoSaveEnabled &&
68+
AutoSavePeriod == other.AutoSavePeriod &&
69+
Delimiter.Equals(other.Delimiter) &&
70+
MinimumLogLevel == other.MinimumLogLevel;
71+
}
6072
}
6173
}

‎RetailCoder.VBE/Settings/HotkeySettings.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ public interface IHotkeySettings
1010
HotkeySetting[] Settings { get; set; }
1111
}
1212

13-
public class HotkeySettings : IHotkeySettings
13+
public class HotkeySettings : IHotkeySettings, IEquatable<HotkeySettings>
1414
{
1515
private static readonly HotkeySetting[] Defaults =
1616
{
@@ -79,6 +79,11 @@ private bool IsDuplicate(HotkeySetting candidate)
7979
s.HasShiftModifier == candidate.HasShiftModifier) != null;
8080
}
8181

82+
public bool Equals(HotkeySettings other)
83+
{
84+
return other != null && Settings.SequenceEqual(other.Settings);
85+
}
86+
8287
private static bool IsValid(HotkeySetting candidate)
8388
{
8489
//This feels a bit sleazy...

‎RetailCoder.VBE/Settings/ToDoListSettings.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
using System.Collections.Generic;
1+
using System;
2+
using System.Collections.Generic;
23
using System.Linq;
34
using System.Xml.Serialization;
45
using Rubberduck.UI;
@@ -11,7 +12,7 @@ interface IToDoListSettings
1112
}
1213

1314
[XmlType(AnonymousType = true)]
14-
public class ToDoListSettings : IToDoListSettings
15+
public class ToDoListSettings : IToDoListSettings, IEquatable<ToDoListSettings>
1516
{
1617
private IEnumerable<ToDoMarker> _markers;
1718

@@ -39,5 +40,10 @@ public ToDoListSettings(IEnumerable<ToDoMarker> markers)
3940
{
4041
_markers = markers;
4142
}
43+
44+
public bool Equals(ToDoListSettings other)
45+
{
46+
return other != null && ToDoMarkers.SequenceEqual(other.ToDoMarkers);
47+
}
4248
}
4349
}

‎RetailCoder.VBE/Settings/UnitTestSettings.cs

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
using System.Xml.Serialization;
1+
using System;
2+
using System.Xml.Serialization;
23

34
namespace Rubberduck.Settings
45
{
@@ -25,7 +26,7 @@ public interface IUnitTestSettings
2526
}
2627

2728
[XmlType(AnonymousType = true)]
28-
public class UnitTestSettings : IUnitTestSettings
29+
public class UnitTestSettings : IUnitTestSettings, IEquatable<UnitTestSettings>
2930
{
3031
public UnitTestSettings()
3132
: this(BindingMode.LateBinding, AssertMode.StrictAssert, true, true, false)
@@ -47,5 +48,15 @@ public UnitTestSettings(BindingMode bindingMode, AssertMode assertMode, bool mod
4748
public bool ModuleInit { get; set; }
4849
public bool MethodInit { get; set; }
4950
public bool DefaultTestStubInNewModule { get; set; }
51+
52+
public bool Equals(UnitTestSettings other)
53+
{
54+
return other != null &&
55+
BindingMode == other.BindingMode &&
56+
AssertMode == other.AssertMode &&
57+
ModuleInit == other.ModuleInit &&
58+
MethodInit == other.MethodInit &&
59+
DefaultTestStubInNewModule == other.DefaultTestStubInNewModule;
60+
}
5061
}
5162
}

‎RetailCoder.VBE/Settings/WindowSettings.cs

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
using System.Xml.Serialization;
2-
using Rubberduck.Properties;
1+
using System;
2+
using System.Xml.Serialization;
33
using Rubberduck.UI;
44
using Rubberduck.UI.CodeExplorer;
55
using Rubberduck.UI.Inspections;
@@ -21,7 +21,7 @@ public interface IWindowSettings
2121
}
2222

2323
[XmlType(AnonymousType = true)]
24-
public class WindowSettings : IWindowSettings
24+
public class WindowSettings : IWindowSettings, IEquatable<WindowSettings>
2525
{
2626
public WindowSettings()
2727
: this(false, false, false, false, false)
@@ -71,5 +71,15 @@ public bool IsWindowVisible(DockableToolwindowPresenter candidate)
7171
//Oh. Hello. I have no clue who you are...
7272
return false;
7373
}
74+
75+
public bool Equals(WindowSettings other)
76+
{
77+
return other != null &&
78+
CodeExplorerVisibleOnStartup == other.CodeExplorerVisibleOnStartup &&
79+
CodeInspectionsVisibleOnStartup == other.CodeInspectionsVisibleOnStartup &&
80+
SourceControlVisibleOnStartup == other.SourceControlVisibleOnStartup &&
81+
TestExplorerVisibleOnStartup == other.TestExplorerVisibleOnStartup &&
82+
TodoExplorerVisibleOnStartup == other.TodoExplorerVisibleOnStartup;
83+
}
7484
}
7585
}

‎Rubberduck.Parsing/Symbols/Declaration.cs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,7 @@ public Declaration(
119119
isBuiltIn,
120120
annotations,
121121
attributes)
122-
{
123-
}
122+
{ }
124123

125124
public Declaration(
126125
QualifiedMemberName qualifiedName,
@@ -140,8 +139,7 @@ public Declaration(
140139
IEnumerable<IAnnotation> annotations = null,
141140
Attributes attributes = null)
142141
{
143-
_qualifiedName = qualifiedName;
144-
_project = qualifiedName.QualifiedModuleName.Project;
142+
_qualifiedName = qualifiedName;
145143
_parentDeclaration = parentDeclaration;
146144
_parentScopeDeclaration = _parentDeclaration;
147145
_parentScope = parentScope ?? string.Empty;
@@ -158,7 +156,7 @@ public Declaration(
158156
_attributes = attributes ?? new Attributes();
159157

160158
_projectId = _qualifiedName.QualifiedModuleName.ProjectId;
161-
var projectDeclaration = Declaration.GetProjectParent(parentDeclaration);
159+
var projectDeclaration = GetProjectParent(parentDeclaration);
162160
if (projectDeclaration != null)
163161
{
164162
_projectName = projectDeclaration.IdentifierName;
@@ -406,14 +404,13 @@ public void AddReference(
406404

407405
public QualifiedSelection QualifiedSelection { get { return new QualifiedSelection(_qualifiedName.QualifiedModuleName, _selection); } }
408406

409-
private readonly IVBProject _project;
410407
/// <summary>
411408
/// Gets a reference to the VBProject the declaration is made in.
412409
/// </summary>
413410
/// <remarks>
414411
/// This property is intended to differenciate identically-named VBProjects.
415412
/// </remarks>
416-
public virtual IVBProject Project { get { return _project ?? _parentDeclaration.Project; } }
413+
public virtual IVBProject Project { get { return _parentDeclaration.Project; } }
417414

418415
private readonly string _projectId;
419416
/// <summary>

‎Rubberduck.Parsing/Symbols/DeclarationFinder.cs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -619,7 +619,7 @@ private IEnumerable<Declaration> FindAllInReferencedProjectByPriority(Declaratio
619619
}
620620
}
621621

622-
public static ConcurrentBag<Declaration> FindBuiltInEventHandlers(IEnumerable<Declaration> declarations)
622+
public ConcurrentBag<Declaration> FindBuiltInEventHandlers(IEnumerable<Declaration> declarations)
623623
{
624624
var declarationList = declarations.ToList();
625625

@@ -632,18 +632,15 @@ public static ConcurrentBag<Declaration> FindBuiltInEventHandlers(IEnumerable<De
632632
: new[] { e.ParentDeclaration.IdentifierName + "_" + e.IdentifierName };
633633
});
634634

635-
var user = declarationList.FirstOrDefault(decl => !decl.IsBuiltIn);
636-
var host = user != null ? user.Project.VBE.HostApplication() : null;
637-
638635
var handlers = declarationList.Where(item =>
639636
// class module built-in events
640637
(item.DeclarationType == DeclarationType.Procedure &&
641638
item.ParentDeclaration.DeclarationType == DeclarationType.ClassModule && (
642639
item.IdentifierName.Equals("Class_Initialize", StringComparison.InvariantCultureIgnoreCase) ||
643640
item.IdentifierName.Equals("Class_Terminate", StringComparison.InvariantCultureIgnoreCase))) ||
644641
// standard module built-in handlers (Excel specific):
645-
(host != null &&
646-
host.ApplicationName.Equals("Excel", StringComparison.InvariantCultureIgnoreCase) &&
642+
(_hostApp != null &&
643+
_hostApp.ApplicationName.Equals("Excel", StringComparison.InvariantCultureIgnoreCase) &&
647644
item.DeclarationType == DeclarationType.Procedure &&
648645
item.ParentDeclaration.DeclarationType == DeclarationType.ProceduralModule && (
649646
item.IdentifierName.Equals("auto_open", StringComparison.InvariantCultureIgnoreCase) ||

0 commit comments

Comments
 (0)