Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace/add funcInternal by funcHook #18

Open
toeklk opened this issue Dec 9, 2017 · 0 comments
Open

replace/add funcInternal by funcHook #18

toeklk opened this issue Dec 9, 2017 · 0 comments

Comments

@toeklk
Copy link
Collaborator

toeklk commented Dec 9, 2017

migrated from Bugzilla #476
status NEW severity enhancement in component core for ---
Reported in version trunk on platform All
Assigned to: BFL mailinglist

On 2008-01-02 13:16:49 +0100, Klaas Gadeyne wrote:

  To lower the code-readability-threshold of orocos-subprojects it would be nice if BFL would use the same conventions as the RTT in terms of function naming. Hence I suggest to replace the funcInternal() functions by a funcHook() equivalent and add some Hook() extensions to other non-public class methods to stress the fact they are not part of the public API of BFL classes. e.g. - replace UpdateInternal with UpdateHook for most filter classes - add Hook() extensions to the SysUpdate() and MeasUpdate() functions of KalmanFilter classes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant