CBATeam
Remote-Promise system
Just noticed that i have had this "in the pipe" since 2018 ... finally creating the PR for it
When merged this pull request will:
- Add a promise system, allowing to poll the server, calling a local method afterwards
It bugs me that the function names are
CBA_fnc_promise_xyz
and not
cba_promise_fnc_xyz
Would prefer either PREP or camel case: CBA_fnc_promiseXyz
Why does it constantly talk about "methods"? Shouldn't it be "functions" or is this specifically a thing about promises I am missing.
Why does it constantly talk about "methods"? Shouldn't it be "functions" or is this specifically a thing about promises I am missing.
Can alter that. There is nothing you are missing here.
It bugs me that the function names are
CBA_fnc_promise_xyz
and not
cba_promise_fnc_xyz
Would prefer either PREP or camel case: CBA_fnc_promiseXyz
Which one of the two would you prefer?
Can alter that. There is nothing you are missing here.
Please do.
Which one of the two would you prefer?
Camel case, since PREP is not used much for public functions due to the advantage of the functions viewer.