opendev/gerrit
Code Issues Proposed changes

Skip unnecessary Change.Id parsing from Refs

Browse Source 
No need to parse Ref.getName()->ChangeId in VisibleRefFilter
if we are not asking for any change-refs.

Change-Id: I1e0604f3e80fadc966400a732df1fc0fe71df4d9
This commit is contained in:
Gustaf Lundh
2013-01-30 14:07:14 +01:00
parent a0610125bd
commit 96b4ad1814
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/git/VisibleRefFilter.java
View File

@@ -79,7 +79,7 @@ public class VisibleRefFilter extends AbstractAdvertiseRefsHook {
if (PatchSet.isRef(ref.getName())) { if (PatchSet.isRef(ref.getName())) {
// Reference to a patch set is visible if the change is visible. // Reference to a patch set is visible if the change is visible.
// //
if (visibleChanges.contains(Change.Id.fromRef(ref.getName()))) { if (showChanges && visibleChanges.contains(Change.Id.fromRef(ref.getName()))) {
result.put(ref.getName(), ref); result.put(ref.getName(), ref);
} }