Skip to content
Snippets Groups Projects
Commit b7b48e81 authored by Shaun Roe's avatar Shaun Roe Committed by Graeme Stewart
Browse files

fix coverity 17136 (VP1Gui-00-08-14)

	* fix coverity 17136
17136 09/07/2014 (Medium) Uninitialized pointer field :/graphics/VP1/VP1Gui/src/VP1Gui.cxx in function "Imp"
	* tag as VP1Gui-00-08-14
2016-09-09 Shaun Roe
  * Fix Coverity:
  16179 09/07/2014 (Medium) Function address comparison :/graphics/VP1/VP1Gui/src/VP1EventDisplayEditor.cxx in function "getEventDetailsLabel"
16915 09/07/2014 (Medium) Uninitialized pointer field :/graphics/VP1/VP1Gui/src/VP1EventDisplaySceneView.cxx in function "VP1EventDisplaySceneView"
17114 09/07/2014 (Medium) Uninitialized pointer field :/graphics/VP1/VP1Gui/src/VP1IncomingMessageDialog.cxx in function "Imp"
17003 09/07/2014 (Medium) Uninitialized scalar field :/graphics/VP1/VP1Gui/src/VP1TabManager.cxx in function "Imp"
16381 09/07/2014 (Medium) Unchecked dynamic_cast :/graphics/VP1/VP1Gui/src/VP1TabManager.cxx in function "launchStereoEditorCurrentTab"
  *tag as VP1Gui-00-08-13
