Закрома Родины Опции

[Trollz0r]

Тема для разгрузки общего обсуждения ЛА от непрофильных разговоров

Ссылки на архивы постить запрещено, но ведь не запрещено обсуждать их содержимое!

[HikeR]

СДК ещё вообще не собирал никто.

собрать редакторы не проблема, проблема в их запуске.

В самом NVTT пришлось поправить пару багов.

это "поправить" более походит на костыли для компилятора, а не баги. а вот замена vsscanf на обычный sscanf — это вообще вредительство, азбучный пример при такой замене выдает ожидаемый appcrach.

[Flammable]

это "поправить" более походит на костыли для компилятора, а не баги. а вот замена vsscanf на обычный sscanf — это вообще вредительство, азбучный пример при такой замене выдает ожидаемый appcrach.

Стянул весь проект nvtt из репозитория xp-dev, чтобы не возиться с адаптацией оригинального. Даже не знал, что оно там так "поправлено". Верну взад

nvtt/src/src/nvimage/ConeMap.cpp:82

Код

float ch = d - img->pixel(xx, yy, 0);

Как и должно быть, получаем здесь undefined identifier, потому что xx и yy не видны в текущем скопе.
В какой-нибудь VS6 оно могло компилиться из-за бага самой студии, VS2013 такую шляпу уже не проглатывает.
Если оно где-то скомпилируется, xx и yy будут равны w и h соответственно, что приведет к чтению за пределами буфера.

nvtt/src/src/nvmath/nvmath.h:88

Код

// Replace default functions with asserting ones.
#define sqrt sqrt_assert
#define sqrtf sqrtf_assert
#define acos acos_assert
#define acosf acosf_assert
#define asin asin_assert
#define asinf asinf_assert

Эта проблема уже была замечена и вместе с решением описана здесь.

Вот и возникает вопрос. Репозиториев наделали, nvdxt на nvtt заменили, а скомпилить никто даже не пробовал.
Весь код в репозитории должен быть валидным в любое время в независимости от состояния разработки, имхо.

Сообщение отредактировал Flammable - 24.08.2014, 10:28

[HikeR]

с "xx/yy" все проще, ConeMap.cpp/h не использовался в проекте как минимум с 2007 года. однако даже VC6 не сможет увидеть инициализированные в цикле переменные, реально странный код )

p.s.
гмм... этот vsscanf оказывается тоже.

Сообщение отредактировал HikeR - 24.08.2014, 11:05

[Flammable]

Коммитил эти "поправки" K.D., интересно было бы узнать его соображения.
Интересно, что в оригинальном nvtt Tokenizer.h тоже отключен.

Сообщение отредактировал Flammable - 24.08.2014, 11:13

[HikeR]

вероятно, сорцы кучей скопировали без учета отсутствующих файлов в родном проекте. а компилятор такого прикола не воспринял )

Сообщение отредактировал HikeR - 24.08.2014, 11:39

[Flammable]

Пофиксил "поправки" в NVTT.
В xrDXT из репозитория на xp-dev.com нашел еще одну проблему:
xrDXT/DXT.cpp:237

Код

HFILE fileout = _open( out_name, _O_WRONLY|_O_BINARY|_O_CREAT|_O_TRUNC,_S_IWRITE);
if (fileout==-1){
    fprintf(stderr, "Can't open output file %s\n", out_name);
    return false;
}

В этом месте файл с именем out_name уже открыт (идентичным кодом из начала функции), поэтому DXTCompressImage вернет false, а файловый дескриптор утечет.

[Vampir35]

Такой вопрос, а почему свет от фонарика представляет собой шестиугольник?

[macron]

а почему свет от фонарика представляет собой шестиугольник?

В 1.0006 было из-за r2-правки "ВИРТУАЛЬНЫЙ РАДИУС ЛАМП".

[RayTwitty]

В 1.0006 было из-за r2-правки "ВИРТУАЛЬНЫЙ РАДИУС ЛАМП".

да ладно.

[Modera]

Этот убогий фонарик из ласт лайта никакими углами не испортишь. Хуже не будет.

К тому же, смотрим в левый угол...

Сообщение отредактировал Modera - 26.08.2014, 00:00

[RayTwitty]

Этот убогий фонарик из ласт лайта никакими углами не испортишь.

Ну даа, убогий. Чувак, ты не играл в некоторые билдомоды

К тому же, смотрим в левый угол...

Там поинт-свет стоит, К.О. Он и в оригинале при выкрученных значениях яркости/блюма имеет явные края (см. свечение от монолита).

Сообщение отредактировал Shadows - 26.08.2014, 00:18

[Ронин]

Скажите пожалуйста, может кто знает, в чем разница между форматами .skl\skls от СДК 0.4 и СДК 0.7?

[Kontro-zzz]

в чем разница между форматами .skl\skls от СДК 0.4 и СДК 0.7?

х.з. наверное никакой, т.к. конвертер одинаково тчешные и зпешные омф распаковывает, и некоторые старые skl анимации нормально работают в старших версиях СДК(игре), например skl анимации для монстров.

P.S.
Проверил skl файлы из ЗП действительно отличаются от ЧН и ТЧ.

Сообщение отредактировал Kontro-zzz - 27.08.2014, 21:46

[alpet]

С последними правками вскрылось неправильное определение формата нет-пакетов (в se_zones.script), из-за чего в лог может попасть несколько мегабайт предупреждений. Похоже такая ситуация возможна лишь для модов, использующих данный скрипт несоответствующей версии.
Вызывает проблему вот какой код:

Код

packet1:w_u8(packet:r_u8())

Попытка скопировать слишком много от пакета. Простейшее ограничение что я сделал:

Код

local copy = size - packet1:r_tell()
        
    if copy > packet:r_elapsed() then
       copy = packet:r_elapsed()  -- не пытаться скопировать много
    end
    
    for i=1,copy,1 do
        if packet:r_eof() then        
           break
        else  
           packet1:w_u8(packet:r_u8())            
        end    
    end

Сообщение отредактировал alpet - 28.08.2014, 11:49

[Flammable]

Кто-нибудь знает, зачем в скриптах this?

[HikeR]

эмуляция указателя на сам объект/класс, чтобы при его изменении не затронуть все остальные отнаследованные объекты/классы (в Lua нет ничего "приватного").

[Flammable]

эмуляция указателя на сам объект/класс

Можно на примере показать? Зачем это нужно?

[HikeR]

в качестве примера придется привести полную эмуляцию класса с наследованием, то есть скопировать главу из Programming in Lua. вы вставьте строчку "print(typeof(this), this)" перед использованием этого this и сами все увидите.

дайте пример "непонятного" кода, будет более предметно.

[Flammable]

Вот такой кейс, например:

xr_use.script

Код

class "action_process_use"
function action_process_use:__init(obj, storage)
    self.object = obj
    self.st = storage
end
function action_process_use:use_callback(victim, who)
end

...

function add_to_binder(npc, ini, scheme, section, storage)
    local action = this.action_process_use(npc, storage)
    storage.action = action
end

[HikeR]

о, да тут все намного проще (xr_3da/xrGame/script_storage.cpp):

Код

-- генерация LPCSTR строк
-- file_header_old =
local function script_name()
    return "%s"
end
local this = {}
%s this %s
setmetatable(this, {__index = _G})
setfenv(1, this)

-- file_header_new =
local function script_name()
    return "%s"
end
local this = {}
this._G = _G
%s this %s
setfenv(1, this)

--skip
    if (strstr(Core.Params,"-_g"))
        file_header            = file_header_new;
    else
        file_header            = file_header_old;

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