Enable -Wall for Linux builds

Since the code is based in Windows/Visual C++ the only warning that had
to be specifically disabled is -Wunknown-pragmas.  The remaining
warnings were cleaned up in the code.

msbuild.proj

@@ -20,7 +20,7 @@
 
     <PropertyGroup>
       <BashExe>%WINDIR%\Sysnative\bash.exe</BashExe>
-      <CPPFLAGS>-I../external-kodi-addon-dev-kit/kodi -I../prebuilt-libcurl/include/curl -I../prebuilt-libuuid/include -I../external-sqlite -Itmp/version</CPPFLAGS>
+      <CPPFLAGS>-Wall -Wno-unknown-pragmas -I../external-kodi-addon-dev-kit/kodi -I../prebuilt-libcurl/include/curl -I../prebuilt-libuuid/include -I../external-sqlite -Itmp/version</CPPFLAGS>
       <CFLAGS>-fPIC</CFLAGS>
       <CXXFLAGS>-fPIC -std=c++14</CXXFLAGS>
       <PackageVersion>$([System.IO.File]::ReadAllText($(SolutionDir)tmp\version\version.txt))</PackageVersion>

src/livestream.h

@@ -132,8 +132,8 @@ class livestream
 
 	// RING BUFFER
 	//
-	std::unique_ptr<uint8_t[]>	m_buffer;					// Ring buffer stroage
 	size_t const				m_buffersize;				// Size of the ring buffer
+	std::unique_ptr<uint8_t[]>	m_buffer;					// Ring buffer stroage
 	bool						m_bufferempty = true;		// Flag for empty ring buffer
 	bool						m_bufferfull = false;		// Flag for full ring buffer
 	std::atomic<size_t>			m_bufferhead{0};			// Head (write) buffer position

src/pvr.cpp

@@ -798,6 +798,7 @@ static void log_message(addoncallbacks::addon_log_t level, _args&&... args)
 {
 	std::ostringstream stream;
 	int unpack[] = {0, ( static_cast<void>(stream << args), 0 ) ... };
+	(void)unpack;
 
 	if(g_addon) g_addon->Log(level, stream.str().c_str());
 	if (level == addoncallbacks::addon_log_t::LOG_ERROR) fprintf(stderr, "ERROR: %s\r\n", stream.str().c_str());

src/string_exception.h

@@ -79,6 +79,7 @@ class string_exception : public std::exception
 	{
 		std::ostringstream stream;
 		int unpack[] = {0, ( static_cast<void>(stream << args), 0 ) ... };
+		(void)unpack;
 
 		return stream.str();
 	}