You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ensure cs_ prefix for all API structs, ids and function names?
Not all API structs and functions have the cs_ prefix. This leads to issues like these: #2316
The question is now, should this be addressed before the v6 Beta? Or before v6 Gold?
Or is API stability more important for everyone and we just keep it as is?
Currently the naming was kinda arbitrary. Sometimes only the arch_ prefix, sometimes cs_arch_.
I think though it is important to make a decision once and then stick with it. And with the many improvements and changes v6 brings, it is most suitable time to do it. So the annoyance is kept to one time only.
NOTE: This would only affect v6 and future versions. v5 would stay as is.
API means here: Everything in include/capstone/. Internally we have more funny stuff going on, but this is out of scope.
This discussion was converted from issue #2636 on February 26, 2025 12:58.
Heading
Bold
Italic
Quote
Code
Link
Numbered list
Unordered list
Task list
Attach files
Mention
Reference
Menu
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Ensure
cs_
prefix for all API structs, ids and function names?Not all API structs and functions have the
cs_
prefix. This leads to issues like these: #2316The question is now, should this be addressed before the v6 Beta? Or before v6 Gold?
Or is API stability more important for everyone and we just keep it as is?
Currently the naming was kinda arbitrary. Sometimes only the
arch_
prefix, sometimescs_arch_
.I think though it is important to make a decision once and then stick with it. And with the many improvements and changes
v6
brings, it is most suitable time to do it. So the annoyance is kept to one time only.NOTE: This would only affect
v6
and future versions.v5
would stay as is.include/capstone/
. Internally we have more funny stuff going on, but this is out of scope.Beta Was this translation helpful? Give feedback.
All reactions