kicad-source/tools/io_benchmark/stdstream_line_reader.cpp
John Beard 814ff7c7fc io_benchmark: construct std::ifstream directly
This appears to normally use the move constructor, but on Jenkins Fedora
20, it tried to use the copy ctor, which is deleted.

Constructing directly fixes this, and is the right way, anyway.
2017-03-02 17:25:58 +01:00

91 lines
2.5 KiB
C++

/*
* This program source code file is part of KiCad, a free EDA CAD application.
*
* Copyright (C) 2017 KiCad Developers, see AUTHORS.txt for contributors.
*
* 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, you may find one here:
* http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
* or you may search the http://www.gnu.org website for the version 2 license,
* or you may write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
*/
#include "stdstream_line_reader.h"
#include <ios>
STDISTREAM_LINE_READER::STDISTREAM_LINE_READER() :
LINE_READER( 0 ),
m_stream( nullptr )
{
line = nullptr;
lineNum = 0;
}
STDISTREAM_LINE_READER::~STDISTREAM_LINE_READER()
{
// this is only a view into a string, it cant be deleted by the base
line = nullptr;
}
char* STDISTREAM_LINE_READER::ReadLine() throw( IO_ERROR )
{
getline( *m_stream, m_buffer );
m_buffer.append( 1, '\n' );
length = m_buffer.size();
line = (char*) m_buffer.data(); //ew why no const??
// lineNum is incremented even if there was no line read, because this
// leads to better error reporting when we hit an end of file.
++lineNum;
return m_stream->eof() ? nullptr : line;
}
void STDISTREAM_LINE_READER::setStream( std::istream& aStream )
{
// Could be done with a virtual getStream function, but the
// virtual function call is a noticable (but minor) penalty within
// ReadLine() in tight loops
m_stream = &aStream;
}
IFSTREAM_LINE_READER::IFSTREAM_LINE_READER( const wxString& aFileName ) throw( IO_ERROR ) :
m_fStream( aFileName )
{
if( !m_fStream.is_open() )
{
wxString msg = wxString::Format(
_( "Unable to open filename '%s' for reading" ), aFileName.GetData() );
THROW_IO_ERROR( msg );
}
setStream( m_fStream );
source = aFileName;
}
void IFSTREAM_LINE_READER::Rewind()
{
m_fStream.clear() ;
m_fStream.seekg(0, std::ios::beg );
}