Remove some flawed condition manipulation.

I think this stuff is unnecessary as wait() is interruptible
by boost::thread::interrupt.  Hence instead of setting a flag
then signalling the condition we can just do interrupt(), the
exception will be thrown and that's that.

1 files changed, 1 insertions, 4 deletions

diff --git a/src/lib/encoder.h b/src/lib/encoder.h
index 7e6fc4f30..6b830abba 100644
--- a/src/lib/encoder.h
+++ b/src/lib/encoder.h
@@ -55,7 +55,7 @@ class Encoder : public boost::noncopyable, public ExceptionStore
 {
 public:
 	Encoder (boost::shared_ptr<const Film>, boost::shared_ptr<Writer>);
-	virtual ~Encoder ();
+	~Encoder ();
 
 	/** Called to indicate that a processing run is about to begin */
 	void begin ();
@@ -94,9 +94,6 @@ private:
 	/** Current DCP frame index */
 	Frame _position;
 
-	/* XXX: probably should be atomic */
-	bool _terminate_enqueue;
-	bool _terminate_encoding;
 	/** Mutex for _threads */
 	mutable boost::mutex _threads_mutex;
 	std::list<boost::thread *> _threads;