Skip to content

Commit ef1ad16

Browse files
authored
Merge pull request #180 from comintern/next
Finish up changes for serialization post processing
2 parents e8ba77b + 42987f7 commit ef1ad16

File tree

2 files changed

+1
-22
lines changed

2 files changed

+1
-22
lines changed

Rubberduck.Parsing/Symbols/ComInformation.cs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@ public ComInformation(TYPEATTR typeAttributes, IMPLTYPEFLAGS implTypeFlags, ITyp
2626
public Declaration ModuleDeclaration { get; internal set; }
2727
public DeclarationType TypeDeclarationType { get; internal set; }
2828

29-
private readonly List<SerializableDeclarationTree> _trees = new List<SerializableDeclarationTree>();
30-
internal IList<SerializableDeclarationTree> MemberTrees { get { return _trees; } }
31-
3229
public override string ToString()
3330
{
3431
return ModuleDeclaration.IdentifierName;

Rubberduck.Parsing/Symbols/ReferencedDeclarationsCollector.cs

Lines changed: 1 addition & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ private string GetTypeName(ITypeInfo info)
146146
return typeName.Equals("LONG_PTR") ? "LongPtr" : typeName; //Quickfix for http://chat.stackexchange.com/transcript/message/30119269#30119269
147147
}
148148

149-
public List<Declaration> GetDeclarationsForReference(IReference reference, out SerializableDeclarationTree tree)
149+
public List<Declaration> GetDeclarationsForReference(IReference reference)
150150
{
151151
var output = new List<Declaration>();
152152
var projectName = reference.Name;
@@ -156,14 +156,12 @@ public List<Declaration> GetDeclarationsForReference(IReference reference, out S
156156
LoadTypeLibEx(path, REGKIND.REGKIND_NONE, out typeLibrary);
157157
if (typeLibrary == null)
158158
{
159-
tree = null;
160159
return output;
161160
}
162161
var projectQualifiedModuleName = new QualifiedModuleName(projectName, path, projectName);
163162
var projectQualifiedMemberName = new QualifiedMemberName(projectQualifiedModuleName, projectName);
164163
var projectDeclaration = new ProjectDeclaration(projectQualifiedMemberName, projectName, isBuiltIn: true);
165164
output.Add(projectDeclaration);
166-
var moduleTrees = new List<SerializableDeclarationTree>();
167165

168166
var typeCount = typeLibrary.GetTypeInfoCount();
169167
for (var i = 0; i < typeCount; i++)
@@ -175,7 +173,6 @@ public List<Declaration> GetDeclarationsForReference(IReference reference, out S
175173
}
176174
catch (NullReferenceException)
177175
{
178-
tree = null;
179176
return output;
180177
}
181178

@@ -290,26 +287,16 @@ public List<Declaration> GetDeclarationsForReference(IReference reference, out S
290287
info.ReleaseTypeAttr(typeAttributesPointer);
291288

292289
output.Add(moduleDeclaration);
293-
var moduleTree = new SerializableDeclarationTree(new SerializableDeclaration(moduleDeclaration), comInfo.MemberTrees);
294-
moduleTrees.Add(moduleTree);
295290
}
296291

297292
foreach (var member in _comInformation.Values)
298293
{
299294
LoadDeclarationsInModule(output, member);
300295
}
301296

302-
tree = new SerializableDeclarationTree(new SerializableDeclaration(projectDeclaration), moduleTrees);
303297
return output;
304298
}
305299

306-
[Obsolete("Use the overload that outputs a SerializableDeclarationTree instead.")]
307-
public List<Declaration> GetDeclarationsForReference(IReference reference)
308-
{
309-
SerializableDeclarationTree tree;
310-
return GetDeclarationsForReference(reference, out tree);
311-
}
312-
313300
private void LoadDeclarationsInModule(List<Declaration> output, ComInformation member)
314301
{
315302
if (member.TypeAttributes.typekind == TYPEKIND.TKIND_COCLASS)
@@ -360,17 +347,12 @@ memberDeclaration is ICanBeDefaultMember &&
360347
{
361348
parameters.Last().IsParamArray = true;
362349
}
363-
364-
var parameterTrees = parameters.Select(p => new SerializableDeclarationTree(p));
365-
var tree = new SerializableDeclarationTree(new SerializableDeclaration(memberDeclaration), parameterTrees);
366-
member.MemberTrees.Add(tree);
367350
}
368351

369352
for (var fieldIndex = 0; fieldIndex < member.TypeAttributes.cVars; fieldIndex++)
370353
{
371354
var declaration = CreateFieldDeclaration(member.TypeInfo, fieldIndex, member.TypeDeclarationType, member.TypeQualifiedModuleName, member.ModuleDeclaration);
372355
output.Add(declaration);
373-
member.MemberTrees.Add(new SerializableDeclarationTree(declaration));
374356
}
375357
}
376358

0 commit comments

Comments
 (0)