diff --git a/src/core/archive.cpp b/src/core/archive.cpp
index 0b6a5a7..fdcbc3e 100644
--- a/src/core/archive.cpp
+++ b/src/core/archive.cpp
@@ -883,11 +883,10 @@ void cLockedTemporaryFileArchive::OpenReadWrite( const TCHAR* filename, uint32 o
             strTempFile += _T("twtempXXXXXX");  
             iFSServices::GetInstance()->MakeTempFilename( strTempFile );
           }
-        catch( eFSServices& e)
+        catch( eFSServices& fileError)
           {
             TSTRING errStr = TSS_GetString( cCore, core::STR_BAD_TEMPDIRECTORY );
-            eArchiveOpen e(strTempFile, errStr);
-            throw e;
+            throw eArchiveOpen(strTempFile, errStr);
           }
       }
     ///////////////////////////////////////////////////////////////////////////////
diff --git a/src/core/displayencoder.cpp b/src/core/displayencoder.cpp
index b680f7d..a9256ee 100644
--- a/src/core/displayencoder.cpp
+++ b/src/core/displayencoder.cpp
@@ -817,8 +817,9 @@ void cEncoder::ValidateSchema() const
 bool cEncoder::OnlyOneCatagoryPerChar() const
 {
     // TODO:BAM - man, is there a better way to do this?
-    TCHAR ach[2] = {0};
     TCHAR ch = std::numeric_limits<TCHAR>::min();
+    TSTRING ach(1,ch);
+
     if( ch != std::numeric_limits<TCHAR>::max() )
     {
         do
@@ -828,8 +829,7 @@ bool cEncoder::OnlyOneCatagoryPerChar() const
             ach[0] = ch;
             for( sack_type::const_iterator atE = m_encodings.begin(); atE != m_encodings.end(); atE++ )
             {
-		TSTRING::const_iterator first(&ach[0]), last(&ach[1]);
-                if( (*atE)->NeedsEncoding( first, last ) )
+	        if( (*atE)->NeedsEncoding( ach.begin(), ach.end() ) )
                 {
                     if( fFailedATest )
                         return false; // each char can only fail one test
diff --git a/src/cryptlib/algebra.h b/src/cryptlib/algebra.h
index fdcd543..6fe9d66 100644
--- a/src/cryptlib/algebra.h
+++ b/src/cryptlib/algebra.h
@@ -55,7 +55,7 @@ template <class T> class AbstractField : public AbstractRing<T>
 {
 public:
 	bool IsUnit(const typename T::Element &a) const
-		{return !Equal(a, this->Zero());}
+		{return !this->Equal(a, this->Zero());}
 };
 
 template <class T> class AbstractEuclideanDomain : public AbstractRing<T>
@@ -273,7 +273,7 @@ template <class T> T AbstractEuclideanDomain<T>::Gcd(const Element &a, const Ele
 	Element g[3]={b, a};
 	unsigned int i0=0, i1=1, i2=2;
 
-	while (!Equal(g[i1], this->Zero()))
+	while (!this->Equal(g[i1], this->Zero()))
 	{
 		g[i2] = Mod(g[i0], g[i1]);
 		unsigned int t = i0; i0 = i1; i1 = i2; i2 = t;
@@ -289,7 +289,7 @@ template <class T> typename QuotientRing<T>::Element QuotientRing<T>::Multiplica
 	typename T::Element y;
 	unsigned int i0=0, i1=1, i2=2;
 
-	while (!Equal(g[i1], Zero()))
+	while (!this->Equal(g[i1], Zero()))
 	{
 		// y = g[i0] / g[i1];
 		// g[i2] = g[i0] % g[i1];
diff --git a/src/fco/fcospec.cpp b/src/fco/fcospec.cpp
index 2d8a09f..d9de8bb 100644
--- a/src/fco/fcospec.cpp
+++ b/src/fco/fcospec.cpp
@@ -51,6 +51,7 @@
 class cDefaultSpecMask : public iFCOSpecMask
 {
 public:
+        cDefaultSpecMask() {}
 	virtual const TSTRING& GetName() const;
 	virtual bool Accept(const iFCO* pFCO) const;
 private:
diff --git a/src/twadmin/twadmincl.cpp b/src/twadmin/twadmincl.cpp
index 5821487..abce41b 100644
--- a/src/twadmin/twadmincl.cpp
+++ b/src/twadmin/twadmincl.cpp
@@ -34,6 +34,7 @@
 //
 
 #include "stdtwadmin.h"
+#include <unistd.h>
 
 #include "twadmincl.h"
 #include "twadminstrings.h"