make SvDetachedEventDescriptor::hasByName return bool

Also, since this is a utility method, and it does not override
a method from an UNO interface, it does not need to be virtual.

Change-Id: I2dd90331e8ff49572bdfcb6ea2e2e3ed2fd95051

include/svtools/unoevent.hxx

@@ -290,11 +290,11 @@ protected:
                 ::com::sun::star::uno::RuntimeException) SAL_OVERRIDE;
 
     /// do we have an event?
-    /// return sal_True: we have a macro for the event
-    /// return sal_False: no macro; getByName() will return an empty macro
+    /// return true: we have a macro for the event
+    /// return false: no macro; getByName() will return an empty macro
     /// IllegalArgumentException: the event is not supported
     using SvBaseEventDescriptor::hasByName;
-    virtual sal_Bool hasByName(
+    bool hasByName(
         const sal_uInt16 nEvent ) const     /// item ID of event
              throw(
                 ::com::sun::star::lang::IllegalArgumentException);

svtools/source/uno/unoevent.cxx

@@ -497,7 +497,7 @@ void SvDetachedEventDescriptor::getByName(
         rMacro = (*aMacros[nIndex]);
 }
 
-sal_Bool SvDetachedEventDescriptor::hasByName(
+bool SvDetachedEventDescriptor::hasByName(
     const sal_uInt16 nEvent ) const     /// item ID of event
         throw(IllegalArgumentException)
 {
@@ -505,7 +505,7 @@ sal_Bool SvDetachedEventDescriptor::hasByName(
     if (-1 == nIndex)
         throw IllegalArgumentException();
 
-    return (NULL == aMacros[nIndex]) ? sal_False : aMacros[nIndex]->HasMacro();
+    return (NULL != aMacros[nIndex]) && aMacros[nIndex]->HasMacro();
 }