Ну так я потому и предложил
static enum, чтобы хотя бы ограничить область видимости констант до одного инклуда и перестраховаться на случай совпадения имён. Вреда от этого никакого не будет в любом случае.
Никто в этой теме слона не раздувает, просто указывают на возможные недочёты.
В подобных случаях обычно так и делают (если не всегда, то точно в подавляющем большинстве из тех случаев, что приходилось наблюдать).
В догонку, ещё один бросившийся в глаза недочёт, о котором забыл написать в прошлый раз: функции и переменные названы в разнобой, как будто над кодом работали два разных человека, ни разу не видевшие код друг у друга.
Например, в том же uiv.inc: "Set
VehicleDrop", "TrunkOrRearDoorsOf
Vehicle", но "Get
VehTrunk".
Или в billboards.inc: массивы с координатами билбордов названы "billboard_objects", "billboard_objects_logo", "billboard_block_objs_t1" и т.д., но массив с информацией о билбордах почему-то назван "BillboardInfo", в стиле CamelCase, как будто это функция. И это не говоря о "BILLBOARDS_NUMS_LEADER" и "BILLBOARDS_NUMS_LOGO", которые не константы.
Также не совсем понятно на счёт этого коммита:
https://github.com/llsdm/LLSDM/commi...16e54895f59d09
Зачем нужно было в одном месте оставлять "return SendClientMessage(...)" как есть, а в другом переделывать в "SendClientMessage(...);\return 1;" ?