scmRTOS, Revision: 172 Опции

[IgorKossak]

Надо бы внести поправку:

Код

namespace OS
{
    ...
    INLINE inline void Sleep(TTimeout t = 0) { TBaseProcess::Sleep(t); }
    ...
}

иначе на OS::Sleep(); (без параметра) выдаёт ошибку.

[dxp]

Надо бы внести поправку:

Код

namespace OS
{
    ...
    INLINE inline void Sleep(TTimeout t = 0) { TBaseProcess::Sleep(t); }
    ...
}

иначе на OS::Sleep(); (без параметра) выдаёт ошибку.

Оперативно следите. Пофиксено.

Скоро будет релиз по последнему состоянию репа. В основном багфикс. Следите за сообщенями в группе.

ИМХО Лучше не править, чтоб явно указывать 0 чтоб уснуть навсегда.

Чем это лучше?

Во-первых, надо придерживаться единства стиля - если TBaseProcess::Sleep() реализована с агрументом по умолчанию, то и тут надо действовать так же. Во-вторых, и в-главных, "это короче, чем я могу написать, а компилятор должен понимать умолчания" (Це) Б.Страуструп. Аргументы по умолчанию для того и созданы. Поведение документированное и предсказуемое. Ну, и, имхо, ноль смотрится некузяво - выражение читается как "спать ноль тиков".