Fixed: clang warnings

--HG--
branch : develop

code/nel/include/nel/gui/group_html.h

@@ -366,7 +366,7 @@ namespace NLGUI
 		bool			_Connecting;
 		double			_TimeoutValue;			// the timeout in seconds
 		double			_ConnectingTimeout;
-		uint32			_RedirectsRemaining;
+		sint			_RedirectsRemaining;
 
 		// minimal embeded lua script support
 		// Note : any embeded script is executed immediately after the closing

code/nel/src/logic/logic_state_machine.cpp

@@ -49,7 +49,7 @@ void xmlCheckNodeName (xmlNodePtr &node, const char *nodeName)
 		// Make an error message
 		char tmp[512];
 		smprintf (tmp, 512, "LogicStateMachine STATE_MACHINE XML Syntax error in block line %d, node %s should be %s",
-			(int)node->line, node->name, nodeName);
+			node ? (int)node->line:-1, node->name, nodeName);
 
 		nlinfo (tmp);
 		nlstop;

code/nel/src/misc/displayer.cpp

@@ -700,6 +700,9 @@ void CMsgBoxDisplayer::doDisplay ( const CLog::TDisplayInfo& args, const char *m
 #		endif
 				abort();
 				break;
+
+			default:
+				break;
 			}
 
 			// no more sent mail for crash

code/nel/src/pacs/global_retriever.cpp

@@ -784,7 +784,7 @@ bool			NLPACS::CGlobalRetriever::buildInstance(const string &id, const NLMISC::C
 	const CRetrieverInstance	&instance = makeInstance(retrieverId, 0, CVector(position));
 
 	// check make instance success
-	if (&instance == NULL || instance.getInstanceId() == -1 || instance.getRetrieverId() != retrieverId)
+	if (instance.getInstanceId() == -1 || instance.getRetrieverId() != retrieverId)
 		return false;
 
 	// links new instance to its neighbors

code/ryzom/client/src/sky_object.cpp

@@ -396,7 +396,7 @@ bool CSkyObject::setup(const CClientDate &date, const CClientDate &animationDate
 		for(uint k = 0; k < SKY_MAX_NUM_STAGE; ++k)
 		{
 			if (TexPanner[k].U != 0.f || TexPanner[k].V != 0.f ||
-				OffsetUBitmap != NULL || OffsetVBitmap != NULL )
+				OffsetUBitmap[k] != NULL || OffsetVBitmap[k] != NULL )
 			{
 				//nlinfo("global date = %f", animTime);
 				// there's tex panning for that stage

code/ryzom/common/src/game_share/utils.h

@@ -425,7 +425,8 @@ inline void CCallStackSingleton::setTopStackEntry(ICallStackEntry* newEntry)
 inline void CCallStackSingleton::display(NLMISC::CLog *log)
 {
 	nlassert(log!=NULL);
-	getTopStackEntry()->displayStack(*log);
+	ICallStackEntry *entry = getTopStackEntry();
+	if (entry) entry->displayStack(*log);
 	log->displayNL("");
 }
 
@@ -468,15 +469,13 @@ inline ICallStackEntry::~ICallStackEntry()
 inline void ICallStackEntry::displayStack(NLMISC::CLog& log) const
 {
 	// stop recursing when we reach a NULL object
-	// (this is implemented in this way in order to ximplify call code)
-	if (this==NULL)
-		return;
+	// (this is implemented in this way in order to simplify call code)
 
 	// display this entry
 	displayEntry(log);
 
 	// recurse through call stack
-	_Next->displayStack(log);
+	if (_Next) _Next->displayStack(log);
 }