summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-02-13 22:13:42 +0100
committerStephan Bergmann <sbergman@redhat.com>2017-02-13 22:13:42 +0100
commitea5fffddcfc6d8741c5492b227daddae1874f100 (patch)
treee98da0820a9a914c12942f1e401ac8486f9a101d
parent7704b2f0a89fe04d089cd66e9e7ca91a64e1bacf (diff)
Remove misguided comment
...introduced with 7757a18cfd77996f4ce8aef2cbb8c70425ae92da "INTEGRATION: CWS oj14", but for one !xDoc.is() is explicitly handled below, and for another xLoadable is only used if !xDoc.is(). So the requirement is apparently that not both are null, indeed. Change-Id: I1b59e99c86c88c5266b33e9ff782054fa40dd34d
-rw-r--r--embeddedobj/source/commonembedding/persistence.cxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/embeddedobj/source/commonembedding/persistence.cxx b/embeddedobj/source/commonembedding/persistence.cxx
index 6acde548f497..26faa84e853c 100644
--- a/embeddedobj/source/commonembedding/persistence.cxx
+++ b/embeddedobj/source/commonembedding/persistence.cxx
@@ -498,7 +498,7 @@ uno::Reference< util::XCloseable > OCommonEmbeddedObject::LoadDocumentFromStorag
uno::Reference< frame::XLoadable > xLoadable( xDocument, uno::UNO_QUERY );
uno::Reference< document::XStorageBasedDocument > xDoc( xDocument, uno::UNO_QUERY );
- if ( !xDoc.is() && !xLoadable.is() ) ///BUG: This should be || instead of && ?
+ if ( !xDoc.is() && !xLoadable.is() )
throw uno::RuntimeException();
::comphelper::NamedValueCollection aLoadArgs;