Fixed up some small errors

bigfug · Feb 25, 2024 · 9a63d19 · 9a63d19
1 parent 1b2a182
commit 9a63d19
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/FugioApp/contextview.cpp b/FugioApp/contextview.cpp
@@ -1467,7 +1467,7 @@ void ContextView::linkAdded( QUuid pPinId1, QUuid pPinId2 )
 
 void ContextView::linkRemoved( QUuid pPinId1, QUuid pPinId2 )
 {
- for( LinkItem *Link : mLinkList )
+ for( LinkItem *Link : mLinkList.toSet() )
  {
  //qDebug() << ( Link->srcPin() ? Link->srcPin()->uuid() : QUuid() ) << ( Link->dstPin() ? Link->dstPin()->uuid() : QUuid() );
 
@@ -2299,7 +2299,7 @@ void ContextView::ungroup( QList<NodeItem *> &pNodeList, QList<NodeItem *> &pGro
  // Search for any links that link from the ungrouping nodes to outside nodes
  // and delete them
 
- for( LinkItem *LI : mLinkList )
+ for( LinkItem *LI : mLinkList.toSet() )
  {
  PinItem *SrcPin = LI->srcPin();
  PinItem *DstPin = LI->dstPin();

diff --git a/FugioApp/undo/cmdremove.h b/FugioApp/undo/cmdremove.h
@@ -114,27 +114,27 @@ class CmdRemove : public QUndoCommand
 
  QMap<QUuid,QUuid> PinMap = mContextView->loadPinIds();
 
- for( QUuid &Src : mInternalLinkList )
+ for( QUuid Src : mInternalLinkList.keys() )
  {
- for( QUuid &Dst : mInternalLinkList.values( Src ) )
+ for( QUuid Dst : mInternalLinkList.values( Src ) )
  {
  Context->connectPins( Src, Dst );
  }
  }
 
- for( QUuid &Src : mExternalLinkList )
+ for( QUuid Src : mExternalLinkList.keys() )
  {
  const QUuid NewSrc = PinMap.value( Src, Src );
 
- for( QUuid &Dst : mExternalLinkList.values( Src ) )
+ for( QUuid Dst : mExternalLinkList.values( Src ) )
  {
  const QUuid NewDst = PinMap.value( Dst, Dst );
 
  Context->connectPins( NewSrc, NewDst );
  }
  }
 
- for( QUuid &ID : mGroupList )
+ for( QUuid ID : mGroupList )
  {
  mContextView->processGroupLinks( ID );
  }
@@ -146,19 +146,19 @@ class CmdRemove : public QUndoCommand
  {
  QSharedPointer<fugio::ContextInterface> Context = mContextView->context();
 
- for( QUuid &Src : mExternalLinkList )
+ for( QUuid Src : mExternalLinkList.keys() )
  {
- for( QUuid &Dst : mExternalLinkList.values( Src ) )
+ for( QUuid Dst : mExternalLinkList.values( Src ) )
  {
  //qDebug() << Src << Dst;
 
  Context->disconnectPins( Src, Dst );
  }
  }
 
- for( QUuid &Src : mInternalLinkList )
+ for( QUuid Src : mInternalLinkList.keys() )
  {
- for( QUuid &Dst : mInternalLinkList.values( Src ) )
+ for( QUuid Dst : mInternalLinkList.values( Src ) )
  {
  Context->disconnectPins( Src, Dst );
  }