haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Replace wxTextFile for writing fp info cache 

wxTextFile uses wxFile which uses write syscalls which means its unbuffered.
This makes file IO more expensive against network shares where it will flush far too aggressively.
wxTextFile is fine for reads however.
This commit is contained in:
Marek Roszko 2020-12-10 20:08:51 -05:00
parent c186a42a54
commit eff9c46e59
5 changed files with 39 additions and 44 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
common/footprint_info.cpp
View File

@ -143,8 +143,7 @@ FOOTPRINT_LIST* FOOTPRINT_LIST::GetInstance( KIWAY& aKiway )
if( !footprintInfo->GetCount() ) if( !footprintInfo->GetCount() )
{ {
wxTextFile footprintInfoCache( aKiway.Prj().GetProjectPath() + "fp-info-cache" ); footprintInfo->ReadCacheFromFile( aKiway.Prj().GetProjectPath() + "fp-info-cache" );
footprintInfo->ReadCacheFromFile( &footprintInfoCache );
} }
return footprintInfo; return footprintInfo;

4
include/footprint_info.h
View File

@ -210,8 +210,8 @@ public:
{ {
} }
virtual void WriteCacheToFile( wxTextFile* aFile ) { }; virtual void WriteCacheToFile( const wxString& aFilePath ) {};
virtual void ReadCacheFromFile( wxTextFile* aFile ) { }; virtual void ReadCacheFromFile( const wxString& aFilePath ){};
/** /**
* @return the number of items stored in list * @return the number of items stored in list

66
pcbnew/footprint_info_impl.cpp
View File

@ -37,6 +37,8 @@
#include <pgm_base.h> #include <pgm_base.h>
#include <wildcards_and_files_ext.h> #include <wildcards_and_files_ext.h>
#include <widgets/progress_reporter.h> #include <widgets/progress_reporter.h>
#include <wx/txtstrm.h>
#include <wx/wfstream.h>
#include <thread> #include <thread>
#include <mutex> #include <mutex>
@ -339,59 +341,55 @@ FOOTPRINT_LIST_IMPL::~FOOTPRINT_LIST_IMPL()
} }
void FOOTPRINT_LIST_IMPL::WriteCacheToFile( wxTextFile* aCacheFile ) void FOOTPRINT_LIST_IMPL::WriteCacheToFile( const wxString& aFilePath )
{ {
if( aCacheFile->Exists() ) wxFFileOutputStream outStream( aFilePath );
{ wxTextOutputStream txtStream( outStream );
if( !aCacheFile->Open() )
return;
aCacheFile->Clear(); if( !outStream.IsOk() )
}
else
{ {
if( !aCacheFile->Create() ) return;
return;
} }
aCacheFile->AddLine( wxString::Format( "%lld", m_list_timestamp ) ); txtStream << wxString::Format( "%lld", m_list_timestamp ) << endl;
for( auto& fpinfo : m_list ) for( std::unique_ptr<FOOTPRINT_INFO>& fpinfo : m_list )
{ {
aCacheFile->AddLine( fpinfo->GetLibNickname() ); txtStream << fpinfo->GetLibNickname() << endl;
aCacheFile->AddLine( fpinfo->GetName() ); txtStream << fpinfo->GetName() << endl;
aCacheFile->AddLine( EscapeString( fpinfo->GetDescription() ) ); txtStream << EscapeString( fpinfo->GetDescription() ) << endl;
aCacheFile->AddLine( EscapeString( fpinfo->GetKeywords() ) ); txtStream << EscapeString( fpinfo->GetKeywords() ) << endl;
aCacheFile->AddLine( wxString::Format( "%d", fpinfo->GetOrderNum() ) ); txtStream << wxString::Format( "%d", fpinfo->GetOrderNum() ) << endl;
aCacheFile->AddLine( wxString::Format( "%u", fpinfo->GetPadCount() ) ); txtStream << wxString::Format( "%u", fpinfo->GetPadCount() ) << endl;
aCacheFile->AddLine( wxString::Format( "%u", fpinfo->GetUniquePadCount() ) ); txtStream << wxString::Format( "%u", fpinfo->GetUniquePadCount() ) << endl;
} }
aCacheFile->Write(); txtStream.Flush();
aCacheFile->Close();
} }
void FOOTPRINT_LIST_IMPL::ReadCacheFromFile( wxTextFile* aCacheFile ) void FOOTPRINT_LIST_IMPL::ReadCacheFromFile( const wxString& aFilePath )
{ {
wxTextFile cacheFile( aFilePath );
m_list_timestamp = 0; m_list_timestamp = 0;
m_list.clear(); m_list.clear();
try try
{ {
if( aCacheFile->Exists() && aCacheFile->Open() ) if( cacheFile.Exists() && cacheFile.Open() )
{ {
aCacheFile->GetFirstLine().ToLongLong( &m_list_timestamp ); cacheFile.GetFirstLine().ToLongLong( &m_list_timestamp );
while( aCacheFile->GetCurrentLine() + 6 < aCacheFile->GetLineCount() ) while( cacheFile.GetCurrentLine() + 6 < cacheFile.GetLineCount() )
{ {
wxString libNickname = aCacheFile->GetNextLine(); wxString libNickname = cacheFile.GetNextLine();
wxString name = aCacheFile->GetNextLine(); wxString name = cacheFile.GetNextLine();
wxString description = UnescapeString( aCacheFile->GetNextLine() ); wxString description = UnescapeString( cacheFile.GetNextLine() );
wxString keywords = UnescapeString( aCacheFile->GetNextLine() ); wxString keywords = UnescapeString( cacheFile.GetNextLine() );
int orderNum = wxAtoi( aCacheFile->GetNextLine() ); int orderNum = wxAtoi( cacheFile.GetNextLine() );
unsigned int padCount = (unsigned) wxAtoi( aCacheFile->GetNextLine() ); unsigned int padCount = (unsigned) wxAtoi( cacheFile.GetNextLine() );
unsigned int uniquePadCount = (unsigned) wxAtoi( aCacheFile->GetNextLine() ); unsigned int uniquePadCount = (unsigned) wxAtoi( cacheFile.GetNextLine() );
auto* fpinfo = new FOOTPRINT_INFO_IMPL( libNickname, name, description, keywords, auto* fpinfo = new FOOTPRINT_INFO_IMPL( libNickname, name, description, keywords,
orderNum, padCount, uniquePadCount ); orderNum, padCount, uniquePadCount );
@ -409,6 +407,6 @@ void FOOTPRINT_LIST_IMPL::ReadCacheFromFile( wxTextFile* aCacheFile )
if( m_list.size() == 0 ) if( m_list.size() == 0 )
m_list_timestamp = 0; m_list_timestamp = 0;
if( aCacheFile->IsOpened() ) if( cacheFile.IsOpened() )
aCacheFile->Close(); cacheFile.Close();
} }

4
pcbnew/footprint_info_impl.h
View File

@ -117,8 +117,8 @@ public:
FOOTPRINT_LIST_IMPL(); FOOTPRINT_LIST_IMPL();
virtual ~FOOTPRINT_LIST_IMPL(); virtual ~FOOTPRINT_LIST_IMPL();
void WriteCacheToFile( wxTextFile* aFile ) override; void WriteCacheToFile( const wxString& aFilePath ) override;
void ReadCacheFromFile( wxTextFile* aFile ) override; void ReadCacheFromFile( const wxString& aFilePath ) override;
bool ReadFootprintFiles( FP_LIB_TABLE* aTable, const wxString* aNickname = nullptr, bool ReadFootprintFiles( FP_LIB_TABLE* aTable, const wxString* aNickname = nullptr,
PROGRESS_REPORTER* aProgressReporter = nullptr ) override; PROGRESS_REPORTER* aProgressReporter = nullptr ) override;

6
pcbnew/pcb_base_edit_frame.cpp
View File

@ -41,8 +41,7 @@ PCB_BASE_EDIT_FRAME::PCB_BASE_EDIT_FRAME( KIWAY* aKiway, wxWindow* aParent,
{ {
if( !GFootprintList.GetCount() ) if( !GFootprintList.GetCount() )
{ {
wxTextFile footprintInfoCache( Prj().GetProjectPath() + "fp-info-cache" ); GFootprintList.ReadCacheFromFile( Prj().GetProjectPath() + "fp-info-cache" );
GFootprintList.ReadCacheFromFile( &footprintInfoCache );
} }
} }
@ -50,8 +49,7 @@ PCB_BASE_EDIT_FRAME::~PCB_BASE_EDIT_FRAME()
{ {
if( wxFileName::IsDirWritable( Prj().GetProjectPath() ) ) if( wxFileName::IsDirWritable( Prj().GetProjectPath() ) )
{ {
wxTextFile footprintInfoCache( Prj().GetProjectPath() + "fp-info-cache" ); GFootprintList.WriteCacheToFile( Prj().GetProjectPath() + "fp-info-cache" );
GFootprintList.WriteCacheToFile( &footprintInfoCache );
} }
} }