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

Perf: Any call operation id() only for supported types #903 #1046

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xor2003
Copy link

@xor2003 xor2003 commented Feb 27, 2020

Operation id() is only available for types: for tk_objref, tk_struct, tk_union, tk_enum, tk_alias,
tk_value, tk_value_box, tk_native, tk_abstract_interface
tk_local_interface, tk_except
tk_component, tk_home and tk_event.
In other cases exception is generated and performance degraded on my tests up to 9%.
In case of exception the whole try/catch have no sense.
See CORBA 3.3 standard Chapter 8.11.1 "The TypeCode Interface" for list of types.

Operation id() is only available for types: for tk_objref, tk_struct, tk_union, tk_enum, tk_alias,
tk_value, tk_value_box, tk_native, tk_abstract_interface
tk_local_interface, tk_except
tk_component, tk_home and tk_event. 
In other cases exception is generated and performance degraded on my tests up to 9%.
In case of exception the whole try/catch have no sense.
See CORBA 3.3 standard Chapter 8.11.1 "The TypeCode Interface" for list of types.
@jwillemsen jwillemsen added the needs review Needs to be reviewed label May 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review Needs to be reviewed
Development

Successfully merging this pull request may close these issues.

2 participants