Как исправить ошибку при компиляции C++ кода?

Question

[Nightmare A]

При сборке проекта возникает эта ошибка . Ума не приложу, не знаю как исправить.

../deps/source-sdk-2013/mp/src/public/tier1/utlmemory.h:915:2: note: in expansion of macro ‘COMPILE_TIME_ASSERT’
  915 |  COMPILE_TIME_ASSERT( (nAlignment & (nAlignment-1)) == 0 );
      |  ^~~~~~~~~~~~~~~~~~~
In file included from ../deps/source-sdk-2013/mp/src/public/mathlib/vector.h:30,
                 from ../deps/source-sdk-2013/mp/src/public/datamap.h:15,
                 from ../deps/source-sdk-2013/mp/src/public/tier1/byteswap.h:13,
                 from ../src/Machinarium/Utils/String.cpp:132:
../deps/cryptopp/cryptlib.h: At global scope:
../deps/cryptopp/cryptlib.h:1200:22: error: expected unqualified-id before ‘const’
 1200 |  virtual bool Verify(const byte *digest)
      |                      ^~~~~
../deps/source-sdk-2013/mp/src/public/tier0/dbg.h:372:35: note: in definition of macro ‘Verify’
  372 | #define  Verify( _exp )          (_exp)
      |                                   ^~~~
compilation terminated due to -Wfatal-errors.
make[1]: *** [Garrys Mod 10.make:355: obj/Release/Garrys Mod 10/String.o] Error 1
make: *** [Makefile:51: Garrys Mod 10] Error 2

