Skip to content

Commit

Permalink
Call finishProcessing in the right place
Browse files Browse the repository at this point in the history
We should call `InterpreterListener.finishProcessing()` only once, but
before that was not necessarily true, as it was being called inside a
loop.

Instead, now we call it in the same method as `startProcessing()`, after
all processing has been finished.

Fixes #51
  • Loading branch information
Edmundo Alvarez committed Jul 20, 2016
1 parent 912802a commit cf55dcb
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -265,6 +265,8 @@ public Messages process(Messages messages, InterpreterListener interpreterListen
}
}
}

interpreterListener.finishProcessing();
// 7. return the processed messages
return new MessageCollection(fullyProcessed);
}
Expand Down Expand Up @@ -374,7 +376,6 @@ public List<Message> processForPipelines(Message message, String msgId, Set<Stri
}
}

interpreterListener.finishProcessing();
// 7. return the processed messages
return result;
}
Expand Down

0 comments on commit cf55dcb

Please sign in to comment.