parent c9e455f7
No related branches found
No related tags found
No related merge requests found
...@@ -52,7 +52,8 @@ public: ...@@ -52,7 +52,8 @@ public:
} }
EventID & operator= (const EventID & o ) EventID & operator= (const EventID & o )
{ {
if (&o == this) return *this;
eventNumber = o.eventNumber; eventNumber = o.eventNumber;
runNumber = o.runNumber; runNumber = o.runNumber;
return *this; return *this;
......
...@@ -371,7 +371,7 @@ QString VP1EventDisplayEditor::getEventDetailsLabel() ...@@ -371,7 +371,7 @@ QString VP1EventDisplayEditor::getEventDetailsLabel()
QString evtstr = "Run: "+QString::number(_runNumber)+"\n" QString evtstr = "Run: "+QString::number(_runNumber)+"\n"
+ "Event: "+QString::number(_eventNumber)+"\n" + "Event: "+QString::number(_eventNumber)+"\n"
+ QString(time>0 ? QDateTime::fromTime_t(_eventTimestamp).toString(Qt::ISODate).replace('T',' ')+ " CEST" : ""); + QString(QDateTime::fromTime_t(_eventTimestamp).toString(Qt::ISODate).replace('T',' ')+ " CEST" );
//VP1Msg::messageVerbose(evtstr); //VP1Msg::messageVerbose(evtstr);
......
...@@ -37,9 +37,10 @@ void GraphicsView::wheelEvent(QWheelEvent *e) ...@@ -37,9 +37,10 @@ void GraphicsView::wheelEvent(QWheelEvent *e)
} }
VP1EventDisplaySceneView::VP1EventDisplaySceneView(const QString &name, bool preview, QWidget *parent) VP1EventDisplaySceneView::VP1EventDisplaySceneView(const QString &name, bool preview, QWidget *parent)
: QFrame(parent) : QFrame(parent),_frame(nullptr),_line(nullptr)
{ {
_preview = preview; _preview = preview;
setFrameStyle(Sunken | StyledPanel); setFrameStyle(Sunken | StyledPanel);
......
...@@ -25,14 +25,20 @@ ...@@ -25,14 +25,20 @@
//____________________________________________________________________ //____________________________________________________________________
class VP1Gui::Imp { class VP1Gui::Imp {
public: public:
Imp() : the_scheduler(0) {} Imp() : the_scheduler(nullptr), sg(nullptr), detstore(nullptr),
svclocator(nullptr), toolSvc(nullptr),
initialvp1files{},
initialCruiseSeconds{},
localFileCacheLimit{}{
//nop
}
VP1ExecutionScheduler * the_scheduler; VP1ExecutionScheduler * the_scheduler;
//We hold the arguments here until init is called: //We hold the arguments here until init is called:
StoreGateSvc* sg; StoreGateSvc * sg;
StoreGateSvc* detstore; StoreGateSvc * detstore;
ISvcLocator* svclocator; ISvcLocator * svclocator;
IToolSvc*toolSvc; IToolSvc * toolSvc;
QStringList initialvp1files; QStringList initialvp1files;
QString initialCruiseMode; QString initialCruiseMode;
unsigned initialCruiseSeconds; unsigned initialCruiseSeconds;
......
...@@ -23,7 +23,13 @@ ...@@ -23,7 +23,13 @@
//____________________________________________________________________ //____________________________________________________________________
class VP1IncomingMessageDialog::Imp { class VP1IncomingMessageDialog::Imp {
public: public:
Imp(const VP1ExternalRequest& r) : request(r) {} Imp(const VP1ExternalRequest& r) : tabmanager{}, channelmanager{}, requestqueue{},request(r),
lcdNumber_pending{}, lcdNumber_pendingsender{}, lcdNumber_pendingexactsame{},
blockallmessages{}, pushButton_allclear{}, pushButton_senderclear{},
pushButton_messageclear{}, messages_blockedsenders{}, messages_blockedexactmessages{}
{
//nop
}
VP1TabManager* tabmanager; VP1TabManager* tabmanager;
VP1ChannelManager* channelmanager; VP1ChannelManager* channelmanager;
QQueue<VP1ExternalRequest>* requestqueue; QQueue<VP1ExternalRequest>* requestqueue;
......
...@@ -53,7 +53,9 @@ class VP1TabManager::Imp { ...@@ -53,7 +53,9 @@ class VP1TabManager::Imp {
public: public:
//Constructor, tabwidget & channel manager: //Constructor, tabwidget & channel manager:
Imp(VP1TabManager*tm,VP1TabWidget* tw, VP1ChannelManager* cm) Imp(VP1TabManager*tm,VP1TabWidget* tw, VP1ChannelManager* cm)
: tabmanager(tm), tabwidget(tw),channelmanager(cm),selecteddockwidget(0), : tabmanager(tm), tabwidget(tw),channelmanager(cm),
dontEmitVisibilityChanges{},
selecteddockwidget(0),
fullscreen_dockwidget(0),fullscreen_channelwidget(0), fullscreen_dockwidget(0),fullscreen_channelwidget(0),
fullscreen_tabindex(-1),fullscreen_tab(0),tabcruisemode(false) {} fullscreen_tabindex(-1),fullscreen_tab(0),tabcruisemode(false) {}
...@@ -130,12 +132,10 @@ VP1TabManager::~VP1TabManager(){ ...@@ -130,12 +132,10 @@ VP1TabManager::~VP1TabManager(){
//___________________________________________________________________________________ //___________________________________________________________________________________
void VP1TabManager::launchStereoEditorCurrentTab() void VP1TabManager::launchStereoEditorCurrentTab(){
{
VP1Msg::messageVerbose("VP1TabManager::launchStereoEditorCurrentTab()"); VP1Msg::messageVerbose("VP1TabManager::launchStereoEditorCurrentTab()");
IVP13DStandardChannelWidget* channel = dynamic_cast<IVP13DStandardChannelWidget*>(selectedChannelWidget());
if (dynamic_cast<IVP13DStandardChannelWidget*>(selectedChannelWidget()) ) { if (channel ) {
IVP13DStandardChannelWidget* channel = dynamic_cast<IVP13DStandardChannelWidget*>(selectedChannelWidget());
channel->launchStereoEditor(); channel->launchStereoEditor();
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment