I really feel sorry for Mike, improvements such as gcc4.0, vs2005, and
-Wall, would really drive me insane.
On 9/3/06, [EMAIL PROTECTED]
[EMAIL PROTECTED] wrote:
I think the stress must've driven him to an early retirement.
At 2006/09/03 08:15 AM, Teddy wrote:
Any updates on this Mike? It's
This is almost certainly a bug caused by a change I made. It most like
related to some linker funkiness with unused modules being removed at
link time by the compiler. I'll investigate on our end and post a fix
for you guys when I'm back in the office later this week. If you look
at the macros,
Well that -Wall stuff should be easy, I already posted patches that do a lot of
that, and it's all trivial - would take a few hours to fix them all. The
reason I haven't done more recently however is I'm worried about the increased
difficulty in merging future Valve changes if Valve doesn't
It's not clear to me what this code is trying to accomplish, but memcmping a
struct is of course always dangerous and never a good idea. In this case, it's
actually memcmping an inhereted class which is even worse, as the vtable
pointer gets involved, causing the memcmp to seemingly never
This is a bug, the line should read:
if ( Q_memcmp( fog,
CFogController::s_pFogController-m_fog, sizeof(fog) ))
Jay
-Original Message-
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of
[EMAIL PROTECTED]
Sent: Monday, September 04, 2006 11:48 AM
To:
I agree more documentation of the closed source API's would be nice.
On 9/3/06, [EMAIL PROTECTED]
[EMAIL PROTECTED] wrote:
There's already a MDLCACHE_CRITICAL_SECTION(); in C_BaseEntity::PostDataUpdate
that should take care of that one. Post a stack trace if you're certain you
haven't
--
[ Picked text/plain from multipart/alternative ]
Any documentation would be nice.. I haven't seen many so far, except the
fixme!-comments
But well.. Every programmer hates to write documentation...
On 9/4/06, Nick [EMAIL PROTECTED] wrote:
I agree more documentation of the closed source
Well I think you meant:
if (Q_memcmp(fog, CFogController::s_pFogController-m_fog, sizeof(fog)))
which certainly makes more sense than what was there before. However that
memcmp is still never going to return 0, because you're still memcmp'ing two
objects with vtable entries, due to virtual
8 matches
Mail list logo
