opendev/gerrit
Code Issues Proposed changes

Use Providers.of() instead of hand-rolled Provider

Browse Source 
Change-Id: I6073fb02b1a5eda7f5889b0a27d4aaa3821cbcb7
This commit is contained in:
Shawn O. Pearce
2012-05-02 10:50:16 -07:00
parent 9a3d9a1362
commit c65ff855af
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-server/src/main/java/gerrit/PRED_current_user_2.java
View File

@@ -24,6 +24,7 @@ import com.google.gerrit.server.AnonymousUser;
import com.google.gerrit.server.CurrentUser;
import com.google.gerrit.server.IdentifiedUser;
import com.google.inject.Provider;
import com.google.inject.util.Providers;
import com.googlecode.prolog_cafe.lang.HashtableOfTerm;
import com.googlecode.prolog_cafe.lang.IllegalTypeException;
@@ -103,11 +104,7 @@ class PRED_current_user_2 extends Predicate.P2 {
final ReviewDb db = StoredValues.REVIEW_DB.getOrNull(engine);
IdentifiedUser.GenericFactory userFactory = userFactory(engine);
if (db != null) {
user = userFactory.create(new Provider<ReviewDb>() {
public ReviewDb get() {
return db;
}
}, accountId);
user = userFactory.create(Providers.of(db), accountId);
} else {
user = userFactory.create(accountId);
}