Remove unnecessary synchronization from Case.addReport()

This commit is contained in:
Richard Cordovano 2014-06-06 11:56:14 -04:00
parent 54e1b2a2fc
commit e082584959

View File

@ -1182,13 +1182,11 @@ public class Case implements SleuthkitCase.ErrorObserver {
*/ */
public void addReport(String localPath, String srcModuleName, String reportName) throws TskCoreException { public void addReport(String localPath, String srcModuleName, String reportName) throws TskCoreException {
Report report = this.db.addReport(localPath, srcModuleName, reportName); Report report = this.db.addReport(localPath, srcModuleName, reportName);
synchronized (this) { try {
try { Case.pcs.firePropertyChange(Events.REPORT_ADDED.toString(), null, report);
Case.pcs.firePropertyChange(Events.REPORT_ADDED.toString(), null, report); } catch (Exception ex) {
} catch (Exception ex) { String errorMessage = String.format("A Case %s listener threw an exception", Events.REPORT_ADDED.toString());
String errorMessage = String.format("A Case %s listener threw an exception", Events.REPORT_ADDED.toString()); logger.log(Level.SEVERE, errorMessage, ex);
logger.log(Level.SEVERE, errorMessage, ex);
}
} }
} }