Fix some compiling warnings and errors in OS X

git-svn-id: svn://localhost/ardour2/branches/3.0@4358 d708f5d6-7413-0410-9779-e7cbd77b26cf

4 files changed, 22 insertions, 21 deletions

diff --git a/libs/ardour/ardour/export_channel.h b/libs/ardour/ardour/export_channel.h
index d9e39c8850..213bd2d9fa 100644
--- a/libs/ardour/ardour/export_channel.h
+++ b/libs/ardour/ardour/export_channel.h
@@ -38,6 +38,9 @@ class Session;
 class ExportChannel
 {
   public:
+
+	virtual ~ExportChannel () {}
+
 	virtual void read (Sample * data, nframes_t frames) const = 0;
 	virtual bool empty () const = 0;
 	
@@ -69,13 +72,13 @@ class PortExportChannel : public ExportChannel
 
 	PortExportChannel () {}
 	
-	virtual void read (Sample * data, nframes_t frames) const;
-	virtual bool empty () const { return ports.empty(); }
+	void read (Sample * data, nframes_t frames) const;
+	bool empty () const { return ports.empty(); }
 	
-	virtual void get_state (XMLNode * node) const;
-	virtual void set_state (XMLNode * node, Session & session);
+	void get_state (XMLNode * node) const;
+	void set_state (XMLNode * node, Session & session);
 	
-	virtual bool operator< (ExportChannel const & other) const;
+	bool operator< (ExportChannel const & other) const;
 
 	void add_port (AudioPort * port) { ports.insert (port); }
 	PortSet const & get_ports () { return ports; }
@@ -126,12 +129,12 @@ class RegionExportChannel : public ExportChannel
 	friend class RegionExportChannelFactory;
 
   public:
-	virtual void read (Sample * data, nframes_t frames_to_read) const { factory.read (channel, data, frames_to_read); }
-	virtual void get_state (XMLNode * node) const {};
-	virtual void set_state (XMLNode * node, Session & session) {};
-	virtual bool empty () const { return false; }
+	void read (Sample * data, nframes_t frames_to_read) const { factory.read (channel, data, frames_to_read); }
+	void get_state (XMLNode * node) const {};
+	void set_state (XMLNode * node, Session & session) {};
+	bool empty () const { return false; }
 	// Region export should never have duplicate channels, so there need not be any semantics here
-	virtual bool operator< (ExportChannel const & other) const { return this < &other; }
+	bool operator< (ExportChannel const & other) const { return this < &other; }
 
   private:
 
diff --git a/libs/ardour/ardour/export_file_io.h b/libs/ardour/ardour/export_file_io.h
index 6705b7736a..0e15981c25 100644
--- a/libs/ardour/ardour/export_file_io.h
+++ b/libs/ardour/ardour/export_file_io.h
@@ -77,8 +77,9 @@ class SndfileWriterBase : public ExportFileWriter
 template <typename T>
 class SndfileWriter : public SndfileWriterBase, public GraphSink<T>
 {
-  protected:
+	// FIXME: having this protected doesn't work with Apple's gcc
 	// Should only be created vie ExportFileFactory and derived classes
+  public: // protected
 	friend class ExportFileFactory;
 	SndfileWriter (int channels, nframes_t samplerate, int format, string const & path);
 
diff --git a/libs/ardour/export_file_io.cc b/libs/ardour/export_file_io.cc
index d4aa1870ba..793b1dbea4 100644
--- a/libs/ardour/export_file_io.cc
+++ b/libs/ardour/export_file_io.cc
@@ -400,8 +400,6 @@ ExportFileFactory::create_sndfile (FormatPtr format, unsigned int channels, ustr
 	typedef boost::shared_ptr<SndfileWriter<int> > IntWriterPtr;
 	typedef boost::shared_ptr<SndfileWriter<float> > FloatWriterPtr;
 	
-	FilePair ret;
-
 	int real_format = format->format_id() | format->sample_format() | format->endianness();
 
 	uint32_t data_width = sndfile_data_width (real_format);
@@ -422,14 +420,13 @@ ExportFileFactory::create_sndfile (FormatPtr format, unsigned int channels, ustr
 		
 		return std::make_pair (boost::static_pointer_cast<FloatSink> (sfc), boost::static_pointer_cast<ExportFileWriter> (sfw));
 
-	} else {
-	
-		FloatConverterPtr sfc = FloatConverterPtr (new SampleFormatConverter<float> (channels, format->dither_type(), data_width));
-		FloatWriterPtr sfw = FloatWriterPtr (new SndfileWriter<float> (channels, format->sample_rate(), real_format, filename));
-		sfc->pipe_to (sfw);
-		
-		return std::make_pair (boost::static_pointer_cast<FloatSink> (sfc), boost::static_pointer_cast<ExportFileWriter> (sfw));;
 	}
+	
+	FloatConverterPtr sfc = FloatConverterPtr (new SampleFormatConverter<float> (channels, format->dither_type(), data_width));
+	FloatWriterPtr sfw = FloatWriterPtr (new SndfileWriter<float> (channels, format->sample_rate(), real_format, filename));
+	sfc->pipe_to (sfw);
+	
+	return std::make_pair (boost::static_pointer_cast<FloatSink> (sfc), boost::static_pointer_cast<ExportFileWriter> (sfw));
 }
 
 bool
diff --git a/libs/ardour/midi_buffer.cc b/libs/ardour/midi_buffer.cc
index 705b94f422..1f5b0c5fa9 100644
--- a/libs/ardour/midi_buffer.cc
+++ b/libs/ardour/midi_buffer.cc
@@ -74,7 +74,7 @@ MidiBuffer::resize (size_t size)
 	_capacity = size;
 
 #ifdef NO_POSIX_MEMALIGN
-	_events = (Evoral::Event *) malloc(sizeof(Evoral::Event) * _capacity);
+	_events = (Evoral::MIDIEvent *) malloc(sizeof(Evoral::MIDIEvent) * _capacity);
 	_data = (uint8_t *) malloc(sizeof(uint8_t) * _capacity * MAX_EVENT_SIZE);
 #else
 	posix_memalign((void**)&_events, CPU_CACHE_ALIGN, sizeof(Evoral::Event) * _capacity);