{"id":224371,"date":"2014-05-28T09:39:03","date_gmt":"2014-05-28T05:39:03","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=224371"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=224371","title":{"rendered":"CppCat \u043f\u0440\u043e\u0432\u0435\u0440\u0435\u044f\u0435\u0442 OpenMW: \u0432\u043e \u0432\u0441\u0435\u043b\u0435\u043d\u043d\u043e\u0439 Morrowind \u043d\u0435 \u0432\u0441\u0451 \u0433\u043b\u0430\u0434\u043a\u043e<\/span>"},"content":{"rendered":"
\n
\"CppCat<\/div>\n

\u042f \u043f\u0440\u043e\u0432\u0435\u0440\u0438\u043b \u043f\u0440\u043e\u0435\u043a\u0442 OpenMW \u0441 \u043f\u043e\u043c\u043e\u0449\u044c\u044e CppCat \u0438 \u043d\u0430\u043f\u0438\u0441\u0430\u043b \u044d\u0442\u0443 \u043a\u0440\u043e\u0448\u0435\u0447\u043d\u0443\u044e \u0441\u0442\u0430\u0442\u044c\u044e. \u041d\u0430\u0448\u043b\u043e\u0441\u044c \u0441\u043b\u0438\u0448\u043a\u043e\u043c \u043c\u0430\u043b\u043e \u043e\u0448\u0438\u0431\u043e\u043a. \u041d\u043e \u043c\u0435\u043d\u044f \u043f\u0440\u043e\u0441\u0438\u043b\u0438 \u043d\u0430\u043f\u0438\u0441\u0430\u0442\u044c \u043f\u0440\u043e \u044d\u0442\u043e \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u044d\u0442\u043e\u0433\u043e \u043f\u0440\u043e\u0435\u043a\u0442\u0430 \u0441\u0442\u0430\u0442\u044c\u044e, \u0438 \u0432\u043e\u0442 \u043e\u043d\u0430.
<\/a> <\/p>\n

OpenMW<\/h2>\n

OpenMW \u2014 \u044d\u0442\u043e \u043f\u043e\u043f\u044b\u0442\u043a\u0430 \u0432\u043e\u0441\u0441\u043e\u0437\u0434\u0430\u0442\u044c \u043f\u043e\u043f\u0443\u043b\u044f\u0440\u043d\u0443\u044e RPG Morrowind, \u043f\u043e\u043b\u043d\u043e\u0446\u0435\u043d\u043d\u0430\u044f \u0440\u0435\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u044f \u0432\u0441\u0435\u0445 \u043e\u0441\u043e\u0431\u0435\u043d\u043d\u043e\u0441\u0442\u0435\u0439 \u0438\u0433\u0440\u044b \u0441 \u043e\u0442\u043a\u0440\u044b\u0442\u044b\u043c \u0438\u0441\u0445\u043e\u0434\u043d\u044b\u043c \u043a\u043e\u0434\u043e\u043c. \u0414\u043b\u044f \u0437\u0430\u043f\u0443\u0441\u043a\u0430 OpenMW \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u0442\u044c\u0441\u044f \u043e\u0440\u0438\u0433\u0438\u043d\u0430\u043b\u044c\u043d\u044b\u0439 \u0434\u0438\u0441\u043a Morrowind.<\/p>\n

\u0418\u0441\u0445\u043e\u0434\u043d\u044b\u0439 \u043a\u043e\u0434 \u0434\u043e\u0441\u0442\u0443\u043f\u0435\u043d \u043f\u043e \u0430\u0434\u0440\u0435\u0441\u0443: https:\/\/code.google.com\/p\/openmw\/<\/a><\/p>\n

CppCat<\/h2>\n

CppCat \u2014 \u043c\u043b\u0430\u0434\u0448\u0438\u0439 \u0431\u0440\u0430\u0442 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0433\u043e \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 PVS-Studio. \u041e\u0431 \u043e\u0442\u043b\u0438\u0447\u0438\u044f\u0445: "\u0421\u0440\u0430\u0432\u043d\u0435\u043d\u0438\u0435 \u0432\u043e\u0437\u043c\u043e\u0436\u043d\u043e\u0441\u0442\u0435\u0439 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0445 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u0432 \u043a\u043e\u0434\u0430 PVS-Studio \u0438 CppCat<\/a>".<\/p>\n

\u041d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0435 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0435 \u043c\u0435\u0441\u0442\u0430<\/h2>\n

\u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N1<\/b> <\/p>\n

std::string getUtf8(unsigned char c,   ToUTF8::Utf8Encoder& encoder, ToUTF8::FromType encoding) {   ....   conv[0xa2] = 0xf3;   conv[0xa3] = 0xbf;   conv[0xa4] = 0x0;   conv[0xe1] = 0x8c;   conv[0xe1] = 0x8c;   <<<<====   conv[0xe3] = 0x0;   .... }<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V519 The ‘conv[0xe1]’ variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 103, 104. openmw fontloader.cpp 104<\/p>\n
  \u041d\u0430\u0432\u0435\u0440\u043d\u043e\u0435, \u044d\u0442\u043e \u043e\u043f\u0435\u0447\u0430\u0442\u043a\u0430. \u0412 \u0432\u044b\u0434\u0435\u043b\u0435\u043d\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u043a\u0435, \u0441\u043a\u043e\u0440\u0435\u0435 \u0432\u0441\u0435\u0433\u043e \u0434\u043e\u043b\u0436\u0435\u043d \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c\u0441\u044f \u0438\u043d\u0434\u0435\u043a\u0441 0xe2.<\/p>\n
  \u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N2<\/b>   <\/p>\n
enum Flags {   ....   NoDuration = 0x4,   .... }  bool CastSpell::cast (const ESM::Ingredient* ingredient) {   ....   if (!magicEffect->mData.mFlags & ESM::MagicEffect::NoDuration)   .... }<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V564 The ‘&’ operator is applied to bool type value. You’ve probably forgotten to include parentheses or intended to use the ‘&&’ operator. openmw spellcasting.cpp 717<\/p>\n
  \u041e\u0448\u0438\u0431\u043a\u0430 \u0441\u0432\u044f\u0437\u0430\u043d\u0430 \u0441 \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442\u043e\u043c \u043e\u043f\u0435\u0440\u0430\u0446\u0438\u0439. \u0412 \u043d\u0430\u0447\u0430\u043b\u0435, \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0435 (!magicEffect->mData.mFlags). \u0420\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442: 0 \u0438\u043b\u0438 1. \u0417\u0430\u0442\u0435\u043c \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0435: 0 & 4 \u0438\u043b\u0438 1 & 4. \u042d\u0442\u043e \u043d\u0435 \u0438\u043c\u0435\u0435\u0442 \u043d\u0438\u043a\u0430\u043a\u043e\u0433\u043e \u043f\u0440\u0430\u043a\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0433\u043e \u0441\u043c\u044b\u0441\u043b\u0430. \u0421\u043a\u043e\u0440\u0435\u0435 \u0432\u0441\u0435\u0433\u043e, \u043a\u043e\u0434 \u0434\u043e\u043b\u0436\u0435\u043d \u0432\u044b\u0433\u043b\u044f\u0434\u0435\u0442\u044c \u0442\u0430\u043a:  <\/p>\n
if ( ! (magicEffect->mData.mFlags & ESM::MagicEffect::NoDuration) )<\/code><\/pre>\n
  \u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N3<\/b>   <\/p>\n
void Clothing::blank() {   mData.mType = 0;   mData.mWeight = 0;   mData.mValue = 0;   mData.mEnchant = 0;   mParts.mParts.clear();   mName.clear();   mModel.clear();   mIcon.clear();   mIcon.clear();   mEnchant.clear();   mScript.clear(); }<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V586 The ‘clear’ function is called twice for deallocation of the same resource. Check lines: 48, 49. components loadclot.cpp 49<\/p>\n
  \u0414\u0432\u0430 \u0440\u0430\u0437\u0430 \u043e\u0447\u0438\u0449\u0430\u0435\u0442\u0441\u044f \u043e\u0431\u044a\u0435\u043a\u0442 ‘mIcon’. \u0412\u0442\u043e\u0440\u0430\u044f \u043e\u0447\u0438\u0441\u0442\u043a\u0430 \u043b\u0438\u0448\u043d\u044f\u044f \u0438\u043b\u0438 \u043d\u0430\u0434\u043e \u0431\u044b\u043b\u043e \u043e\u0447\u0438\u0441\u0442\u0438\u0442\u044c \u0447\u0442\u043e-\u0442\u043e \u0435\u0449\u0451.<\/p>\n
  \u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N4<\/b>   <\/p>\n
void Storage::loadDataFromStream(   ContainerType& container, std::istream& stream) {   std::string line;   while (!stream.eof())   {     std::getline( stream, line );     ....   }   .... }<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V663 Infinite loop is possible. The ‘cin.eof()’ condition is insufficient to break from the loop. Consider adding the ‘cin.fail()’ function call to the conditional expression. components translation.cpp 45<\/p>\n
  \u041f\u0440\u0438 \u0440\u0430\u0431\u043e\u0442\u0435 \u0441 \u043a\u043b\u0430\u0441\u0441\u043e\u043c ‘std::istream’ \u043d\u0435\u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0432\u044b\u0437\u043e\u0432\u0430 \u0444\u0443\u043d\u043a\u0446\u0438\u0438 ‘eof()’ \u0434\u043b\u044f \u0437\u0430\u0432\u0435\u0440\u0448\u0435\u043d\u0438\u044f \u0446\u0438\u043a\u043b\u0430. \u0412 \u0441\u043b\u0443\u0447\u0430\u0435 \u0432\u043e\u0437\u043d\u0438\u043a\u043d\u043e\u0432\u0435\u043d\u0438\u044f \u0441\u0431\u043e\u044f \u043f\u0440\u0438 \u0447\u0442\u0435\u043d\u0438\u0438 \u0434\u0430\u043d\u043d\u044b\u0445, \u0432\u044b\u0437\u043e\u0432 \u0444\u0443\u043d\u043a\u0446\u0438\u0438 ‘eof()’ \u0431\u0443\u0434\u0435\u0442 \u0432\u0441\u0435\u0433\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 ‘false’. \u0414\u043b\u044f \u0437\u0430\u0432\u0435\u0440\u0448\u0435\u043d\u0438\u044f \u0446\u0438\u043a\u043b\u0430 \u0432 \u044d\u0442\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u043d\u0435\u043e\u0431\u0445\u043e\u0434\u0438\u043c\u0430 \u0434\u043e\u043f\u043e\u043b\u043d\u0438\u0442\u0435\u043b\u044c\u043d\u0430\u044f \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f, \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u043c\u043e\u0433\u043e \u0444\u0443\u043d\u043a\u0446\u0438\u0435\u0439 ‘fail()’.<\/p>\n
  \u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N5<\/b>  <\/p>\n
class Factory {   ....   bool getReadSourceCache() { return mReadSourceCache; }   bool getWriteSourceCache() { return mReadSourceCache; }   ....   bool mReadSourceCache;   bool mWriteSourceCache;   .... };<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V524 It is odd that the body of ‘getWriteSourceCache’ function is fully equivalent to the body of ‘getReadSourceCache’ function. components factory.hpp 209<\/p>\n
  \u041c\u043d\u0435 \u043a\u0430\u0436\u0435\u0442\u0441\u044f, \u0444\u0443\u043d\u043a\u0446\u0438\u044f getWriteSourceCache() \u0434\u043e\u043b\u0436\u043d\u0430 \u0431\u044b\u0442\u044c \u0442\u0430\u043a\u043e\u0439:  <\/p>\n
bool getWriteSourceCache() { return mWriteSourceCache; }<\/code><\/pre>\n
  \u0424\u0440\u0430\u0433\u043c\u0435\u043d\u0442 N6, N7, N8<\/b>  <\/p>\n
std::string rangeTypeLabel(int idx) {   const char* rangeTypeLabels [] = {     "Self",     "Touch",     "Target"   };   if (idx >= 0 && idx <= 3)     return rangeTypeLabels[idx];   else     return "Invalid"; }<\/code><\/pre>\n
  \u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 CppCat: V557 Array overrun is possible. The value of ‘idx’ index could reach 3. esmtool labels.cpp 502<\/p>\n
  \u041d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u0430\u044f \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0438\u043d\u0434\u0435\u043a\u0441\u0430 \u043c\u0430\u0441\u0441\u0438\u0432\u0430. \u0415\u0441\u043b\u0438 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f ‘idx’ \u0431\u0443\u0434\u0435\u0442 \u0440\u0430\u0432\u043d\u0430 3, \u0442\u043e \u043f\u0440\u043e\u0438\u0437\u043e\u0439\u0434\u0443\u0442 \u0432\u044b\u0445\u043e\u0434 \u0437\u0430 \u0433\u0440\u0430\u043d\u0438\u0446\u0443 \u043c\u0430\u0441\u0441\u0438\u0432\u0430.<\/p>\n
  \u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e:  <\/p>\n
if (idx >= 0 && idx < 3)<\/code><\/pre>\n
  \u0410\u043d\u0430\u043b\u043e\u0433\u0438\u0447\u043d\u0430\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u0432\u0441\u0442\u0440\u0435\u0442\u0438\u043b\u0438\u0441\u044c \u0435\u0449\u0451 \u0432 \u0434\u0432\u0443\u0445 \u043c\u0435\u0441\u0442\u0430\u0445:<\/p>\n