From 0bd3f11f41a8ad1a01f75890e7839e8eec917fda Mon Sep 17 00:00:00 2001 From: "v.audacity" Date: Sat, 26 Mar 2011 01:11:51 +0000 Subject: [PATCH] Make "FIXME" comments be "FIX-ME" for consistency with themselves and "ANSWER-ME" comments, in case somebody ever wants to search for these things... ;-) --- src/Menus.cpp | 2 +- src/Resample.cpp | 2 +- src/WaveTrack.cpp | 2 +- src/export/ExportFFmpegDialogs.cpp | 4 ++-- src/widgets/Ruler.cpp | 10 +++++----- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/Menus.cpp b/src/Menus.cpp index 5af636ea4..7d69984c9 100644 --- a/src/Menus.cpp +++ b/src/Menus.cpp @@ -1331,7 +1331,7 @@ void AudacityProject::RebuildMenuBar() // Under Windows we delete the menus, since we will soon recreate them. // rather oddly, the menus don't vanish as a result of doing this. // Under Linux we can't delete them as this crashes gtk2.... -// FIXME: So we have a memory leak of menu items under linux? Oops. +// FIX-ME: So we have a memory leak of menu items under linux? Oops. #ifdef __WXMSW__ wxMenuBar *menuBar = GetMenuBar(); diff --git a/src/Resample.cpp b/src/Resample.cpp index 719c6ac3f..106fcc18f 100644 --- a/src/Resample.cpp +++ b/src/Resample.cpp @@ -180,7 +180,7 @@ Resample::Resample(bool useBestMethod, double minFactor, double maxFactor) { if (!src_is_valid_ratio (minFactor) || !src_is_valid_ratio (maxFactor)) { fprintf(stderr, "libsamplerate supports only resampling factors between 1/SRC_MAX_RATIO and SRC_MAX_RATIO.\n"); - // FIXME: Audacity will hang after this if branch. + // FIX-ME: Audacity will hang after this if branch. mHandle = NULL; return; } diff --git a/src/WaveTrack.cpp b/src/WaveTrack.cpp index 572b9b0c7..b632d753f 100644 --- a/src/WaveTrack.cpp +++ b/src/WaveTrack.cpp @@ -2115,7 +2115,7 @@ bool WaveTrack::Resample(int rate, ProgressDialog *progress) for (WaveClipList::compatibility_iterator it=GetClipIterator(); it; it=it->GetNext()) if (!it->GetData()->Resample(rate, progress)) { - // FIXME: The track is now in an inconsistent state since some + // FIX-ME: The track is now in an inconsistent state since some // clips are resampled and some are not return false; } diff --git a/src/export/ExportFFmpegDialogs.cpp b/src/export/ExportFFmpegDialogs.cpp index 66bf29cd2..75a76a4b0 100644 --- a/src/export/ExportFFmpegDialogs.cpp +++ b/src/export/ExportFFmpegDialogs.cpp @@ -502,7 +502,7 @@ FFmpegPresets::FFmpegPresets() FFmpegPresets::~FFmpegPresets() { XMLFileWriter writer; - // FIXME: Catch XMLFileWriterException + // FIX-ME: Catch XMLFileWriterException wxFileName xmlFileName(FileNames::DataDir(), wxT("ffmpeg_presets.xml")); writer.Open(xmlFileName.GetFullPath(),wxT("wb")); WriteXMLHeader(writer); @@ -519,7 +519,7 @@ void FFmpegPresets::ImportPresets(wxString &filename) void FFmpegPresets::ExportPresets(wxString &filename) { XMLFileWriter writer; - // FIXME: Catch XMLFileWriterException + // FIX-ME: Catch XMLFileWriterException writer.Open(filename,wxT("wb")); WriteXMLHeader(writer); WriteXML(writer); diff --git a/src/widgets/Ruler.cpp b/src/widgets/Ruler.cpp index 96e2e1a80..913e75028 100644 --- a/src/widgets/Ruler.cpp +++ b/src/widgets/Ruler.cpp @@ -689,7 +689,7 @@ void Ruler::Tick(int pos, double d, bool major, bool minor) wxCoord strW, strH, strD, strL; int strPos, strLen, strLeft, strTop; - // FIXME: We don't draw a tick if of end of our label arrays + // FIX-ME: We don't draw a tick if of end of our label arrays // But we shouldn't have an array of labels. if( mNumMinorMinor >= mLength ) return; @@ -749,7 +749,7 @@ void Ruler::Tick(int pos, double d, bool major, bool minor) } - // FIXME: we shouldn't even get here if strPos < 0. + // FIX-ME: we shouldn't even get here if strPos < 0. // Ruler code currently does not handle very small or // negative sized windows (i.e. don't draw) properly. if( strPos < 0 ) @@ -801,7 +801,7 @@ void Ruler::TickCustom(int labelIdx, bool major, bool minor) wxCoord strW, strH, strD, strL; int strPos, strLen, strLeft, strTop; - // FIXME: We don't draw a tick if of end of our label arrays + // FIX-ME: We don't draw a tick if of end of our label arrays // But we shouldn't have an array of labels. if( mNumMinor >= mLength ) return; @@ -863,7 +863,7 @@ void Ruler::TickCustom(int labelIdx, bool major, bool minor) } - // FIXME: we shouldn't even get here if strPos < 0. + // FIX-ME: we shouldn't even get here if strPos < 0. // Ruler code currently does not handle very small or // negative sized windows (i.e. don't draw) properly. if( strPos < 0 ) @@ -978,7 +978,7 @@ void Ruler::Update(TimeTrack* timetrack)// Envelope *speedEnv, long minSpeed, lo mRect = wxRect(0,0, 0,mLength); } - // FIXME: Surely we do not need to allocate storage for the labels? + // FIX-ME: Surely we do not need to allocate storage for the labels? // We can just recompute them as we need them? Yes, but only if // mCustom is false!!!!