From 0938a42440cc82ce8d0cb064840c258c863714ab Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Fri, 30 Sep 2011 17:55:14 +0000 Subject: fixes for 98% of all the warnings/errors reported by OS X gcc on tiger git-svn-id: svn://localhost/ardour2/branches/3.0@10179 d708f5d6-7413-0410-9779-e7cbd77b26cf --- libs/appleutility/CAAudioChannelLayout.h | 2 +- libs/appleutility/CAAudioUnit.cpp | 16 ++++----- libs/appleutility/CAReferenceCounted.h | 6 ++-- libs/ardour/ardour/audio_unit.h | 5 ++- libs/ardour/ardour/audiosource.h | 2 +- libs/ardour/ardour/caimportable.h | 6 ++-- libs/ardour/ardour/coreaudiosource.h | 2 +- libs/ardour/ardour/export_channel.h | 2 +- libs/ardour/ardour/lxvst_plugin.h | 8 ++--- libs/ardour/ardour/plugin.h | 3 +- libs/ardour/audio_playlist_source.cc | 2 +- libs/ardour/audio_port.cc | 2 +- libs/ardour/audio_unit.cc | 40 +++++++++++++--------- libs/ardour/export_channel.cc | 4 +-- libs/ardour/file_source.cc | 2 +- libs/ardour/ladspa_plugin.cc | 2 +- libs/ardour/lxvst_plugin.cc | 6 ++-- libs/ardour/midi_diskstream.cc | 1 + libs/ardour/midi_playlist_source.cc | 12 +++---- libs/ardour/midi_port.cc | 2 +- libs/ardour/midi_stretch.cc | 2 +- libs/ardour/pannable.cc | 2 +- libs/ardour/playlist_source.cc | 4 +-- libs/ardour/plugin.cc | 4 +-- libs/ardour/plugin_manager.cc | 2 +- libs/ardour/port.cc | 4 +-- libs/ardour/processor.cc | 1 + libs/ardour/session.cc | 2 +- libs/ardour/session_process.cc | 2 +- libs/ardour/speakers.cc | 1 + libs/ardour/vstfxinfofile.cc | 6 ++-- libs/audiographer/audiographer/debuggable.h | 1 + libs/audiographer/audiographer/process_context.h | 2 +- libs/evoral/evoral/PatchChange.hpp | 4 ++- libs/evoral/src/ControlSet.cpp | 2 +- libs/evoral/src/Sequence.cpp | 2 +- libs/evoral/src/libsmf/smf_load.c | 9 +++-- libs/evoral/src/libsmf/smf_save.c | 1 + .../libgnomecanvas/gnome-canvas-bpath.c | 2 +- .../gnomecanvas/libgnomecanvas/gnome-canvas-line.c | 2 +- .../libgnomecanvas/gnome-canvas-rich-text.c | 24 ++++++------- .../libgnomecanvas/gnome-canvas-widget.c | 10 +++--- libs/gnomecanvas/libgnomecanvas/gnome-canvas.c | 4 +-- libs/gtkmm2ext/bindings.cc | 12 +++---- libs/gtkmm2ext/cairocell.cc | 4 +-- libs/gtkmm2ext/gtkapplication_quartz.mm | 15 ++++---- libs/gtkmm2ext/gtkapplication_x11.c | 5 ++- .../gtkmm2ext/cell_renderer_pixbuf_multi.h | 7 ---- .../gtkmm2ext/cell_renderer_pixbuf_toggle.h | 7 ---- libs/gtkmm2ext/gtkmm2ext/dndvbox.h | 6 ++-- libs/gtkmm2ext/keyboard.cc | 2 +- libs/gtkmm2ext/motionfeedback.cc | 6 ++-- libs/gtkmm2ext/treeutils.cc | 4 +-- libs/midi++2/midi++/names.h | 5 ++- libs/panners/vbap/vbap.cc | 8 +++-- libs/pbd/boost_debug.cc | 16 ++++----- libs/qm-dsp/dsp/onsets/DetectionFunction.cpp | 4 +-- libs/qm-dsp/dsp/onsets/PeakPicking.cpp | 5 +-- libs/qm-dsp/dsp/signalconditioning/DFProcess.cpp | 2 +- libs/qm-dsp/dsp/wavelet/Wavelet.cpp | 2 +- libs/qm-dsp/maths/CosineDistance.cpp | 2 +- libs/qm-dsp/maths/MathUtilities.cpp | 11 +++--- libs/qm-dsp/maths/Polyfit.h | 9 ++--- libs/rubberband/src/StretcherProcess.cpp | 2 ++ libs/surfaces/generic_midi/midicontrollable.cc | 1 - libs/surfaces/osc/osc.cc | 2 +- libs/surfaces/osc/osc.h | 6 ++-- libs/taglib/taglib/fileref.h | 1 + libs/taglib/taglib/mpeg/id3v1/id3v1tag.h | 1 + libs/vamp-sdk/src/vamp-sdk/PluginAdapter.cpp | 2 +- 70 files changed, 189 insertions(+), 176 deletions(-) (limited to 'libs') diff --git a/libs/appleutility/CAAudioChannelLayout.h b/libs/appleutility/CAAudioChannelLayout.h index 796f197c3b..5c351c29b5 100644 --- a/libs/appleutility/CAAudioChannelLayout.h +++ b/libs/appleutility/CAAudioChannelLayout.h @@ -150,7 +150,7 @@ private: private: ACLRefCounter () : mLayout(NULL) { } - ACLRefCounter(const ACLRefCounter&) : mLayout(NULL) { } + ACLRefCounter(const ACLRefCounter& other) : CAReferenceCounted (other), mLayout(NULL) { } ACLRefCounter& operator=(const ACLRefCounter&) { return *this; } }; diff --git a/libs/appleutility/CAAudioUnit.cpp b/libs/appleutility/CAAudioUnit.cpp index aaf57f233d..131969b66c 100644 --- a/libs/appleutility/CAAudioUnit.cpp +++ b/libs/appleutility/CAAudioUnit.cpp @@ -191,7 +191,7 @@ private: private: // get the compiler to tell us when we do a bad thing!!! AUState () {} - AUState (const AUState&) {} + AUState (const AUState& other) : CAReferenceCounted (other) {} AUState& operator= (const AUState&) { return *this; } }; @@ -1084,13 +1084,13 @@ bool CAAudioUnit::HasCustomView () const OSStatus CAAudioUnit::GetParameter(AudioUnitParameterID inID, AudioUnitScope scope, AudioUnitElement element, Float32 &outValue) const { - return mDataPtr ? mDataPtr->GetParameter (inID, scope, element, outValue) : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->GetParameter (inID, scope, element, outValue) : paramErr; } OSStatus CAAudioUnit::SetParameter(AudioUnitParameterID inID, AudioUnitScope scope, AudioUnitElement element, Float32 value, UInt32 bufferOffsetFrames) { - return mDataPtr ? mDataPtr->SetParameter (inID, scope, element, value, bufferOffsetFrames) : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->SetParameter (inID, scope, element, value, bufferOffsetFrames) : paramErr; } OSStatus CAAudioUnit::MIDIEvent (UInt32 inStatus, @@ -1098,7 +1098,7 @@ OSStatus CAAudioUnit::MIDIEvent (UInt32 inStatus, UInt32 inData2, UInt32 inOffsetSampleFrame) { - return mDataPtr ? mDataPtr->MIDIEvent (inStatus, inData1, inData2, inOffsetSampleFrame) : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->MIDIEvent (inStatus, inData1, inData2, inOffsetSampleFrame) : paramErr; } OSStatus CAAudioUnit::StartNote (MusicDeviceInstrumentID inInstrument, @@ -1107,15 +1107,15 @@ OSStatus CAAudioUnit::StartNote (MusicDeviceInstrumentID inInstrument, UInt32 inOffsetSampleFrame, const MusicDeviceNoteParams * inParams) { - return mDataPtr ? mDataPtr->StartNote (inInstrument, inGroupID, outNoteInstanceID, inOffsetSampleFrame, inParams) - : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->StartNote (inInstrument, inGroupID, outNoteInstanceID, inOffsetSampleFrame, inParams) + : paramErr; } OSStatus CAAudioUnit::StopNote (MusicDeviceGroupID inGroupID, NoteInstanceID inNoteInstanceID, UInt32 inOffsetSampleFrame) { - return mDataPtr ? mDataPtr->StopNote (inGroupID, inNoteInstanceID, inOffsetSampleFrame) : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->StopNote (inGroupID, inNoteInstanceID, inOffsetSampleFrame) : paramErr; } #pragma mark __Render @@ -1126,7 +1126,7 @@ OSStatus CAAudioUnit::Render (AudioUnitRenderActionFlags * ioActionFlags, UInt32 inNumberFrames, AudioBufferList * ioData) { - return mDataPtr ? mDataPtr->Render (ioActionFlags, inTimeStamp, inOutputBusNumber, inNumberFrames, ioData) : paramErr; + return mDataPtr ? (OSStatus) mDataPtr->Render (ioActionFlags, inTimeStamp, inOutputBusNumber, inNumberFrames, ioData) : paramErr; } static AURenderCallbackStruct sRenderCallback; diff --git a/libs/appleutility/CAReferenceCounted.h b/libs/appleutility/CAReferenceCounted.h index fc18dc8ca4..a9c8dccca3 100644 --- a/libs/appleutility/CAReferenceCounted.h +++ b/libs/appleutility/CAReferenceCounted.h @@ -70,12 +70,12 @@ public: } protected: - virtual ~CAReferenceCounted() { } - + virtual ~CAReferenceCounted() { } + CAReferenceCounted(const CAReferenceCounted &) : mRefCount(0) { } + private: SInt32 mRefCount; - CAReferenceCounted(const CAReferenceCounted &) : mRefCount(0) { } CAReferenceCounted operator=(const CAReferenceCounted &) { return *this; } }; diff --git a/libs/ardour/ardour/audio_unit.h b/libs/ardour/ardour/audio_unit.h index f36e89073b..0a786d5462 100644 --- a/libs/ardour/ardour/audio_unit.h +++ b/libs/ardour/ardour/audio_unit.h @@ -97,7 +97,6 @@ class AUPlugin : public ARDOUR::Plugin int set_state(const XMLNode& node, int); - PresetRecord save_preset (std::string name); bool load_preset (PresetRecord); std::string current_preset() const; @@ -140,6 +139,10 @@ class AUPlugin : public ARDOUR::Plugin static std::string maybe_fix_broken_au_id (const std::string&); + protected: + std::string do_save_preset (std::string name); + void do_remove_preset (std::string); + private: void find_presets (); diff --git a/libs/ardour/ardour/audiosource.h b/libs/ardour/ardour/audiosource.h index 67d5ddce3a..c9eb3bfc45 100644 --- a/libs/ardour/ardour/audiosource.h +++ b/libs/ardour/ardour/audiosource.h @@ -145,7 +145,7 @@ class AudioSource : virtual public Source, virtual framecnt_t read_unlocked (Sample *dst, framepos_t start, framecnt_t cnt) const = 0; virtual framecnt_t write_unlocked (Sample *dst, framecnt_t cnt) = 0; virtual std::string peak_path(std::string audio_path) = 0; - virtual std::string find_broken_peakfile (std::string missing_peak_path, + virtual std::string find_broken_peakfile (std::string /* missing_peak_path */, std::string audio_path) { return peak_path (audio_path); } virtual int read_peaks_with_fpp (PeakData *peaks, diff --git a/libs/ardour/ardour/caimportable.h b/libs/ardour/ardour/caimportable.h index 2e86ff2c90..1ed8676ee6 100644 --- a/libs/ardour/ardour/caimportable.h +++ b/libs/ardour/ardour/caimportable.h @@ -33,11 +33,11 @@ class CAImportableSource : public ImportableSource { CAImportableSource (const std::string& path); virtual ~CAImportableSource(); - nframes_t read (Sample* buffer, nframes_t nframes); + framecnt_t read (Sample* buffer, framecnt_t nframes); uint32_t channels() const; framecnt_t length() const; - nframes_t samplerate() const; - void seek (nframes_t pos); + framecnt_t samplerate() const; + void seek (framepos_t pos); framepos_t natural_position() const { return 0; } bool clamped_at_unity () const { return false; } diff --git a/libs/ardour/ardour/coreaudiosource.h b/libs/ardour/ardour/coreaudiosource.h index 2f283dfd36..820fa0b9d8 100644 --- a/libs/ardour/ardour/coreaudiosource.h +++ b/libs/ardour/ardour/coreaudiosource.h @@ -54,7 +54,7 @@ class CoreAudioSource : public AudioFileSource { uint16_t n_channels; void init_cafile (); - int safe_read (Sample*, nframes_t start, framecnt_t cnt, AudioBufferList&) const; + int safe_read (Sample*, framepos_t start, framecnt_t cnt, AudioBufferList&) const; }; }; /* namespace ARDOUR */ diff --git a/libs/ardour/ardour/export_channel.h b/libs/ardour/ardour/export_channel.h index 1a71f14c3c..a73d04aa36 100644 --- a/libs/ardour/ardour/export_channel.h +++ b/libs/ardour/ardour/export_channel.h @@ -47,7 +47,7 @@ class ExportChannel : public boost::less_than_comparable virtual ~ExportChannel () {} - virtual void set_max_buffer_size(framecnt_t frames) { } + virtual void set_max_buffer_size(framecnt_t) { } virtual void read (Sample const *& data, framecnt_t frames) const = 0; virtual bool empty () const = 0; diff --git a/libs/ardour/ardour/lxvst_plugin.h b/libs/ardour/ardour/lxvst_plugin.h index 24793772ef..108437b6a3 100755 --- a/libs/ardour/ardour/lxvst_plugin.h +++ b/libs/ardour/ardour/lxvst_plugin.h @@ -73,10 +73,10 @@ class LXVSTPlugin : public ARDOUR::Plugin std::string state_node_name() const { return "lxvst"; } void print_parameter (uint32_t, char*, uint32_t len) const; - bool parameter_is_audio(uint32_t i) const { return false; } - bool parameter_is_control(uint32_t i) const { return true; } - bool parameter_is_input(uint32_t i) const { return true; } - bool parameter_is_output(uint32_t i) const { return false; } + bool parameter_is_audio(uint32_t) const { return false; } + bool parameter_is_control(uint32_t) const { return true; } + bool parameter_is_input(uint32_t) const { return true; } + bool parameter_is_output(uint32_t) const { return false; } bool load_preset (PresetRecord); int first_user_preset_index () const; diff --git a/libs/ardour/ardour/plugin.h b/libs/ardour/ardour/plugin.h index 2c819e96f2..fcd7d6d19b 100644 --- a/libs/ardour/ardour/plugin.h +++ b/libs/ardour/ardour/plugin.h @@ -136,8 +136,7 @@ class Plugin : public PBD::StatefulDestructible, public Latent typedef std::map ScalePoints; - virtual boost::shared_ptr - get_scale_points(uint32_t port_index) const { + virtual boost::shared_ptr get_scale_points(uint32_t /*port_index*/) const { return boost::shared_ptr(); } diff --git a/libs/ardour/audio_playlist_source.cc b/libs/ardour/audio_playlist_source.cc index 79c0c12691..141a8495f9 100644 --- a/libs/ardour/audio_playlist_source.cc +++ b/libs/ardour/audio_playlist_source.cc @@ -171,7 +171,7 @@ AudioPlaylistSource::read_unlocked (Sample* dst, framepos_t start, framecnt_t cn } framecnt_t -AudioPlaylistSource::write_unlocked (Sample *src, framecnt_t cnt) +AudioPlaylistSource::write_unlocked (Sample *, framecnt_t) { fatal << string_compose (_("programming error: %1"), "AudioPlaylistSource::write() called - should be impossible") << endmsg; /*NOTREACHED*/ diff --git a/libs/ardour/audio_port.cc b/libs/ardour/audio_port.cc index 556f6e0c2a..ae64d797c9 100644 --- a/libs/ardour/audio_port.cc +++ b/libs/ardour/audio_port.cc @@ -53,7 +53,7 @@ AudioPort::cycle_start (pframes_t nframes) } void -AudioPort::cycle_end (pframes_t nframes) +AudioPort::cycle_end (pframes_t) { if (sends_output() && !_buffer->written()) { /* we can't use nframes here because the current buffer capacity may diff --git a/libs/ardour/audio_unit.cc b/libs/ardour/audio_unit.cc index a4d2dd241e..04efb2e50c 100644 --- a/libs/ardour/audio_unit.cc +++ b/libs/ardour/audio_unit.cc @@ -1341,7 +1341,7 @@ AUPlugin::get_beat_and_tempo_callback (Float64* outCurrentBeat, float beat; beat = metric.meter().beats_per_bar() * bbt.bars; beat += bbt.beats; - beat += bbt.ticks / BBT_Time::ticks_per_beat; + beat += bbt.ticks / Timecode::BBT_Time::ticks_per_beat; *outCurrentBeat = beat; } @@ -1381,8 +1381,9 @@ AUPlugin::get_musical_time_location_callback (UInt32* outDeltaSampleOffsetToNe /* on the beat */ *outDeltaSampleOffsetToNextBeat = 0; } else { - *outDeltaSampleOffsetToNextBeat = (UInt32) floor (((BBT_Time::ticks_per_beat - bbt.ticks)/BBT_Time::ticks_per_beat) * // fraction of a beat to next beat - metric.tempo().frames_per_beat(_session.frame_rate(), metric.meter())); // frames per beat + *outDeltaSampleOffsetToNextBeat = (UInt32) + floor (((Timecode::BBT_Time::ticks_per_beat - bbt.ticks)/Timecode::BBT_Time::ticks_per_beat) * // fraction of a beat to next beat + metric.tempo().frames_per_beat(_session.frame_rate(), metric.meter())); // frames per beat } } @@ -1473,7 +1474,7 @@ AUPlugin::get_transport_state_callback (Boolean* outIsPlaying, float beat; beat = metric.meter().beats_per_bar() * bbt.bars; beat += bbt.beats; - beat += bbt.ticks / BBT_Time::ticks_per_beat; + beat += bbt.ticks / Timecode::BBT_Time::ticks_per_beat; *outCycleStartBeat = beat; } @@ -1485,7 +1486,7 @@ AUPlugin::get_transport_state_callback (Boolean* outIsPlaying, float beat; beat = metric.meter().beats_per_bar() * bbt.bars; beat += bbt.beats; - beat += bbt.ticks / BBT_Time::ticks_per_beat; + beat += bbt.ticks / Timecode::BBT_Time::ticks_per_beat; *outCycleEndBeat = beat; } @@ -1554,7 +1555,7 @@ AUPlugin::parameter_is_output (uint32_t) const } void -AUPlugin::add_state (XMLNode* root) +AUPlugin::add_state (XMLNode* root) const { LocaleGuard lg (X_("POSIX")); @@ -1573,7 +1574,7 @@ AUPlugin::add_state (XMLNode* root) if (!xmlData) { error << _("Could not create XML version of property list") << endmsg; - return *root; + return; } /* re-parse XML bytes to create a libxml++ XMLTree that we can merge into @@ -1662,7 +1663,7 @@ AUPlugin::set_state(const XMLNode& node, int version) } bool -AUPlugin::load_preset (PluginRecord r) +AUPlugin::load_preset (PresetRecord r) { Plugin::load_preset (r); @@ -1675,7 +1676,7 @@ AUPlugin::load_preset (PluginRecord r) /* look first in "user" presets */ - if ((ux = user_preset_map.find (preset_label)) != user_preset_map.end()) { + if ((ux = user_preset_map.find (r.label)) != user_preset_map.end()) { if ((propertyList = load_property_list (ux->second)) != 0) { TRACE_API ("set preset from user presets\n"); @@ -1692,7 +1693,7 @@ AUPlugin::load_preset (PluginRecord r) CFRelease(propertyList); } - } else if ((fx = factory_preset_map.find (preset_label)) != factory_preset_map.end()) { + } else if ((fx = factory_preset_map.find (r.label)) != factory_preset_map.end()) { AUPreset preset; @@ -1725,8 +1726,13 @@ AUPlugin::load_preset (PluginRecord r) #endif } -PresetRecord -AUPlugin::save_preset (string preset_name) +void +AUPlugin::do_remove_preset (std::string) +{ +} + +string +AUPlugin::do_save_preset (string preset_name) { #ifdef AU_STATE_SUPPORT CFPropertyListRef propertyList; @@ -1776,7 +1782,7 @@ AUPlugin::save_preset (string preset_name) CFRelease(propertyList); - return ret; + return string ("file:///") + user_preset_path; #else if (!seen_saving_message) { info << string_compose (_("Saving AudioUnit presets is not supported in this build of %1. Consider paying for a newer version"), @@ -1784,7 +1790,7 @@ AUPlugin::save_preset (string preset_name) << endmsg; seen_saving_message = true; } - return false; + return string(); #endif } @@ -2006,7 +2012,7 @@ AUPlugin::find_presets () /* now fill the vector with the names we have */ for (UserPresetMap::iterator i = user_preset_map.begin(); i != user_preset_map.end(); ++i) { - _presets.insert (i->second, Plugin::PresetRecord (i->second, i->first)); + _presets.insert (make_pair (i->second, Plugin::PresetRecord (i->second, i->first))); } /* add factory presets */ @@ -2014,7 +2020,7 @@ AUPlugin::find_presets () for (FactoryPresetMap::iterator i = factory_preset_map.begin(); i != factory_preset_map.end(); ++i) { /* XXX: dubious */ string const uri = string_compose ("%1", _presets.size ()); - _presets.push_back (uri, Plugin::PresetRecord (uri, i->first)); + _presets.insert (make_pair (uri, Plugin::PresetRecord (uri, i->first))); } #endif @@ -2082,10 +2088,12 @@ AUPluginInfo::discover () PluginInfoList* plugs = new PluginInfoList; +#if 0 discover_fx (*plugs); discover_music (*plugs); discover_generators (*plugs); discover_instruments (*plugs); +#endif return plugs; } diff --git a/libs/ardour/export_channel.cc b/libs/ardour/export_channel.cc index 785648393e..4593c4de9f 100644 --- a/libs/ardour/export_channel.cc +++ b/libs/ardour/export_channel.cc @@ -241,13 +241,13 @@ RouteExportChannel::read (Sample const *& data, framecnt_t frames) const } void -RouteExportChannel::get_state (XMLNode * node) const +RouteExportChannel::get_state (XMLNode *) const { // TODO } void -RouteExportChannel::set_state (XMLNode * node, Session & session) +RouteExportChannel::set_state (XMLNode *, Session &) { // TODO } diff --git a/libs/ardour/file_source.cc b/libs/ardour/file_source.cc index 877a0a4939..e35ee81364 100644 --- a/libs/ardour/file_source.cc +++ b/libs/ardour/file_source.cc @@ -235,7 +235,7 @@ FileSource::move_to_trash (const string& trash_dir_name) */ bool FileSource::find (Session& s, DataType type, const string& path, bool must_exist, - bool& isnew, uint16_t& chan, string& found_path) + bool& isnew, uint16_t& /* chan */, string& found_path) { bool ret = false; string keeppath; diff --git a/libs/ardour/ladspa_plugin.cc b/libs/ardour/ladspa_plugin.cc index 1fc8b2f1a0..ef1a6b6682 100644 --- a/libs/ardour/ladspa_plugin.cc +++ b/libs/ardour/ladspa_plugin.cc @@ -761,7 +761,7 @@ LadspaPlugin::load_preset (PresetRecord r) /* XXX: should be in liblrdf */ static void -lrdf_remove_preset (const char *source, const char *setting_uri) +lrdf_remove_preset (const char* /*source*/, const char *setting_uri) { lrdf_statement p; lrdf_statement *q; diff --git a/libs/ardour/lxvst_plugin.cc b/libs/ardour/lxvst_plugin.cc index 737d1c7360..d42679742b 100755 --- a/libs/ardour/lxvst_plugin.cc +++ b/libs/ardour/lxvst_plugin.cc @@ -125,7 +125,7 @@ LXVSTPlugin::set_block_size (pframes_t nframes) } float -LXVSTPlugin::default_value (uint32_t port) +LXVSTPlugin::default_value (uint32_t) { return 0; } @@ -346,7 +346,7 @@ LXVSTPlugin::load_plugin_preset (PresetRecord r) int index; int const p = sscanf (r.uri.c_str(), "VST:%d:%d", &id, &index); assert (p == 2); - + _vstfx->want_program = index; return true; } @@ -653,7 +653,7 @@ LXVSTPlugin::has_editor () const } void -LXVSTPlugin::print_parameter (uint32_t param, char *buf, uint32_t len) const +LXVSTPlugin::print_parameter (uint32_t param, char *buf, uint32_t /*len*/) const { char *first_nonws; diff --git a/libs/ardour/midi_diskstream.cc b/libs/ardour/midi_diskstream.cc index bf5be343e2..28ee12cc5a 100644 --- a/libs/ardour/midi_diskstream.cc +++ b/libs/ardour/midi_diskstream.cc @@ -291,6 +291,7 @@ MidiDiskstream::use_copy_playlist () int MidiDiskstream::set_destructive (bool yn) { + yn = 0; // stop pedantic gcc complaints about unused parameter assert( ! destructive()); assert( ! yn); return -1; diff --git a/libs/ardour/midi_playlist_source.cc b/libs/ardour/midi_playlist_source.cc index 3b2c8dbb48..6270ad4339 100644 --- a/libs/ardour/midi_playlist_source.cc +++ b/libs/ardour/midi_playlist_source.cc @@ -58,7 +58,7 @@ on other ways to approach this issue. ********************************************************************************/ MidiPlaylistSource::MidiPlaylistSource (Session& s, const ID& orig, const std::string& name, boost::shared_ptr p, - uint32_t chn, frameoffset_t begin, framecnt_t len, Source::Flag flags) + uint32_t /*chn*/, frameoffset_t begin, framecnt_t len, Source::Flag flags) : Source (s, DataType::MIDI, name) , MidiSource (s, name, flags) , PlaylistSource (s, orig, name, p, DataType::MIDI, begin, len, flags) @@ -128,9 +128,9 @@ MidiPlaylistSource::length (framepos_t) const framepos_t MidiPlaylistSource::read_unlocked (Evoral::EventSink& dst, - framepos_t position, + framepos_t /*position*/, framepos_t start, framecnt_t cnt, - MidiStateTracker* tracker) const + MidiStateTracker*) const { boost::shared_ptr mp = boost::dynamic_pointer_cast (_playlist); @@ -143,8 +143,8 @@ MidiPlaylistSource::read_unlocked (Evoral::EventSink& dst, framepos_t MidiPlaylistSource::write_unlocked (MidiRingBuffer& dst, - framepos_t position, - framecnt_t cnt) + framepos_t, + framecnt_t) { fatal << string_compose (_("programming error: %1"), "MidiPlaylistSource::write_unlocked() called - should be impossible") << endmsg; /*NOTREACHED*/ @@ -159,7 +159,7 @@ MidiPlaylistSource::append_event_unlocked_beats(const Evoral::Event& ev, framepos_t source_start) +MidiPlaylistSource::append_event_unlocked_frames(const Evoral::Event& /* ev */, framepos_t /*source_start*/) { fatal << string_compose (_("programming error: %1"), "MidiPlaylistSource::append_event_unlocked_frames() called - should be impossible") << endmsg; /*NOTREACHED*/ diff --git a/libs/ardour/midi_port.cc b/libs/ardour/midi_port.cc index c27332fce1..50d4ce64e8 100644 --- a/libs/ardour/midi_port.cc +++ b/libs/ardour/midi_port.cc @@ -155,7 +155,7 @@ MidiPort::resolve_notes (void* jack_buffer, MidiBuffer::TimeType when) } void -MidiPort::flush_buffers (pframes_t nframes, framepos_t time) +MidiPort::flush_buffers (pframes_t nframes, framepos_t /*time*/) { if (sends_output ()) { diff --git a/libs/ardour/midi_stretch.cc b/libs/ardour/midi_stretch.cc index d9ac1403a9..2ae86a6d1c 100644 --- a/libs/ardour/midi_stretch.cc +++ b/libs/ardour/midi_stretch.cc @@ -42,7 +42,7 @@ MidiStretch::~MidiStretch () } int -MidiStretch::run (boost::shared_ptr r, Progress* progress) +MidiStretch::run (boost::shared_ptr r, Progress*) { SourceList nsrcs; char suffix[32]; diff --git a/libs/ardour/pannable.cc b/libs/ardour/pannable.cc index ba4632f2b0..4dad58ed12 100644 --- a/libs/ardour/pannable.cc +++ b/libs/ardour/pannable.cc @@ -182,7 +182,7 @@ Pannable::get_state () } XMLNode& -Pannable::state (bool full) +Pannable::state (bool /*full*/) { XMLNode* node = new XMLNode (X_("Pannable")); diff --git a/libs/ardour/playlist_source.cc b/libs/ardour/playlist_source.cc index f68033c815..4c01831b02 100644 --- a/libs/ardour/playlist_source.cc +++ b/libs/ardour/playlist_source.cc @@ -44,7 +44,7 @@ using namespace ARDOUR; using namespace PBD; PlaylistSource::PlaylistSource (Session& s, const ID& orig, const std::string& name, boost::shared_ptr p, DataType type, - frameoffset_t begin, framecnt_t len, Source::Flag flags) + frameoffset_t begin, framecnt_t len, Source::Flag /*flags*/) : Source (s, type, name) , _playlist (p) , _original (orig) @@ -92,7 +92,7 @@ PlaylistSource::add_state (XMLNode& node) } int -PlaylistSource::set_state (const XMLNode& node, int version) +PlaylistSource::set_state (const XMLNode& node, int /*version*/) { /* check that we have a playlist ID */ diff --git a/libs/ardour/plugin.cc b/libs/ardour/plugin.cc index 153c624a5a..12760e5d6e 100644 --- a/libs/ardour/plugin.cc +++ b/libs/ardour/plugin.cc @@ -236,7 +236,7 @@ Plugin::preset_by_uri (const string& uri) int Plugin::connect_and_run (BufferSet& bufs, - ChanMapping in_map, ChanMapping out_map, + ChanMapping /*in_map*/, ChanMapping /*out_map*/, pframes_t nframes, framecnt_t offset) { if (bufs.count().n_midi() > 0) { @@ -306,7 +306,7 @@ Plugin::set_parameter (uint32_t which, float val) } int -Plugin::set_state (const XMLNode& node, int version) +Plugin::set_state (const XMLNode& node, int /*version*/) { XMLProperty const * p = node.property (X_("last-preset-uri")); if (p) { diff --git a/libs/ardour/plugin_manager.cc b/libs/ardour/plugin_manager.cc index 204867a43d..613fc85c6d 100644 --- a/libs/ardour/plugin_manager.cc +++ b/libs/ardour/plugin_manager.cc @@ -652,7 +652,7 @@ PluginManager::add_lxvst_directory (string path) return -1; } -static bool lxvst_filter (const string& str, void *arg) +static bool lxvst_filter (const string& str, void *) { /* Not a dotfile, has a prefix before a period, suffix is "so" */ diff --git a/libs/ardour/port.cc b/libs/ardour/port.cc index 5a7c859dbd..b418943bba 100644 --- a/libs/ardour/port.cc +++ b/libs/ardour/port.cc @@ -230,7 +230,7 @@ Port::reset () } void -Port::cycle_start (pframes_t nframes) +Port::cycle_start (pframes_t) { _port_buffer_offset = 0; } @@ -307,7 +307,7 @@ Port::private_latency_range (bool playback) const } jack_latency_range_t -Port::public_latency_range (bool playback) const +Port::public_latency_range (bool /*playback*/) const { jack_latency_range_t r; diff --git a/libs/ardour/processor.cc b/libs/ardour/processor.cc index f7afeb99a7..8b54bd4f55 100644 --- a/libs/ardour/processor.cc +++ b/libs/ardour/processor.cc @@ -76,6 +76,7 @@ Processor::Processor (const Processor& other) : Evoral::ControlSet (other) , SessionObject (other.session(), other.name()) , Automatable (other.session()) + , Latent (other) , _pending_active(other._pending_active) , _active(other._active) , _next_ab_is_active(false) diff --git a/libs/ardour/session.cc b/libs/ardour/session.cc index 5b1dd3894f..a42f2d4430 100644 --- a/libs/ardour/session.cc +++ b/libs/ardour/session.cc @@ -2511,7 +2511,7 @@ Session::io_name_is_legal (const std::string& name) } void -Session::set_exclusive_input_active (boost::shared_ptr rt, bool others_on) +Session::set_exclusive_input_active (boost::shared_ptr rt, bool /*others_on*/) { RouteList rl; vector connections; diff --git a/libs/ardour/session_process.cc b/libs/ardour/session_process.cc index 947bceac9d..4d974d4830 100644 --- a/libs/ardour/session_process.cc +++ b/libs/ardour/session_process.cc @@ -617,7 +617,7 @@ Session::calculate_moving_average_of_slave_delta (int dir, framecnt_t this_delta } void -Session::track_slave_state (float slave_speed, framepos_t slave_transport_frame, framecnt_t this_delta) +Session::track_slave_state (float slave_speed, framepos_t slave_transport_frame, framecnt_t /*this_delta*/) { if (slave_speed != 0.0f) { diff --git a/libs/ardour/speakers.cc b/libs/ardour/speakers.cc index 71e524f490..51bae16585 100644 --- a/libs/ardour/speakers.cc +++ b/libs/ardour/speakers.cc @@ -71,6 +71,7 @@ Speakers::Speakers () } Speakers::Speakers (const Speakers& s) + : Stateful () { _speakers = s._speakers; } diff --git a/libs/ardour/vstfxinfofile.cc b/libs/ardour/vstfxinfofile.cc index 2d54f4d1cd..076cdad55f 100755 --- a/libs/ardour/vstfxinfofile.cc +++ b/libs/ardour/vstfxinfofile.cc @@ -297,12 +297,12 @@ static VSTFXInfo* vstfx_info_from_plugin(VSTFX *vstfx) /* A simple 'dummy' audiomaster callback which should be ok, we will only be instantiating the plugin in order to get its info*/ -static long simple_master_callback(struct AEffect *fx, long opcode, long index, long value, void *ptr, float opt) +static long simple_master_callback(struct AEffect *, long opcode, long, long, void *, float) { - if(opcode == audioMasterVersion) + if (opcode == audioMasterVersion) return 2; - else + else return 0; } diff --git a/libs/audiographer/audiographer/debuggable.h b/libs/audiographer/audiographer/debuggable.h index 5ef382890b..79e0f80dd3 100644 --- a/libs/audiographer/audiographer/debuggable.h +++ b/libs/audiographer/audiographer/debuggable.h @@ -44,6 +44,7 @@ class Debuggable bool debug_level (DebugLevel level) { #ifdef NDEBUG + level = DEFAULT_DEBUG_LEVEL; /* stop pedantic gcc complaints about unused parameter */ return false; #else return L >= level; diff --git a/libs/audiographer/audiographer/process_context.h b/libs/audiographer/audiographer/process_context.h index 843c09b618..b73f5d69d5 100644 --- a/libs/audiographer/audiographer/process_context.h +++ b/libs/audiographer/audiographer/process_context.h @@ -43,7 +43,7 @@ public: /// Normal copy constructor ProcessContext (ProcessContext const & other) - : _data (other._data), _frames (other._frames), _channels (other._channels), _flags (other._flags) + : _data (other._data), _frames (other._frames), _channels (other._channels), _flags (other._flags) { /* No need to validate data */ } /// "Copy constructor" with unique data, frame and channel count, but copies flags diff --git a/libs/evoral/evoral/PatchChange.hpp b/libs/evoral/evoral/PatchChange.hpp index 576a1d2e98..aaa481516f 100644 --- a/libs/evoral/evoral/PatchChange.hpp +++ b/libs/evoral/evoral/PatchChange.hpp @@ -146,7 +146,9 @@ public: case 2: return _program_change; default: - assert (false); + abort (); + /*NOTREACHED*/ + return _program_change; } } diff --git a/libs/evoral/src/ControlSet.cpp b/libs/evoral/src/ControlSet.cpp index 6b793a08ea..bdc73121d2 100644 --- a/libs/evoral/src/ControlSet.cpp +++ b/libs/evoral/src/ControlSet.cpp @@ -32,7 +32,7 @@ ControlSet::ControlSet() { } -ControlSet::ControlSet (const ControlSet& other) +ControlSet::ControlSet (const ControlSet&) : noncopyable () { /* derived class must copy controls */ diff --git a/libs/evoral/src/Sequence.cpp b/libs/evoral/src/Sequence.cpp index fb10bfac7b..a7d58348e2 100644 --- a/libs/evoral/src/Sequence.cpp +++ b/libs/evoral/src/Sequence.cpp @@ -1249,7 +1249,7 @@ template void Sequence