Merge pull request #357 from Jimilian/clean_running_job_from_stupid_m…

…istakes

Clean runningJob class from stupid mistakes

src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger.java

@@ -2243,8 +2243,8 @@ public void scheduled(ChangeBasedEvent event) {
             }
 
             List<ChangeBasedEvent> outdatedEvents = new ArrayList<ChangeBasedEvent>();
-            Iterator<GerritTriggeredEvent> it = runningJobs.iterator();
             synchronized (runningJobs) {
+                Iterator<GerritTriggeredEvent> it = runningJobs.iterator();
                 while (it.hasNext()) {
                     GerritTriggeredEvent runningEvent = it.next();
                     // Find all entries in runningJobs with the same Change #.
@@ -2395,7 +2395,7 @@ private boolean checkCausedByGerrit(GerritTriggeredEvent event, Collection<Cause
          */
         public boolean remove(ChangeBasedEvent event) {
             logger.debug("Removing future job " + event.getPatchSet().getNumber());
-            return runningJobs.add(event);
+            return runningJobs.remove(event);
         }
     }