Changeset View
Changeset View
Standalone View
Standalone View
plugins/standardoutputview/outputwidget.cpp
| Show First 20 Lines • Show All 228 Lines • ▼ Show 20 Line(s) | 224 | { | |||
|---|---|---|---|---|---|
| 229 | { | 229 | { | ||
| 230 | m_stackwidget->setCurrentWidget( view ); | 230 | m_stackwidget->setCurrentWidget( view ); | ||
| 231 | } | 231 | } | ||
| 232 | } | 232 | } | ||
| 233 | 233 | | |||
| 234 | void OutputWidget::changeDelegate( int id ) | 234 | void OutputWidget::changeDelegate( int id ) | ||
| 235 | { | 235 | { | ||
| 236 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) { | 236 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) { | ||
| 237 | m_views.value(id)->setItemDelegate(data->outputdata.value(id)->delegate); | 237 | m_views.value(id).view->setItemDelegate(data->outputdata.value(id)->delegate); | ||
| 238 | } else { | 238 | } else { | ||
| 239 | addOutput(id); | 239 | addOutput(id); | ||
| 240 | } | 240 | } | ||
| 241 | } | 241 | } | ||
| 242 | 242 | | |||
| 243 | void OutputWidget::changeModel( int id ) | 243 | void OutputWidget::changeModel( int id ) | ||
| 244 | { | 244 | { | ||
| 245 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) | 245 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) | ||
| 246 | { | 246 | { | ||
| 247 | OutputData* od = data->outputdata.value(id); | 247 | OutputData* od = data->outputdata.value(id); | ||
| 248 | m_views.value( id )->setModel(od->model); | 248 | m_views.value(id).view->setModel(od->model); | ||
| 249 | } | 249 | } | ||
| 250 | else | 250 | else | ||
| 251 | { | 251 | { | ||
| 252 | addOutput( id ); | 252 | addOutput( id ); | ||
| 253 | } | 253 | } | ||
| 254 | } | 254 | } | ||
| 255 | 255 | | |||
| 256 | void OutputWidget::removeOutput( int id ) | 256 | void OutputWidget::removeOutput( int id ) | ||
| 257 | { | 257 | { | ||
| 258 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) | 258 | if( data->outputdata.contains( id ) && m_views.contains( id ) ) | ||
| 259 | { | 259 | { | ||
| 260 | QTreeView* view = m_views.value(id); | 260 | QTreeView *view = m_views.value(id).view; | ||
kossebau: KDevelop coding style is to have the * with the type, so please keep that (`QTreeView*`) | |||||
Got it. I understood it vice versa after reading the wiki. Will do it right way from now on. vkorneev: Got it. I understood it vice versa after reading the wiki. Will do it right way from now on. | |||||
| 261 | if( data->type & KDevelop::IOutputView::MultipleView || data->type & KDevelop::IOutputView::HistoryView ) | 261 | if( data->type & KDevelop::IOutputView::MultipleView || data->type & KDevelop::IOutputView::HistoryView ) | ||
| 262 | { | 262 | { | ||
| 263 | if( data->type & KDevelop::IOutputView::MultipleView ) | 263 | if( data->type & KDevelop::IOutputView::MultipleView ) | ||
| 264 | { | 264 | { | ||
| 265 | int idx = m_tabwidget->indexOf( view ); | 265 | int idx = m_tabwidget->indexOf( view ); | ||
| 266 | if( idx != -1 ) | 266 | if (idx != -1) | ||
| 267 | { | 267 | { | ||
| 268 | m_tabwidget->removeTab( idx ); | 268 | m_tabwidget->removeTab( idx ); | ||
| 269 | if( m_proxyModels.contains( id ) ) | | |||
| 270 | { | | |||
| 271 | delete m_proxyModels.take( id ); | | |||
| 272 | m_filters.remove( id ); | | |||
| 273 | } | | |||
| 274 | } | 269 | } | ||
| 275 | } else | 270 | } else | ||
| 276 | { | 271 | { | ||
| 277 | int idx = m_stackwidget->indexOf( view ); | 272 | int idx = m_stackwidget->indexOf( view ); | ||
| 278 | if( idx != -1 && m_proxyModels.contains( id ) ) | 273 | if (idx != -1) | ||
| 279 | { | 274 | { | ||
| 280 | delete m_proxyModels.take( id ); | | |||
| 281 | m_filters.remove( id ); | | |||
| 282 | } | | |||
| 283 | m_stackwidget->removeWidget( view ); | 275 | m_stackwidget->removeWidget(view); | ||
| 284 | } | 276 | } | ||
| 285 | delete view; | | |||
| 286 | } else | | |||
| 287 | { | | |||
| 288 | m_views.value( id )->setModel( nullptr ); | | |||
| 289 | m_views.value( id )->setItemDelegate( nullptr ); | | |||
| 290 | if( m_proxyModels.contains( 0 ) ) { | | |||
| 291 | delete m_proxyModels.take( 0 ); | | |||
| 292 | m_filters.remove( 0 ); | | |||
| 293 | } | 277 | } | ||
| 278 | m_views[id].destroy(); | ||||
| 279 | m_views.remove(id); | ||||
This can be simplified in one single lookup: auto view = m_views.take(id); view.destroy(); pino: This can be simplified in one single lookup:
lang=c++
auto view = m_views.take(id);
view. | |||||
| 280 | } else { // KDevelop::IOutputView::OneView | ||||
| 281 | /* TODO: this branch of execution has no result because of the "m_views.remove( id );" | ||||
| 282 | * after the if-else block. Need to find out which behavior has sense. | ||||
| 283 | */ | ||||
| 284 | FilteredView& fview = m_views[id]; | ||||
| 285 | fview.view->setModel(nullptr); | ||||
| 286 | fview.view->setItemDelegate(nullptr); | ||||
aaronpuchert: You can also write `fview.proxyModel.reset()`. | |||||
| 287 | if (fview.proxyModel) { | ||||
| 288 | delete fview.proxyModel; | ||||
| 289 | fview.filter = QString(); | ||||
I think this line is redundant. You're doing m_views.remove(id); outside the if-else stmt anyway(?) kfunk: I think this line is redundant. You're doing `m_views.remove(id);` outside the if-else stmt… | |||||
Yes, thanks, it's redundant, it leaked into the code because I was thinking about the issue, described in the TODO comment below. I'll remove it now. vkorneev: Yes, thanks, it's redundant, it leaked into the code because I was thinking about the issue… | |||||
| 294 | } | 290 | } | ||
| 291 | } | ||||
| 292 | m_views[id].destroy(); | ||||
| 295 | m_views.remove( id ); | 293 | m_views.remove( id ); | ||
pino: Ditto. | |||||
| 296 | emit outputRemoved( data->toolViewId, id ); | 294 | emit outputRemoved( data->toolViewId, id ); | ||
| 297 | } | 295 | } | ||
| 298 | enableActions(); | 296 | enableActions(); | ||
| 299 | } | 297 | } | ||
| 300 | 298 | | |||
| 301 | void OutputWidget::closeActiveView() | 299 | void OutputWidget::closeActiveView() | ||
| 302 | { | 300 | { | ||
| 303 | QWidget* widget = m_tabwidget->currentWidget(); | 301 | QWidget* widget = m_tabwidget->currentWidget(); | ||
| 304 | if( !widget ) | 302 | if( !widget ) | ||
| 305 | return; | 303 | return; | ||
| 306 | foreach( int id, m_views.keys() ) | 304 | foreach( int id, m_views.keys() ) | ||
| 307 | { | 305 | { | ||
| 308 | if( m_views.value(id) == widget ) | 306 | if (m_views.value(id).view == widget) | ||
| 309 | { | 307 | { | ||
| 310 | OutputData* od = data->outputdata.value(id); | 308 | OutputData* od = data->outputdata.value(id); | ||
| 311 | if( od->behaviour & KDevelop::IOutputView::AllowUserClose ) | 309 | if( od->behaviour & KDevelop::IOutputView::AllowUserClose ) | ||
| 312 | { | 310 | { | ||
| 313 | data->plugin->removeOutput( id ); | 311 | data->plugin->removeOutput( id ); | ||
| 314 | } | 312 | } | ||
| 315 | } | 313 | } | ||
| 316 | } | 314 | } | ||
| 317 | enableActions(); | 315 | enableActions(); | ||
| 318 | } | 316 | } | ||
| 319 | 317 | | |||
| 320 | void OutputWidget::closeOtherViews() | 318 | void OutputWidget::closeOtherViews() | ||
| 321 | { | 319 | { | ||
| 322 | QWidget* widget = m_tabwidget->currentWidget(); | 320 | QWidget* widget = m_tabwidget->currentWidget(); | ||
| 323 | if (!widget) | 321 | if (!widget) | ||
| 324 | return; | 322 | return; | ||
| 325 | 323 | | |||
| 326 | foreach (int id, m_views.keys()) { | 324 | foreach (int id, m_views.keys()) { | ||
| 327 | if (m_views.value(id) == widget) { | 325 | if (m_views.value(id).view == widget) { | ||
| 328 | continue; // leave the active view open | 326 | continue; // leave the active view open | ||
| 329 | } | 327 | } | ||
| 330 | 328 | | |||
| 331 | OutputData* od = data->outputdata.value(id); | 329 | OutputData* od = data->outputdata.value(id); | ||
| 332 | if (od->behaviour & KDevelop::IOutputView::AllowUserClose) { | 330 | if (od->behaviour & KDevelop::IOutputView::AllowUserClose) { | ||
| 333 | data->plugin->removeOutput( id ); | 331 | data->plugin->removeOutput( id ); | ||
| 334 | } | 332 | } | ||
| 335 | } | 333 | } | ||
| 336 | enableActions(); | 334 | enableActions(); | ||
| 337 | } | 335 | } | ||
| 338 | 336 | | |||
| 339 | QWidget* OutputWidget::currentWidget() const | 337 | QWidget* OutputWidget::currentWidget() const | ||
| 340 | { | 338 | { | ||
| 341 | QWidget* widget; | 339 | QWidget* widget; | ||
| 342 | if( data->type & KDevelop::IOutputView::MultipleView ) | 340 | if( data->type & KDevelop::IOutputView::MultipleView ) | ||
| 343 | { | 341 | { | ||
| 344 | widget = m_tabwidget->currentWidget(); | 342 | widget = m_tabwidget->currentWidget(); | ||
| 345 | } else if( data->type & KDevelop::IOutputView::HistoryView ) | 343 | } else if( data->type & KDevelop::IOutputView::HistoryView ) | ||
| 346 | { | 344 | { | ||
| 347 | widget = m_stackwidget->currentWidget(); | 345 | widget = m_stackwidget->currentWidget(); | ||
| 348 | } else | 346 | } else | ||
| 349 | { | 347 | { | ||
| 350 | widget = m_views.begin().value(); | 348 | widget = m_views.begin().value().view; | ||
| 351 | } | 349 | } | ||
| 352 | return widget; | 350 | return widget; | ||
| 353 | } | 351 | } | ||
| 354 | 352 | | |||
| 355 | KDevelop::IOutputViewModel *OutputWidget::outputViewModel() const | 353 | KDevelop::IOutputViewModel *OutputWidget::outputViewModel() const | ||
| 356 | { | 354 | { | ||
| 357 | auto view = qobject_cast<QAbstractItemView*>(currentWidget()); | 355 | auto view = qobject_cast<QAbstractItemView*>(currentWidget()); | ||
| 358 | if( !view || !view->isVisible()) | 356 | if( !view || !view->isVisible()) | ||
| Show All 26 Lines | |||||
| 385 | } | 383 | } | ||
| 386 | 384 | | |||
| 387 | void OutputWidget::activateIndex(const QModelIndex &index, QAbstractItemView *view, KDevelop::IOutputViewModel *iface) | 385 | void OutputWidget::activateIndex(const QModelIndex &index, QAbstractItemView *view, KDevelop::IOutputViewModel *iface) | ||
| 388 | { | 386 | { | ||
| 389 | if( ! index.isValid() ) | 387 | if( ! index.isValid() ) | ||
| 390 | return; | 388 | return; | ||
| 391 | QModelIndex sourceIndex = index; | 389 | QModelIndex sourceIndex = index; | ||
| 392 | QModelIndex viewIndex = index; | 390 | QModelIndex viewIndex = index; | ||
| 393 | int id = m_views.key(qobject_cast<QTreeView*>(view)); | 391 | auto fvIt = getFilteredView(view); | ||
| 394 | if( QAbstractProxyModel* proxy = m_proxyModels.value(id) ) { | 392 | if (fvIt != m_views.end() && fvIt->proxyModel) { | ||
| 393 | auto proxy = fvIt->proxyModel; | ||||
| 395 | if ( index.model() == proxy ) { | 394 | if ( index.model() == proxy ) { | ||
| 396 | // index is from the proxy, map it to the source | 395 | // index is from the proxy, map it to the source | ||
| 397 | sourceIndex = proxy->mapToSource(index); | 396 | sourceIndex = proxy->mapToSource(index); | ||
| 398 | } else if (proxy == view->model()) { | 397 | } else if (proxy == view->model()) { | ||
| 399 | // index is from the source, map it to the proxy | 398 | // index is from the source, map it to the proxy | ||
| 400 | viewIndex = proxy->mapFromSource(index); | 399 | viewIndex = proxy->mapFromSource(index); | ||
| 401 | } | 400 | } | ||
| 402 | } | 401 | } | ||
| Show All 30 Lines | |||||
| 433 | { | 432 | { | ||
| 434 | auto view = outputView(); | 433 | auto view = outputView(); | ||
| 435 | auto iface = outputViewModel(); | 434 | auto iface = outputViewModel(); | ||
| 436 | if ( ! view || ! iface ) | 435 | if ( ! view || ! iface ) | ||
| 437 | return; | 436 | return; | ||
| 438 | eventuallyDoFocus(); | 437 | eventuallyDoFocus(); | ||
| 439 | 438 | | |||
| 440 | auto index = view->currentIndex(); | 439 | auto index = view->currentIndex(); | ||
| 441 | int id = m_views.key(qobject_cast<QTreeView*>(view)); | 440 | auto fvIt = getFilteredView(view); | ||
| 442 | if (QAbstractProxyModel* proxy = m_proxyModels.value(id)) { | 441 | if (fvIt != m_views.end() && fvIt->proxyModel) { | ||
| 442 | auto proxy = fvIt->proxyModel; | ||||
| 443 | if ( index.model() == proxy ) { | 443 | if ( index.model() == proxy ) { | ||
| 444 | // index is from the proxy, map it to the source | 444 | // index is from the proxy, map it to the source | ||
| 445 | index = proxy->mapToSource(index); | 445 | index = proxy->mapToSource(index); | ||
| 446 | } | 446 | } | ||
| 447 | } | 447 | } | ||
| 448 | 448 | | |||
| 449 | QModelIndex newIndex; | 449 | QModelIndex newIndex; | ||
| 450 | switch (selectionMode) { | 450 | switch (selectionMode) { | ||
| ▲ Show 20 Lines • Show All 67 Lines • ▼ Show 20 Line(s) | 501 | { | |||
| 518 | { | 518 | { | ||
| 519 | if( m_views.isEmpty() ) | 519 | if( m_views.isEmpty() ) | ||
| 520 | { | 520 | { | ||
| 521 | listview = createHelper(); | 521 | listview = createHelper(); | ||
| 522 | 522 | | |||
| 523 | layout()->addWidget( listview ); | 523 | layout()->addWidget( listview ); | ||
| 524 | } else | 524 | } else | ||
| 525 | { | 525 | { | ||
| 526 | listview = m_views.begin().value(); | 526 | listview = m_views.begin().value().view; | ||
| 527 | newView = false; | 527 | newView = false; | ||
| 528 | } | 528 | } | ||
| 529 | } | 529 | } | ||
| 530 | m_views[id] = listview; | 530 | m_views[id].view = listview; | ||
aaronpuchert: And here you can write `m_views[id].view.reset(listview)`. | |||||
It's not correct, 3 line above you get a raw managed pointer, here you make a distinct managed pointer with same source. It'll result in double deletion when goes out of scope. anthonyfieroni: It's not correct, 3 line above you get a raw managed pointer, here you make a distinct managed… | |||||
As @anthonyfieroni said, there is a bug here: listview = m_views.begin().value().view.data(); takes a war copy of the pointer otherwise controlled by a QSharedPointer value. Please fix this by making listview a QSharedPointer itself, so in case an existing pointer is taking, there will be only one set of QSharedPointer around that. kossebau: As @anthonyfieroni said, there is a bug here:
```
listview = m_views.begin().value().view.data… | |||||
| 531 | 531 | | |||
| 532 | changeModel( id ); | 532 | changeModel( id ); | ||
| 533 | changeDelegate( id ); | 533 | changeDelegate( id ); | ||
| 534 | 534 | | |||
| 535 | if (newView) | 535 | if (newView) | ||
| 536 | listview->scrollToBottom(); | 536 | listview->scrollToBottom(); | ||
| 537 | } else | 537 | } else | ||
| 538 | { | 538 | { | ||
| 539 | listview = m_views.value(id); | 539 | listview = m_views.value(id).view; | ||
| 540 | } | 540 | } | ||
| 541 | enableActions(); | 541 | enableActions(); | ||
| 542 | return listview; | 542 | return listview; | ||
| 543 | } | 543 | } | ||
| 544 | 544 | | |||
| 545 | void OutputWidget::raiseOutput(int id) | 545 | void OutputWidget::raiseOutput(int id) | ||
| 546 | { | 546 | { | ||
| 547 | if( m_views.contains(id) ) | 547 | if( m_views.contains(id) ) | ||
| 548 | { | 548 | { | ||
| 549 | if( data->type & KDevelop::IOutputView::MultipleView ) | 549 | if( data->type & KDevelop::IOutputView::MultipleView ) | ||
| 550 | { | 550 | { | ||
| 551 | int idx = m_tabwidget->indexOf( m_views.value(id) ); | 551 | int idx = m_tabwidget->indexOf(m_views.value(id).view); | ||
| 552 | if( idx >= 0 ) | 552 | if( idx >= 0 ) | ||
| 553 | { | 553 | { | ||
| 554 | m_tabwidget->setCurrentIndex( idx ); | 554 | m_tabwidget->setCurrentIndex( idx ); | ||
| 555 | } | 555 | } | ||
| 556 | } else if( data->type & KDevelop::IOutputView::HistoryView ) | 556 | } else if( data->type & KDevelop::IOutputView::HistoryView ) | ||
| 557 | { | 557 | { | ||
| 558 | int idx = m_stackwidget->indexOf( m_views.value(id) ); | 558 | int idx = m_stackwidget->indexOf(m_views.value(id).view); | ||
| 559 | if( idx >= 0 ) | 559 | if( idx >= 0 ) | ||
| 560 | { | 560 | { | ||
| 561 | m_stackwidget->setCurrentIndex( idx ); | 561 | m_stackwidget->setCurrentIndex( idx ); | ||
| 562 | } | 562 | } | ||
| 563 | } | 563 | } | ||
| 564 | } | 564 | } | ||
| 565 | enableActions(); | 565 | enableActions(); | ||
| 566 | } | 566 | } | ||
| ▲ Show 20 Lines • Show All 76 Lines • ▼ Show 20 Line(s) | |||||
| 643 | } | 643 | } | ||
| 644 | 644 | | |||
| 645 | void OutputWidget::outputFilter(const QString& filter) | 645 | void OutputWidget::outputFilter(const QString& filter) | ||
| 646 | { | 646 | { | ||
| 647 | QAbstractItemView *view = qobject_cast<QAbstractItemView*>(currentWidget()); | 647 | QAbstractItemView *view = qobject_cast<QAbstractItemView*>(currentWidget()); | ||
| 648 | if( !view ) | 648 | if( !view ) | ||
| 649 | return; | 649 | return; | ||
| 650 | 650 | | |||
| 651 | int id = m_views.key(qobject_cast<QTreeView*>(view)); | 651 | auto fvIt = getFilteredView(view); | ||
| 652 | auto proxyModel = qobject_cast<QSortFilterProxyModel*>(view->model()); | 652 | auto proxyModel = qobject_cast<QSortFilterProxyModel*>(view->model()); | ||
| 653 | if( !proxyModel ) | 653 | if( !proxyModel ) | ||
| 654 | { | 654 | { | ||
| 655 | proxyModel = new QSortFilterProxyModel(view->model()); | 655 | proxyModel = new QSortFilterProxyModel(view->model()); | ||
| 656 | proxyModel->setDynamicSortFilter(true); | 656 | proxyModel->setDynamicSortFilter(true); | ||
| 657 | proxyModel->setSourceModel(view->model()); | 657 | proxyModel->setSourceModel(view->model()); | ||
| 658 | m_proxyModels.insert(id, proxyModel); | 658 | fvIt->proxyModel = proxyModel; | ||
| 659 | view->setModel(proxyModel); | 659 | view->setModel(proxyModel); | ||
| 660 | } | 660 | } | ||
| 661 | QRegExp regExp(filter, Qt::CaseInsensitive); | 661 | QRegExp regExp(filter, Qt::CaseInsensitive); | ||
| 662 | proxyModel->setFilterRegExp(regExp); | 662 | proxyModel->setFilterRegExp(regExp); | ||
| 663 | m_filters[id] = filter; | 663 | fvIt->filter = filter; | ||
| 664 | } | 664 | } | ||
| 665 | 665 | | |||
| 666 | void OutputWidget::updateFilter(int index) | 666 | void OutputWidget::updateFilter(int index) | ||
| 667 | { | 667 | { | ||
| 668 | QWidget *view = (data->type & KDevelop::IOutputView::MultipleView) | 668 | QWidget *view = (data->type & KDevelop::IOutputView::MultipleView) | ||
| 669 | ? m_tabwidget->widget(index) : m_stackwidget->widget(index); | 669 | ? m_tabwidget->widget(index) : m_stackwidget->widget(index); | ||
| 670 | int id = m_views.key(qobject_cast<QTreeView*>(view)); | 670 | auto fvIt = getFilteredView(qobject_cast<QAbstractItemView*>(view)); | ||
| 671 | 671 | | |||
| 672 | if(m_filters.contains(id)) | 672 | if (fvIt != m_views.end() && !fvIt->filter.isEmpty()) | ||
| 673 | { | 673 | { | ||
| 674 | m_filterInput->setText(m_filters[id]); | 674 | m_filterInput->setText(fvIt->filter); | ||
| 675 | } else | 675 | } else | ||
| 676 | { | 676 | { | ||
| 677 | m_filterInput->clear(); | 677 | m_filterInput->clear(); | ||
| 678 | } | 678 | } | ||
| 679 | } | 679 | } | ||
| 680 | 680 | | |||
| 681 | void OutputWidget::setTitle(int outputId, const QString& title) | 681 | void OutputWidget::setTitle(int outputId, const QString& title) | ||
| 682 | { | 682 | { | ||
| 683 | QTreeView* view = m_views.value(outputId, nullptr); | 683 | auto fview = m_views.value(outputId, FilteredView{}); | ||
| 684 | if(view && (data->type & KDevelop::IOutputView::MultipleView)) { | 684 | if (fview.view && (data->type & KDevelop::IOutputView::MultipleView)) { | ||
| 685 | int idx = m_tabwidget->indexOf(view); | 685 | int idx = m_tabwidget->indexOf(fview.view); | ||
| 686 | if (idx >= 0) { | 686 | if (idx >= 0) { | ||
| 687 | m_tabwidget->setTabText(idx, title); | 687 | m_tabwidget->setTabText(idx, title); | ||
| 688 | } | 688 | } | ||
| 689 | } | 689 | } | ||
| 690 | } | 690 | } | ||
| 691 | | ||||
| 692 | QMap<int, OutputWidget::FilteredView>::iterator OutputWidget::getFilteredView(QAbstractItemView *view) | ||||
| 693 | { | ||||
| 694 | for (auto it = m_views.begin(); it != m_views.end(); ++it) { | ||||
| 695 | if (it->view == view) { | ||||
| 696 | return it; | ||||
| 697 | } | ||||
| 698 | } | ||||
| 699 | return m_views.end(); | ||||
| 700 | } | ||||
| 701 | | ||||
| 702 | void OutputWidget::FilteredView::destroy() | ||||
| 703 | { | ||||
| 704 | if (view) { | ||||
| 705 | delete view; | ||||
| 706 | view = nullptr; | ||||
| 707 | } | ||||
| 708 | if (proxyModel) { | ||||
| 709 | delete proxyModel; | ||||
| 710 | proxyModel = nullptr; | ||||
| 711 | } | ||||
delete is a no-op on null pointers, so you can simply do: delete foo; foo = nullptr; Of course, assuming you do not need to use the pointer just before its deletion. pino: `delete` is a no-op on null pointers, so you can simply do:
lang=c++
delete foo;
foo =… | |||||
| 712 | } | ||||
KDevelop coding style is to have the * with the type, so please keep that (QTreeView*)