Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/1145
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/1145/1
API CHANGE: com.sun.star.lang.IllegalArgumentException
... derives from com.sun.star.uno.RuntimeException instead of
com.sun.star.uno.Exception.
Only test that breaks with this change is jurt_uno/AnyConverter_Test,
which for mysterious reasons effectively tests that
IllegalArgumentException is a subclass of Exception and not
RuntimeException. Presumably this is just a generic exception test that
happens to use IllegalArgumentException.
Some further testing indicates there are no problems expected at
runtime:
Running "make subsequentcheck" with all Java test code compiled against
a ridl.jar that does not contain the change, running against a soffice
that uses ridl.jar and rdbs with the change + ridl.jar with the change
on the test side yields exactly the same AnyConverter_Test failure, with
no other failures.
Change-Id: Iad183de76ec7e0d56648084e97cdcc160b5b033d
---
M dbaccess/source/ui/uno/copytablewizard.cxx
M desktop/source/deployment/manager/dp_manager.cxx
M desktop/source/deployment/registry/dp_backend.cxx
M forms/source/runtime/formoperations.cxx
M framework/qa/complex/path_settings/PathSettingsTest.java
M jurt/test/com/sun/star/uno/AnyConverter_Test.java
M offapi/type_reference/types.rdb
M package/source/xstor/xstorage.cxx
M pyuno/source/module/pyuno_adapter.cxx
M qadevOOo/runner/util/BasicMacroTools.java
M qadevOOo/tests/java/mod/_pcr/ObjectInspectorModel.java
M qadevOOo/tests/java/mod/_remotebridge/uno/various.java
M qadevOOo/tests/java/mod/_remotebridge/various.java
M qadevOOo/tests/java/mod/_svtools/AccessibleBrowseBox.java
M qadevOOo/tests/java/mod/_sw/SwXCellRange.java
M qadevOOo/tests/java/mod/_sw/SwXDocumentIndex.java
M qadevOOo/tests/java/mod/_sw/SwXTextTable.java
M scripting/java/com/sun/star/script/framework/provider/ScriptProvider.java
M sfx2/source/doc/sfxbasemodel.cxx
M swext/mediawiki/src/com/sun/star/wiki/WikiOptionsEventHandlerImpl.java
M ucb/source/ucp/hierarchy/hierarchydata.cxx
M ucb/source/ucp/package/pkgcontent.cxx
M udkapi/com/sun/star/lang/IllegalArgumentException.idl
M wizards/com/sun/star/wizards/common/NumericalHelper.java
M wizards/com/sun/star/wizards/common/PlaceholderTextElement.java
M wizards/com/sun/star/wizards/common/PropertySetHelper.java
M wizards/com/sun/star/wizards/common/SystemDialog.java
M wizards/com/sun/star/wizards/query/Finalizer.java
M wizards/com/sun/star/wizards/report/DBColumn.java
M wizards/com/sun/star/wizards/report/ReportTextDocument.java
M wizards/com/sun/star/wizards/ui/FilterComponent.java
M wizards/com/sun/star/wizards/ui/UnoDialog.java
M wizards/com/sun/star/wizards/ui/WizardDialog.java
33 files changed, 58 insertions(+), 100 deletions(-)
--
To view, visit https://gerrit.libreoffice.org/1145
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iad183de76ec7e0d56648084e97cdcc160b5b033d
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Michael Stahl <mstahl@redhat.com>
Context
- [PATCH] API CHANGE: com.sun.star.lang.IllegalArgumentException · Michael Stahl (via Code Review)
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.