Skip to content

Commit

Permalink
Made the signatures of both handleBiddingStateEvent() methods match.
Browse files Browse the repository at this point in the history
  • Loading branch information
jbrains committed May 2, 2011
1 parent 9d234bb commit ef328d0
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions system-test/source/ca/jbrains/auction/test/Main.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,13 @@ public void processMessage(Chat chat, Message message) {

public static final class BidsForSniperMessageListener extends
FooMessageListener {

private final Main main;

public BidsForSniperMessageListener(Main main) {
this.main = main;
}

// SMELL This nested class makes cyclic dependencies too
// easy
@Override
public void processMessage(Chat chat, Message message) {
final Object event = Messages.parse(message);
Expand Down Expand Up @@ -59,10 +57,10 @@ public UpdatesMainWindowMessageListener(Main main) {

@Override
public void processMessage(Chat chat, Message message) {
Object event = Messages.parse(message);
final Object event = Messages.parse(message);
if (event instanceof BiddingState) {
BiddingState biddingState = (BiddingState) event;
handleBiddingStateEvent(biddingState);
handleBiddingStateEvent(chat, biddingState);
} else {
handleAllOtherEvents();
}
Expand All @@ -72,7 +70,8 @@ private void handleAllOtherEvents() {
main.signalAuctionClosed();
}

private void handleBiddingStateEvent(BiddingState biddingState) {
private void handleBiddingStateEvent(
@SuppressWarnings("unused") Chat chat, BiddingState biddingState) {
if (!Main.SNIPER_XMPP_ID.equals(biddingState.getBidderName())) {
main.signalSniperIsBidding();
}
Expand Down

0 comments on commit ef328d0

Please sign in to comment.