diff --git a/libretroshare/src/pqi/pqistreamer.cc b/libretroshare/src/pqi/pqistreamer.cc
index a17baf366..456848f32 100644
--- a/libretroshare/src/pqi/pqistreamer.cc
+++ b/libretroshare/src/pqi/pqistreamer.cc
@@ -323,9 +323,8 @@ int	pqistreamer::queue_outpqi(RsItem *pqi,uint32_t& pktsize)
 #endif
 
 	/* decide which type of packet it is */
-	RsFileData *dta = dynamic_cast<RsFileData *>(pqi);		// This is the old test method
 
-   pktsize = rsSerialiser->size(pqi);
+	pktsize = rsSerialiser->size(pqi);
 	void *ptr = malloc(pktsize);
 
 #ifdef DEBUG_PQISTREAMER
diff --git a/libretroshare/src/retroshare/rsplugin.h b/libretroshare/src/retroshare/rsplugin.h
index b42f57c8d..7caa7ea08 100644
--- a/libretroshare/src/retroshare/rsplugin.h
+++ b/libretroshare/src/retroshare/rsplugin.h
@@ -83,7 +83,7 @@ class RsPlugin
 		virtual MainPage       *qt_page()       		const	{ return NULL ; }
 		virtual QWidget        *qt_config_panel()		const	{ return NULL ; }
 		virtual QIcon          *qt_icon()       		const	{ return NULL ; }
-		virtual QTranslator    *qt_translator(QApplication *app, const QString& languageCode) const	{ return NULL ; }
+		virtual QTranslator    *qt_translator(QApplication */*app*/, const QString& /*languageCode*/) const	{ return NULL ; }
 
 		virtual std::string configurationFileName() const { return std::string() ; }
 		virtual std::string getShortPluginDescription() const = 0 ;
diff --git a/retroshare-gui/src/gui/common/RsCollectionDialog.cpp b/retroshare-gui/src/gui/common/RsCollectionDialog.cpp
index b99d20c18..15589de10 100644
--- a/retroshare-gui/src/gui/common/RsCollectionDialog.cpp
+++ b/retroshare-gui/src/gui/common/RsCollectionDialog.cpp
@@ -130,7 +130,7 @@ void RsCollectionDialog::updateSizes()
 	uint64_t total_size = 0 ;
 	uint32_t total_files = 0 ;
 
-	for(int i=0;i<_dlinfos.size();++i)
+	for(size_t i=0;i<_dlinfos.size();++i)
 		if(_cboxes[i]->isChecked())
 		{
 			total_size += _dlinfos[i].size ;
@@ -143,14 +143,14 @@ void RsCollectionDialog::updateSizes()
 void RsCollectionDialog::selectAll() const
 {
 	std::cerr << "Selecting all !" << std::endl;
-	for(int i=0;i<_dlinfos.size();++i)
+	for(size_t i=0;i<_dlinfos.size();++i)
 		_cboxes[i]->setChecked(true) ;
 }
 
 void RsCollectionDialog::deselectAll() const
 {
 	std::cerr << "Deselecting all !" << std::endl;
-	for(int i=0;i<_dlinfos.size();++i)
+	for(size_t i=0;i<_dlinfos.size();++i)
 		_cboxes[i]->setChecked(false) ;
 }