mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Nonsensical OUString null check

Browse Source 
...ever since at least d32b3a714f "#i106421#: move
svx/source/cui to cui".  (That happened to redundantly check for non-empty url,
too; maybe in the distant past GetScriptURL returned a pointer.)

Change-Id: I6139db1d4b1fdcf5325895569de293dd89e36d9f
This commit is contained in:
Stephan Bergmann
2016-10-31 12:45:32 +01:00
parent f2de7d0545
commit d6b9fea9b8
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cui/source/customize/selector.cxx
View File

@@ -993,7 +993,7 @@ void
SvxScriptSelectorDialog::UpdateUI() SvxScriptSelectorDialog::UpdateUI()
{ {
OUString url = GetScriptURL(); OUString url = GetScriptURL();
if ( url != nullptr && !url.isEmpty() ) if ( !url.isEmpty() )
{ {
OUString sMessage = OUString sMessage =
m_pCommands->GetHelpText(m_pCommands->FirstSelected()); m_pCommands->GetHelpText(m_pCommands->FirstSelected());