Suppress warnings
authorTimePath <andrew.hardaker1995@gmail.com>
Sun, 6 Aug 2017 04:19:41 +0000 (14:19 +1000)
committerTimePath <andrew.hardaker1995@gmail.com>
Sun, 6 Aug 2017 04:19:41 +0000 (14:19 +1000)
libs/profile/file.cpp
libs/splines/math_matrix.h
libs/stream/stringstream.h
libs/xml/xmltextags.cpp
radiant/environment.cpp
radiant/environment.h
radiant/main.cpp
radiant/texwindow.cpp

index ec23096..9f83393 100644 (file)
@@ -271,7 +271,7 @@ int MemStream::Seek( offset_type lOff, int nFrom ){
                lNewPos = m_nFileSize + lOff;
        }
        else{
-               return ( position_type ) - 1;
+               return -1;
        }
 
        m_nPosition = lNewPos;
index 597f1d0..306b9bc 100644 (file)
@@ -83,8 +83,8 @@ friend mat3_t   SkewSymmetric( idVec3 const &src );
 ID_INLINE mat3_t::mat3_t() {
 }
 
-ID_INLINE mat3_t::mat3_t( float src[ 3 ][ 3 ] ) {
-       memcpy( mat, src, sizeof( src ) );
+ID_INLINE mat3_t::mat3_t(float src[3][3]) {
+    memcpy(mat, src, sizeof(float) * 3 * 3);
 }
 
 ID_INLINE mat3_t::mat3_t( idVec3 const &x, idVec3 const &y, idVec3 const &z ) {
index 386093f..8511bd4 100644 (file)
@@ -105,7 +105,7 @@ typedef StringBuffer::const_iterator const_iterator;
 
 StringOutputStream(){
 }
-StringOutputStream( std::size_t capacity ) : m_string( capacity ){
+explicit StringOutputStream( std::size_t capacity ) : m_string( capacity ){
 }
 std::size_t write( const char* buffer, std::size_t length ){
        m_string.push_range( buffer, buffer + length );
index c9d2b06..00f57a0 100644 (file)
@@ -476,7 +476,7 @@ void XmlTagBuilder::GetShaderTags( const char* shader, std::vector<CopiedString>
           returns a vector containing the tags
         */
 
-       charexpression;
+       char const *expression;
 
        if ( shader == NULL ) { // get all tags from all shaders
                expression = "/root/*/*/tag";
@@ -510,7 +510,7 @@ void XmlTagBuilder::GetUntagged( std::set<CopiedString>& shaders ){
           returns a set containing the shaders (with path)
         */
 
-       charexpression = "/root/*/*[not(child::tag)]";
+       char const *expression = "/root/*/*[not(child::tag)]";
 
        xmlXPathObjectPtr xpathPtr = XpathEval( expression );
        xmlNodeSetPtr nodePtr;
@@ -540,7 +540,7 @@ void XmlTagBuilder::GetAllTags( std::set<CopiedString>& tags ){
           returns a set containing all used tags
         */
 
-       charexpression = "/root/*/*/tag";
+       char const *expression = "/root/*/*/tag";
 
        xmlXPathObjectPtr xpathPtr = XpathEval( expression );
        xmlNodeSetPtr nodePtr;
index 4e15c81..a28524e 100644 (file)
@@ -30,9 +30,9 @@
 #include "cmdlib.h"
 
 int g_argc;
-char** g_argv;
+char const** g_argv;
 
-void args_init( int argc, char* argv[] ){
+void args_init( int argc, char const* argv[] ){
        int i, j, k;
 
        for ( i = 1; i < argc; i++ )
@@ -54,8 +54,8 @@ void args_init( int argc, char* argv[] ){
        g_argv = argv;
 }
 
-char *gamedetect_argv_buffer[1024];
-void gamedetect_found_game( char *game, char *path ){
+char const *gamedetect_argv_buffer[1024];
+void gamedetect_found_game( char const *game, char *path ){
        int argc;
        static char buf[128];
 
@@ -79,7 +79,7 @@ void gamedetect_found_game( char *game, char *path ){
        g_argv = gamedetect_argv_buffer;
 }
 
-bool gamedetect_check_game( char *gamefile, const char *checkfile1, const char *checkfile2, char *buf /* must have 64 bytes free after bufpos */, int bufpos ){
+bool gamedetect_check_game( char const *gamefile, const char *checkfile1, const char *checkfile2, char *buf /* must have 64 bytes free after bufpos */, int bufpos ){
        buf[bufpos] = '/';
 
        strcpy( buf + bufpos + 1, checkfile1 );
@@ -196,7 +196,7 @@ const char* LINK_NAME =
 ;
 
 /// brief Returns the filename of the executable belonging to the current process, or 0 if not found.
-char* getexename( char *buf ){
+char const* getexename( char *buf ){
        /* Now read the symbolic link */
        int ret = readlink( LINK_NAME, buf, PATH_MAX );
 
@@ -224,7 +224,7 @@ char* getexename( char *buf ){
        return buf;
 }
 
-void environment_init( int argc, char* argv[] ){
+void environment_init( int argc, char const* argv[] ){
        // Give away unnecessary root privileges.
        // Important: must be done before calling gtk_init().
        char *loginname;
index 7ee16c6..b2828f8 100644 (file)
 #if !defined( INCLUDED_ENVIRONMENT_H )
 #define INCLUDED_ENVIRONMENT_H
 
-void environment_init( int argc, char* argv[] );
+void environment_init( int argc, char const* argv[] );
 const char* environment_get_home_path();
 const char* environment_get_app_path();
 
 extern int g_argc;
-extern char** g_argv;
+extern char const** g_argv;
 
 #endif
index 93de25c..d80fd0b 100644 (file)
@@ -593,7 +593,7 @@ int main( int argc, char* argv[] ){
 
        GlobalDebugMessageHandler::instance().setHandler( GlobalPopupDebugMessageHandler::instance() );
 
-       environment_init( argc, argv );
+       environment_init(argc, (char const **) argv);
 
        paths_init();
 
index 7d1782e..8ee515f 100644 (file)
@@ -1989,7 +1989,7 @@ void TextureBrowser_SetNotex(){
        name << GlobalRadiant().getAppPath() << "bitmaps/" NOTEX_BASENAME ".png";
        g_notex = name.c_str();
 
-       name = NULL;
+       name = StringOutputStream(256);
        name << GlobalRadiant().getAppPath() << "bitmaps/" SHADERNOTEX_BASENAME " .png";
        g_shadernotex = name.c_str();
 }