Fix remaining compile warnings. #1555

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:warnings changing 2 files +13 −5

TheBlueMatt commented at 2:40 PM on July 4, 2012: member

(aside from some boost::thread ones on win32) Fixes: src/net.h: In member function ‘void CNode::EndMessage()’: src/net.h:374: warning: invalid access to non-static data member ‘CMessageHeader::nMessageSize’ of NULL object src/net.h:374: warning: (perhaps the ‘offsetof’ macro was used incorrectly) src/net.h:380: warning: invalid access to non-static data member ‘CMessageHeader::nChecksum’ of NULL object src/net.h:380: warning: (perhaps the ‘offsetof’ macro was used incorrectly) src/net.h:381: warning: invalid access to non-static data member ‘CMessageHeader::nChecksum’ of NULL object src/net.h:381: warning: (perhaps the ‘offsetof’ macro was used incorrectly)

Or, put better by clang++: src/net.h:377:46: warning: offset of on non-POD type 'CMessageHeader' [-Winvalid-offsetof]
Fix remaining warnings. 24154ed64b
gavinandresen commented at 2:55 PM on July 4, 2012: contributor

ACK
jgarzik referenced this in commit fe70b09c42 on Jul 4, 2012
jgarzik merged this on Jul 4, 2012
jgarzik closed this on Jul 4, 2012
nifgraup referenced this in commit f332a92182 on Mar 30, 2014
suprnurd referenced this in commit cc4db34f4d on Dec 5, 2017
DrahtBot locked this on Sep 8, 2021

Contributors