Skip to content

Commit

Permalink
Fix a bug introduced by the previous optimization where properties wo…
Browse files Browse the repository at this point in the history
…uld be excluded from search because their "search field" was synthesized and as such not found in the string table.

Now we add two fields for names and values, each of which should be found in the string table.
  • Loading branch information
KirillOsenkov committed May 24, 2016
1 parent 20a3fd9 commit 4902427
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/StructuredLogViewer/Search.cs
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,8 @@ public static void PopulateSearchFields(object node, Action<string> addSearchFie
var nameValueNode = node as NameValueNode;
if (nameValueNode != null)
{
addSearchField(nameValueNode.Name + " = " + nameValueNode.Value);
addSearchField(nameValueNode.Name);
addSearchField(nameValueNode.Value);
}

var diagnostic = node as AbstractDiagnostic;
Expand Down

0 comments on commit 4902427

Please sign in to comment.