реклама на сайте
подробности

 
 
 
Reply to this topicStart new topic
> Нашел несоответсвие между документацией и кодом, TMutex::lock_softly()
Ant_m
сообщение Dec 18 2016, 19:02
Сообщение #1


Знающий
****

Группа: Свой
Сообщений: 756
Регистрация: 14-08-07
Из: Москва
Пользователь №: 29 765



Нашел что в документах (русская версия для для 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
Go to the top of the page
 
+Quote Post
dxp
сообщение Dec 19 2016, 07:09
Сообщение #2


Adept
******

Группа: Свой
Сообщений: 3 469
Регистрация: 6-12-04
Из: Novosibirsk
Пользователь №: 1 343



QUOTE (Ant_m @ Dec 19 2016, 02:02) *
Нашел что в документах (русская версия для для ScmRTOS v5 и v4) упоминается TMutex::lock_softly(), а вот в коде этого уже нет - его заменили на TMutex::try_lock(). В документах TMutex::try_lock() не упоминается.
Стоит, наверное, эту часть документа (стр. 93) обновить...

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


--------------------
«Отыщи всему начало, и ты многое поймёшь» К. Прутков
Go to the top of the page
 
+Quote Post
Ant_m
сообщение Dec 19 2016, 07:30
Сообщение #3


Знающий
****

Группа: Свой
Сообщений: 756
Регистрация: 14-08-07
Из: Москва
Пользователь №: 29 765



Спасибо! sm.gif
Go to the top of the page
 
+Quote Post
&Rey
сообщение Feb 19 2018, 03:49
Сообщение #4





Группа: Новичок
Сообщений: 3
Регистрация: 30-01-18
Пользователь №: 101 423



Возможно данный момент уже исправлен в новой версии документации, но отмечу, что есть несоответствие в описание на ОС версии 5, п.4.2.3 и
коде ОС v5.1 в части именования идентификаторов перечисления стартового состояния процесса TProcessStartState:
в примере п.4.2.3 документации указан префикс идентификаторов «ss», а в коде (scmRTOS_defs.h) – префикс «pss».
Go to the top of the page
 
+Quote Post
dxp
сообщение Feb 19 2018, 06:27
Сообщение #5


Adept
******

Группа: Свой
Сообщений: 3 469
Регистрация: 6-12-04
Из: Novosibirsk
Пользователь №: 1 343



Цитата(&Rey @ Feb 19 2018, 10:49) *
Возможно данный момент уже исправлен в новой версии документации, но отмечу, что есть несоответствие в описание на ОС версии 5, п.4.2.3 и
коде ОС v5.1 в части именования идентификаторов перечисления стартового состояния процесса TProcessStartState:
в примере п.4.2.3 документации указан префикс идентификаторов «ss», а в коде (scmRTOS_defs.h) – префикс «pss».

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


--------------------
«Отыщи всему начало, и ты многое поймёшь» К. Прутков
Go to the top of the page
 
+Quote Post

Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 18th April 2024 - 06:52
Рейтинг@Mail.ru


Страница сгенерированна за 0.01386 секунд с 7
ELECTRONIX ©2004-2016