Robin Gareus
a07bd2d585
If the copy c'tor of ProcessorSelection was actually used, assigning the XMLProcessorSelection processors = other.processors; would lead to duplicate free() of the XMLNode* XMLProcessorSelection would need a dedicated copy c'tor that duplicates allocates a new XMLNode. see also #10 at https://www.viva64.com/en/b/0540/ Anyway, the copy c'tor and assignment is never used. This commit makes this explicit.
44 lines
1.1 KiB
C++
44 lines
1.1 KiB
C++
/*
|
|
* Copyright (C) 2017 Paul Davis (paul@linuxaudiosystems.com)
|
|
*
|
|
* This program is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation; either version 2
|
|
* of the License, or (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
|
*/
|
|
|
|
#include "processor_selection.h"
|
|
|
|
void
|
|
ProcessorSelection::clear_processors ()
|
|
{
|
|
processors.clear ();
|
|
ProcessorsChanged ();
|
|
}
|
|
|
|
void
|
|
ProcessorSelection::add (XMLNode* node)
|
|
{
|
|
// XXX check for duplicate
|
|
processors.add (node);
|
|
ProcessorsChanged();
|
|
}
|
|
|
|
void
|
|
ProcessorSelection::set (XMLNode* node)
|
|
{
|
|
clear_processors ();
|
|
processors.set (node);
|
|
ProcessorsChanged ();
|
|
}
|
|
|