On Thu, 16 Jul 2009, Oliver Stöneberg wrote:

> Compiling XMail with -Wextra produces this warning:
> 
> SysDepUnix.cpp: In function int SysSendFileMMap(SYS_SOCKET, const 
> char*, SYS_OFF_T, SYS_OFF_T, int):
> SysDepUnix.cpp:539: warning: comparison of unsigned expression < 0 is 
> always false
> 
> The affected line is:
> 
> if ((iCurrSend = SysSendData(SockFD, pszBuffer, iCurrSend, iTimeout)) 
> < 0) {
> 
> The issue is, that SysSendData returns an int, but iCurrSend is of 
> type size_t and so the negative value will become a positive one and 
> the errorhandling is never executed.

Thanks will fix. Looks strange since in XMail I use 'i' prefix for 'int'.


- Davide

_______________________________________________
xmail mailing list
xmail@xmailserver.org
http://xmailserver.org/mailman/listinfo/xmail

Reply via email to