Kaydet (Commit) 4450b2a1 authored tarafından Noel Grandin's avatar Noel Grandin

pass area param to DBG_UNHANDLED_EXCEPTION

and update sallogareas plugin to enforce this

Change-Id: Id0782c8a1f619372e10d931aec3c6a4743a4c86a
Reviewed-on: https://gerrit.libreoffice.org/52249Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst d7628186
......@@ -1187,7 +1187,7 @@ void DialogWindow::StoreData()
}
catch (const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
MarkDocumentModified( GetDocument() );
m_pEditor->ClearModifyFlag();
......
......@@ -84,7 +84,7 @@ VclPtr<DialogWindow> Shell::CreateDlgWin( const ScriptDocument& rDocument, const
}
catch (const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
else
......
......@@ -692,7 +692,7 @@ void Shell::UpdateWindows()
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -723,7 +723,7 @@ void Shell::UpdateWindows()
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......
......@@ -339,7 +339,7 @@ void TreeListBox::ImpCreateLibSubEntries( SvTreeListEntry* pLibRootEntry, const
}
catch ( const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......@@ -374,7 +374,7 @@ void TreeListBox::ImpCreateLibSubEntries( SvTreeListEntry* pLibRootEntry, const
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......@@ -493,7 +493,7 @@ void TreeListBox::ImpCreateLibSubSubEntriesInVBAMode( SvTreeListEntry* pLibSubRo
}
catch ( const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -761,7 +761,7 @@ void TreeListBox::GetRootEntryBitmaps( const ScriptDocument& rDocument, Image& r
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
if ( !sFactoryURL.isEmpty() )
......
......@@ -209,7 +209,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......
......@@ -97,7 +97,7 @@ namespace basctl { namespace docs {
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -143,7 +143,7 @@ namespace basctl { namespace docs {
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......@@ -163,7 +163,7 @@ namespace basctl { namespace docs {
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......
......@@ -337,7 +337,7 @@ bool CheckBox::EditedEntry( SvTreeListEntry* pEntry, const OUString& rNewName )
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
return false;
}
}
......@@ -1567,7 +1567,7 @@ void createLibImpl(weld::Window* pWin, const ScriptDocument& rDocument,
}
catch (const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......
......@@ -399,7 +399,7 @@ TriState ExtTreeListBox::NotifyCopyingMoving( SvTreeListEntry* pTarget, SvTreeLi
}
catch (const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
else // copy
......@@ -434,7 +434,7 @@ TriState ExtTreeListBox::NotifyCopyingMoving( SvTreeListEntry* pTarget, SvTreeLi
}
catch ( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -913,7 +913,7 @@ void ObjectPage::DeleteCurrent()
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......@@ -1049,7 +1049,7 @@ SbModule* createModImpl(weld::Window* pWin, const ScriptDocument& rDocument,
}
catch (const container::NoSuchElementException& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return pModule;
......
......@@ -144,7 +144,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -319,7 +319,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
m_bValid = false;
}
......@@ -352,7 +352,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return xContainer;
}
......@@ -373,7 +373,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return bIsReadOnly;
......@@ -449,7 +449,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return xContainer;
......@@ -466,7 +466,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return bHas;
}
......@@ -488,7 +488,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return xLibrary;
}
......@@ -504,7 +504,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -528,7 +528,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return false;
......@@ -549,7 +549,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -573,7 +573,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -646,7 +646,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -679,7 +679,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
return false;
}
......@@ -700,7 +700,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -718,7 +718,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return false;
}
......@@ -755,7 +755,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return _out_rDialogProvider.is();
......@@ -773,7 +773,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
}
......@@ -791,7 +791,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return bIsModified;
......@@ -829,7 +829,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
......@@ -860,7 +860,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return sURL;
......@@ -879,7 +879,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
}
return bAllow;
......@@ -901,7 +901,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return _out_rxFrame.is();
......@@ -957,7 +957,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return bIsShared;
......@@ -1141,7 +1141,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
// sort document list by doc title?
......@@ -1225,7 +1225,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
// sort
......@@ -1505,7 +1505,7 @@ namespace basctl
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl.basicide");
}
return bIsActive;
}
......
......@@ -219,7 +219,7 @@ void PropBrw::ImplDestroyController()
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl");
}
m_xBrowserController.clear();
......@@ -515,7 +515,7 @@ void PropBrw::ImplUpdate( const Reference< XModel >& _rxContextDocument, SdrView
catch ( const PropertyVetoException& ) { /* silence */ }
catch ( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basctl");
}
}
......
......@@ -402,7 +402,7 @@ namespace basic
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
}
}
......@@ -513,7 +513,7 @@ namespace basic
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
return false;
}
return true;
......@@ -533,7 +533,7 @@ namespace basic
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
}
return _out_rxBasicLibraries.is() && _out_rxDialogLibraries.is();
}
......
......@@ -1460,7 +1460,7 @@ bool BasicManager::LegacyPsswdBinaryLimitExceeded( std::vector< OUString >& _out
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
}
return false;
}
......
......@@ -1826,7 +1826,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
return;
}
......@@ -1841,7 +1841,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
return;
}
}
......@@ -1917,7 +1917,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
// TODO: error handling?
}
}
......@@ -1972,7 +1972,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto
}
catch(const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
// TODO: error handling
throw;
}
......@@ -2032,7 +2032,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
throw;
}
}
......@@ -3096,7 +3096,7 @@ void SfxLibrary::impl_removeWithoutChecks( const OUString& _rElementName )
}
catch(const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("basic");
}
}
}
......
......@@ -1446,7 +1446,7 @@ IMPL_LINK( ChartController, NotifyUndoActionHdl, SdrUndoAction*, pUndoAction, vo
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
}
......
......@@ -68,7 +68,7 @@ namespace chart
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
return xResult;
}
......@@ -99,7 +99,7 @@ namespace chart
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
......@@ -121,7 +121,7 @@ namespace chart
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
m_xModelClone.clear();
m_xDataClone.clear();
......@@ -150,7 +150,7 @@ namespace chart
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
}
......@@ -238,7 +238,7 @@ namespace chart
}
catch( const Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
......
......@@ -100,7 +100,7 @@ void SAL_CALL UndoCommandDispatch::dispatch(
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
// \--
}
......
......@@ -61,7 +61,7 @@ void UndoGuard::commit()
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
m_bActionPosted = true;
......@@ -126,7 +126,7 @@ HiddenUndoContext::HiddenUndoContext( const Reference< document::XUndoManager >
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
m_xUndoManager.clear();
// prevents the leaveUndoContext in the dtor
}
......@@ -141,7 +141,7 @@ HiddenUndoContext::~HiddenUndoContext()
}
catch( const uno::Exception& )
{
DBG_UNHANDLED_EXCEPTION();
DBG_UNHANDLED_EXCEPTION("chart2");
}
}
......
......@@ -93,8 +93,6 @@ bool SalLogAreas::VisitCallExpr( const CallExpr* call )
area->getLocStart());
return true;
}
if( kind == LogCallKind::DbgUnhandledException ) // below checks don't apply
return true;
if( loplugin::DeclCheck(inFunction).Function("log").Namespace("detail").Namespace("sal").GlobalNamespace()
|| loplugin::DeclCheck(inFunction).Function("sal_detail_logFormat").GlobalNamespace() )