nightmare@nightmare-desktop:/media/nightmare/2DEDC4C311647F4A/Workspace/cpp.machinarium/projects$ g++ --version
g++ (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Copyright (C) 2019 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Answer 1

[Mercury13]

Тут поцапались макрос Verify в dbg.h и функция Verify в cryptlib.h.
Я бы предпочёл переименовать одно из двух — лучше макрос — а затем разобрать, где должен быть макрос, а где функция.

Виноват определённо разработчик макроса: он назвал его простым и распространённым словом Verify. Функция-то где-то внутри объекта, а макрос заменяет идентификатор Verify на всякую дрянь независимо от того, где этот идентификатор и в каком пространстве имён. Кроме того, подобные макросы принято называть как-нибудь DBG_VERIFY — если только это не макрос совместимости, подменяющий, например, более позднее nullptr в старое NULL.

Comments

[Nightmare A]

Решение не работает.

[Mercury13]

Вадим Ушаков, Макросы совместимости я только привёл как пример. Они вам не нужны.
Макрос Verify надо переименовать в DBG_VERIFY

[Mercury13]

Вадим Ушаков, Повторяю, как они поцапались… Код
virtual bool Verify(const byte *digest)

макрос Verify развернул в…
virtual bool (const byte *digest)

Так что одна библиотека просто несовместима с другой. По вине разработчика макроса Verify.

[Mercury13]

Вадим Ушаков, Эту несовместимость можно разрулить, переименовав один из Verify.

[Nightmare A]

Mercury13, Каким образом это лучше сделать, если в точке использования два определения? Как вариант можно использовать переименование при включении, но это не всегда возможно при использовании внешних зависимостей без редактирования исходного кода внешних зависимостей, т.е. при нарушении целостности относительно мысли автора кода не всегда удобно переносить разработку без изменения зависимостей, если не переносить без зависимостей вместе с проектом, что может быть накладно.

[Mercury13]

Вадим Ушаков, Тут сильно зависит от того, что и как включается и как используется чёртов макрос.
Ещё один вариант. Подключить одно, #undef Verify, подключить другое. Сработает негарантированно.

[Nightmare A]

Mercury13,
Ошибка

In file included from ../deps/cryptopp/secblock.h:11,
                 from ../deps/cryptopp/md2.h:11,
                 from ../src/Machinarium/Utils/String.cpp:138:
../deps/cryptopp/misc.h: In function ‘CryptoPP::word16 CryptoPP::ByteReverse(CryptoPP::word16)’:
../deps/cryptopp/misc.h:2033:9: error: ‘bswap_16’ was not declared in this scope
 2033 |  return bswap_16(value);

Возникла после того как я дописал

#undef Verify

/// \brief Interface for hash functions and data processing part of MACs
/// \details HashTransformation objects are stateful. They are created in an initial state,
///  change state as Update() is called, and return to the initial
///  state when Final() is called. This interface allows a large message to
///  be hashed in pieces by calling Update() on each piece followed by
///  calling Final().
/// \sa HashFilter(), HashVerificationFilter()
class CRYPTOPP_DLL CRYPTOPP_NO_VTABLE HashTransformation : public Algorithm
{
public:
	virtual ~HashTransformation() {}

	/// \brief Provides a reference to this object
	/// \return A reference to this object
	/// \details Useful for passing a temporary object to a function that takes a non-const reference
	HashTransformation& Ref() {return *this;}

[Mercury13]

Подключайте byteswap.h.

[Nightmare A]

Подключено. По какой то причине ошибка возникает когда я пишу #undef Verify.

[Mercury13]

Вадим Ушаков, Напишите ошибку по-другому.

[Nightmare A]

Mercury13, Т.е?

[Mercury13]

Вадим Ушаков, Не могу увидеть то, что вы написали.

[Mercury13]

Вадим Ушаков, Увидел ошибку через VPN. И что выводит, когда undef?

[Adamos]

Mercury13,

Увидел ошибку через VPN

Зря. ТС в который раз тыкают носом в то, что ошибки нужно постить текстом, но намеков он не понимает.

[Nightmare A]

Mercury13, При #undef Verify возникает ошибка

In file included from ../deps/cryptopp/secblock.h:11,
                 from ../deps/cryptopp/md2.h:11,
                 from ../src/Machinarium/Utils/String.cpp:138:
../deps/cryptopp/misc.h: In function ‘CryptoPP::word16 CryptoPP::ByteReverse(CryptoPP::word16)’:
../deps/cryptopp/misc.h:2033:9: error: ‘bswap_16’ was not declared in this scope
 2033 |  return bswap_16(value);

Или её не видно?

[Mercury13]

Я даже не понял: это новая ошибка или просто открылась, поскольку старая исчезла?
Код в студию!

[Nightmare A]

Mercury13, dbg.h из source engine?

[Mercury13]

Вадим Ушаков,
Я пока подозреваю два варианта.
1. Вы прилепили byteswap ниже того места, где случилась ошибка.
2. В том месте, где случилась ошибка, есть и другие макросы.

[Nightmare A]

Mercury13, Ну да, верно.

[Nightmare A]

Mercury13,

// #ifdef __linux__
#include <byteswap.h>
// #undef min
// #undef max
// #endif

/// \brief Reverses bytes in a 16-bit value
/// \param value the 16-bit value to reverse
/// \details ByteReverse calls bswap if available. Otherwise the function
///  performs a 8-bit rotate on the word16.
inline word16 ByteReverse(word16 value)
{
#if defined(CRYPTOPP_BYTESWAP_AVAILABLE)
	return bswap_16(value);
#elif (_MSC_VER >= 1400) || (defined(_MSC_VER) && !defined(_DLL))
	return _byteswap_ushort(value);
#else
	return rotlFixed(value, 8U);
#endif
}

In file included from ../deps/cryptopp/secblock.h:11,
                 from ../deps/cryptopp/md2.h:11,
                 from ../src/Machinarium/Utils/String.cpp:132:
../deps/cryptopp/misc.h: In function ‘CryptoPP::word16 CryptoPP::ByteReverse(CryptoPP::word16)’:
../deps/cryptopp/misc.h:2039:9: error: ‘bswap_16’ was not declared in this scope
 2039 |  return bswap_16(value);
      |         ^~~~~~~~~
compilation terminated due to -Wfatal-errors.
make[1]: *** [Garrys Mod 10.make:355: obj/Debug/Garrys Mod 10/String.o] Error 1
make: *** [Makefile:51: Garrys Mod 10] Error 2

М, почему возникает ошибка? Включение гарантирует определение функции bswap_16?

[Mercury13]

Я не понимаю. Тут можно только танцами с бубном собрать ту версию, которая будет компилироваться.

[Nightmare A]

Mercury13, Так же не совсем понятно.

[Mercury13]

Допустим, у нас три варианта функции ByteReverse — мы берём первый. Может, заведётся второй или третий?