artbear

Вот смотри, как в Фабрике сейчас стало:

int CObjectFactory::GetParamDefValue(int nMethIndex, int nParamIndex, CValue * pValue)const
{
    pValue->Reset();
	pValue->type = -1;
    return 1;
}
 

Раньше установки типа в -1 не было.

А между тем, Шрифт по-прежнему, ожидает тип 0 в случае дефолтных параметров.
Вы когда это решили поменять, про Шрифт не вспомнили, что ли?

А для остальных объектов, создаваемых через фабрику, - там как - нет этой проблемы или тоже есть?

P.S.
И вообще-то присваивание беззнаковой переменной отрицательного значения - тоже не здорово.

http://www.1cpp.ru/bugs/show_bug.cgi?id=3690

Подумаю, сам вчера увидел это предупреждение.
Только на самом деле мне помнится, что так написано во внутреннем коде 1С
Т.е. тип type на самом деле не WORD, а все-таки signed short
Лады, еще подумаю сегодня

ЗЫ ОФФ ВидДокумента, наверное, все-таки ты сделай. я намного больше, чем ты, не в теме. Ты хоть это можешь на своих базах проверить

artbear писал(а) 27. Марта 2008 :: 09:39:

Вот хз, кто вносил. Я думал - ты.
Кстати, мысль-то здравая, чтобы отличать незаданные атрибуты от просто пустых.

Но надо же при этом все классы было переработать, которые ожидали тип 0, а получили - тип -1.
Впрочем, что еще сейчас может создаваться через фабрику, кроме Шрифтов?

Цитата:

Чей, интересно.
А где это, воообще?

Цитата:

Да проверяю пока (при создании шрифта) оба типа, и 0 и -1 - считаются дефолтными.

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

artbear писал(а) 27. Марта 2008 :: 09:56:

В том-то и дело, что тот, кто это делал, про шрифты вообще не думал - забыл.

Да ну, Саня нормально бы сделал.

artbear писал(а) 27. Марта 2008 :: 06:48:

Нет, беззнаковый.
Если интересно, можешь посмотреть реализацию

int CType::GetTypeCode(void)const

kms писал(а) 27. Марта 2008 :: 11:48:

Что-то я не пойму, какая между ними разница и нафига это нужно.
Просвети.