Jump to content

    
Sign in to follow this  
Ant_m

Нашел несоответсвие между документацией и кодом

Recommended Posts

Нашел что в документах (русская версия для для ScmRTOS v5 и v4) упоминается TMutex::lock_softly(), а вот в коде этого уже нет - его заменили на TMutex::try_lock(). В документах TMutex::try_lock() не упоминается.

Стоит, наверное, эту часть документа (стр. 93) обновить...

 

Автор: Sergey A. Borshch <sborshch@users.noreply.github.com> 2014-03-11 16:43:54

Сохранивший состояние: Sergey A. Borshch <sborshch@users.noreply.github.com> 2014-03-11 16:43:54

Предок: a747e55973a8cfdbfeb61a1bfd1e31d93382a8ab (make TService inheritance protected to show that no public members provided in TService and it's parents)

Потомок: a718a881c7315bb80afceade53e2ccceafeefffc (Added universal CortexMx port)

Ветвь: master, remotes/origin/develop, remotes/origin/master

Следует за: 4.0.0

Предшествует: sf-end

 

-TMutex improvement: additional try_lock(timeout) member function with

timeout argument. Function returns false if mutex wasn't locked during

requested time.

-TMutex::lock_softly() renamed to to try_lock() (without arguments).

-TMutex::lock_softly() as wrapper to try_lock() added to scmRTOS_OBSOLETE_NAMES section

Share this post


Link to post
Share on other sites
Нашел что в документах (русская версия для для ScmRTOS v5 и v4) упоминается TMutex::lock_softly(), а вот в коде этого уже нет - его заменили на TMutex::try_lock(). В документах TMutex::try_lock() не упоминается.

Стоит, наверное, эту часть документа (стр. 93) обновить...

Благодарю за замечание. Исправлено.

Share this post


Link to post
Share on other sites

Возможно данный момент уже исправлен в новой версии документации, но отмечу, что есть несоответствие в описание на ОС версии 5, п.4.2.3 и

коде ОС v5.1 в части именования идентификаторов перечисления стартового состояния процесса TProcessStartState:

в примере п.4.2.3 документации указан префикс идентификаторов «ss», а в коде (scmRTOS_defs.h) – префикс «pss».

Share this post


Link to post
Share on other sites
Возможно данный момент уже исправлен в новой версии документации, но отмечу, что есть несоответствие в описание на ОС версии 5, п.4.2.3 и

коде ОС v5.1 в части именования идентификаторов перечисления стартового состояния процесса TProcessStartState:

в примере п.4.2.3 документации указан префикс идентификаторов «ss», а в коде (scmRTOS_defs.h) – префикс «pss».

Спасибо за замечание, исправлено в документации (залито в репозиторий.).

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this