[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Patch - suspicious casts, suboptimal code

From: Dmitry <wipedout_at_yandex.ru>
Date: Fri, 02 Sep 2011 11:29:46 +0400

Hey.

Attached are two untested patches.

One addresses suspicious casts - casts where a pointer size type is casted to a 32-bit integer and then is implicitly converted back to a pointer size type. Such casts make code highly non-portable to 64 bit. I'm not sure whether all of them are that bad, but most are.

Another one addresses minor issues like code duplication, repeating checks, hardcoded constants, unneeded deep copying.

Best wishes.
Dmitry.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2834517

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].

Received on 2011-09-02 09:29:54 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.