Skip to content

Commit

Permalink
Undeprecate a few functions that aren't going away
Browse files Browse the repository at this point in the history
Change-Id: Ifd0199102a8316a9439f3ad931e6def9c8c4cc85
  • Loading branch information
nasserg authored and spearce committed May 19, 2011
1 parent 3755e9f commit e389b9a
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ protected BaseServiceImplementation(final Provider<ReviewDb> schema,
this.currentUser = currentUser;
}

@Deprecated
protected Account.Id getAccountId() {
CurrentUser u = currentUser.get();
if (u instanceof IdentifiedUser) {
Expand Down Expand Up @@ -97,7 +96,6 @@ protected <T> void run(final AsyncCallback<T> callback, final Action<T> action)
}

/** Exception whose cause is passed into onFailure. */
@Deprecated
public static class Failure extends Exception {
private static final long serialVersionUID = 1L;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -370,13 +370,11 @@ public Change(final Change.Key newKey, final Change.Id newId,
}

/** Legacy 32 bit integer identity for a change. */
@Deprecated
public Change.Id getId() {
return changeId;
}

/** Legacy 32 bit integer identity for a change. */
@Deprecated
public int getChangeId() {
return changeId.get();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ public boolean isBatchUser() {
return getEffectiveGroups().contains(authConfig.getBatchUsersGroup());
}

@Deprecated
public final boolean isAdministrator() {
return getEffectiveGroups().contains(authConfig.getAdministratorsGroup());
}
Expand Down

0 comments on commit e389b9a

Please sign in to comment.