# HG changeset patch # User cin # Date 1472715811 -10800 # Node ID 86187b01c4e0351502b97624dce3b57bafe6856e # Parent 40d7fed4a09eea66f914d5407ad4d0b7956ca6b9 fixed: the error handler should not handle handlers errors diff -r 40d7fed4a09e -r 86187b01c4e0 Implab/AbstractPromise.cs --- a/Implab/AbstractPromise.cs Mon Aug 29 23:15:51 2016 +0300 +++ b/Implab/AbstractPromise.cs Thu Sep 01 10:43:31 2016 +0300 @@ -27,10 +27,8 @@ if ((m_mask & PromiseEventType.Success) != 0 && m_handler != null) { try { m_handler(); - } catch (Exception err) { - // avoid calling handler twice in case of error - if (m_error != null) - SignalError(err); + // Analysis disable once EmptyGeneralCatchClause + } catch { } } } @@ -55,8 +53,8 @@ if (m_cancel != null) { try { m_cancel(reason); - } catch (Exception err) { - SignalError(err); + // Analysis disable once EmptyGeneralCatchClause + } catch { } } else if ( (m_mask & PromiseEventType.Cancelled) != 0 && m_handler != null) { try { diff -r 40d7fed4a09e -r 86187b01c4e0 Implab/AbstractPromiseT.cs --- a/Implab/AbstractPromiseT.cs Mon Aug 29 23:15:51 2016 +0300 +++ b/Implab/AbstractPromiseT.cs Thu Sep 01 10:43:31 2016 +0300 @@ -39,16 +39,14 @@ if (m_success != null) { try { m_success(result); - } catch(Exception err) { - SignalError(err); + // Analysis disable once EmptyGeneralCatchClause + } catch { } } else if ((m_mask & PromiseEventType.Success) != 0 && m_handler != null) { try { m_handler(); - } catch(Exception err) { - // avoid calling handler twice in case of error - if (m_error != null) - SignalError(err); + // Analysis disable once EmptyGeneralCatchClause + } catch { } } } @@ -73,8 +71,8 @@ if (m_cancel != null) { try { m_cancel(reason); - } catch (Exception err) { - SignalError(err); + // Analysis disable once EmptyGeneralCatchClause + } catch { } } else if ((m_mask & PromiseEventType.Cancelled) != 0 && m_handler != null) { try {