Skip to content
Snippets Groups Projects
Commit 57374453 authored by FBerendsen's avatar FBerendsen
Browse files

ENH: replaced strcmp by equality check

parent 2733afe7
No related branches found
No related tags found
No related merge requests found
...@@ -48,7 +48,7 @@ template<typename FirstInterface, typename ... RestInterfaces> ...@@ -48,7 +48,7 @@ template<typename FirstInterface, typename ... RestInterfaces>
ComponentBase::interfaceStatus Accepting<FirstInterface, RestInterfaces... >::ConnectFromImpl(const char * interfacename, ComponentBase* other) ComponentBase::interfaceStatus Accepting<FirstInterface, RestInterfaces... >::ConnectFromImpl(const char * interfacename, ComponentBase* other)
{ {
// does our component have an accepting interface called interfacename? // does our component have an accepting interface called interfacename?
if (0 ==strcmp(InterfaceName<InterfaceAcceptor<FirstInterface>>::Get(), interfacename)) if (InterfaceName<InterfaceAcceptor<FirstInterface>>::Get() == interfacename)
{ {
// cast always succeeds since we know via the template arguments of the component which InterfaceAcceptors its base classes are. // cast always succeeds since we know via the template arguments of the component which InterfaceAcceptors its base classes are.
InterfaceAcceptor<FirstInterface>* acceptIF = this; InterfaceAcceptor<FirstInterface>* acceptIF = this;
...@@ -82,7 +82,7 @@ int Accepting<FirstInterface, RestInterfaces... >::ConnectFromImpl(ComponentBase ...@@ -82,7 +82,7 @@ int Accepting<FirstInterface, RestInterfaces... >::ConnectFromImpl(ComponentBase
template<typename FirstInterface, typename ... RestInterfaces> template<typename FirstInterface, typename ... RestInterfaces>
bool Accepting<FirstInterface, RestInterfaces... >::HasInterface(const char* interfacename) bool Accepting<FirstInterface, RestInterfaces... >::HasInterface(const char* interfacename)
{ {
if (0 == strcmp(InterfaceName<InterfaceAcceptor<FirstInterface>>::Get(), interfacename)) if (InterfaceName<InterfaceAcceptor<FirstInterface>>::Get() == interfacename)
{ {
return true; return true;
} }
...@@ -92,7 +92,7 @@ bool Accepting<FirstInterface, RestInterfaces... >::HasInterface(const char* int ...@@ -92,7 +92,7 @@ bool Accepting<FirstInterface, RestInterfaces... >::HasInterface(const char* int
template<typename FirstInterface, typename ... RestInterfaces> template<typename FirstInterface, typename ... RestInterfaces>
bool Providing<FirstInterface, RestInterfaces... >::HasInterface(const char* interfacename) bool Providing<FirstInterface, RestInterfaces... >::HasInterface(const char* interfacename)
{ {
if (0 == strcmp(InterfaceName<FirstInterface>::Get(), interfacename)) if (InterfaceName<FirstInterface>::Get() == interfacename)
{ {
return true; return true;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment