scmRTOS, Revision: 172 Опции

[IgorKossak]

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

Код

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

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

[diper]

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

Код

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

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

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

[spf]

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

Почему не в группу про это пишете?

[dxp]

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

Код

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

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

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

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

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

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

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

[IgorKossak]

Почему не в группу про это пишете?

Виноват-с, упустил из виду.

[Сергей Борщ]

Почему не в группу про это пишете?

Вообще-то для этого есть баг-трекер на sourceforge...

[spf]

Вообще-то для этого есть баг-трекер на sourceforge...

Вещь конечно хорошая, но достаточно громоздкая и не для всех интуитивно понятная.
Мне кажется что на текущем этапе обсуждение в группе оптимальнее и удобнее.