diff options
author | Michael Meeks <michael.meeks@novell.com> | 2010-11-22 12:12:58 +0000 |
---|---|---|
committer | Michael Meeks <michael.meeks@novell.com> | 2010-11-22 12:12:58 +0000 |
commit | d58c6e0517d14fb461b4089f4b6ae62c9ea9b26d (patch) | |
tree | 8e2714439161658e593b4410fee82e6d2977279f | |
parent | f3eb77586e067a46211334a78987652815f47744 (diff) |
cleanup floating comments
-rw-r--r-- | extensions/source/plugin/base/nfuncs.cxx | 5 | ||||
-rw-r--r-- | extensions/source/plugin/base/xplugin.cxx | 2 | ||||
-rw-r--r-- | xmlsecurity/source/xmlsec/nss/securityenvironment_nssimpl.cxx | 22 |
3 files changed, 3 insertions, 26 deletions
diff --git a/extensions/source/plugin/base/nfuncs.cxx b/extensions/source/plugin/base/nfuncs.cxx index af7bd4dc2071..297f96a0a441 100644 --- a/extensions/source/plugin/base/nfuncs.cxx +++ b/extensions/source/plugin/base/nfuncs.cxx @@ -584,11 +584,6 @@ NPError SAL_CALL NP_LOADDS NPN_GetValue( NPP instance, NPNVariable variable, vo break; } - /* - provisional code should there ever be NPNVariables that we actually - want to query from the PluginContext - */ - return aResult; } diff --git a/extensions/source/plugin/base/xplugin.cxx b/extensions/source/plugin/base/xplugin.cxx index a98fc3f58aac..dc1a4685f58b 100644 --- a/extensions/source/plugin/base/xplugin.cxx +++ b/extensions/source/plugin/base/xplugin.cxx @@ -723,7 +723,7 @@ sal_Bool XPlugin_Impl::provideNewStream(const OUString& mimetype, // there may be plugins that can use the file length information, // but currently none are known. Since this file opening/seeking/closing - // is rather costly, it is #if'ed out. If there are plugins known to + // is rather costly, it is not implemented. If there are plugins known to // make use of the file length, simply put it in PluginInputStream* pStream = new PluginInputStream( this, aURL.getStr(), diff --git a/xmlsecurity/source/xmlsec/nss/securityenvironment_nssimpl.cxx b/xmlsecurity/source/xmlsec/nss/securityenvironment_nssimpl.cxx index 187cbd34c8d6..0673a2bb8a2e 100644 --- a/xmlsecurity/source/xmlsec/nss/securityenvironment_nssimpl.cxx +++ b/xmlsecurity/source/xmlsec/nss/securityenvironment_nssimpl.cxx @@ -576,32 +576,14 @@ Reference< XCertificate > SecurityEnvironment_NssImpl :: getCertificate( const O if( arena == NULL ) throw RuntimeException() ; - /* - * mmi : because MS Crypto use the 'S' tag (equal to the 'ST' tag in NSS), but the NSS can't recognise - * it, so the 'S' tag should be changed to 'ST' tag - * - * PS : it can work, but inside libxmlsec, the 'S' tag is till used to find cert in NSS engine, so it - * is not useful at all. (comment out now) - */ - - /* end */ - - //Create cert info from issue and serial + // Create cert info from issue and serial rtl::OString ostr = rtl::OUStringToOString( issuerName , RTL_TEXTENCODING_UTF8 ) ; chIssuer = PL_strndup( ( char* )ostr.getStr(), ( int )ostr.getLength() ) ; nmIssuer = CERT_AsciiToName( chIssuer ) ; if( nmIssuer == NULL ) { PL_strfree( chIssuer ) ; PORT_FreeArena( arena, PR_FALSE ) ; - - /* - * i40394 - * - * mmi : no need to throw exception - * just return "no found" - */ - //throw RuntimeException() ; - return NULL; + return NULL; // no need for exception cf. i40394 } derIssuer = SEC_ASN1EncodeItem( arena, NULL, ( void* )nmIssuer, SEC_ASN1_GET( CERT_NameTemplate ) ) ; |