eeva/jamulus
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Catch a CGenErr by const reference rather than by value

Browse source 
This fixes alerts from LGTM.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
This commit is contained in:
Stefan Weil 2020-06-24 08:39:27 +02:00
parent 86de570dbd
commit 172593f7b9
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/clientdlg.cpp
View file

@ -1079,7 +1079,7 @@ void CClientDlg::Connect ( const QString& strSelectedAddress,
} }
} }
catch ( CGenErr generr ) catch ( const CGenErr& generr )
{ {
// show error message and return the function // show error message and return the function
QMessageBox::critical ( this, APP_NAME, generr.GetErrorText(), "Close", nullptr ); QMessageBox::critical ( this, APP_NAME, generr.GetErrorText(), "Close", nullptr );

2
src/main.cpp
View file

@ -706,7 +706,7 @@ int main ( int argc, char** argv )
} }
} }
catch ( CGenErr generr ) catch ( const CGenErr& generr )
{ {
// show generic error // show generic error
#ifndef HEADLESS #ifndef HEADLESS