chart2/source/controller/sidebar/ChartElementsPanel.cxx | 11 chart2/source/controller/sidebar/ChartElementsPanel.hxx | 1 configure.ac | 2 cui/source/factory/dlgfact.cxx | 20 cui/source/factory/dlgfact.hxx | 4 download.lst | 2 editeng/source/accessibility/AccessibleParaManager.cxx | 10 editeng/source/editeng/editdoc.hxx | 37 editeng/source/editeng/editeng.cxx | 4 editeng/source/editeng/editstt2.hxx | 7 editeng/source/editeng/editundo.hxx | 3 editeng/source/editeng/edtspell.cxx | 30 editeng/source/editeng/edtspell.hxx | 4 editeng/source/editeng/eertfpar.hxx | 8 editeng/source/editeng/impedit.hxx | 47 editeng/source/items/paraitem.cxx | 17 editeng/source/misc/splwrap.cxx | 17 editeng/source/misc/svxacorr.cxx | 40 editeng/source/outliner/outleeng.cxx | 5 editeng/source/outliner/outleeng.hxx | 1 editeng/source/outliner/paralist.hxx | 1 external/liblangtag/UnpackedTarball_langtag.mk | 20 external/liblangtag/liblangtag-0.5.1-include-last-record-in-language-subtag-registry.patch | 49 external/liblangtag/liblangtag-0.5.1-msvc-snprintf.patch | 25 external/liblangtag/liblangtag-0.5.1-msvc-ssize_t.patch | 12 external/liblangtag/liblangtag-0.5.1-msvc-strtoull.patch | 15 external/liblangtag/liblangtag-0.5.1-msvc-warning.patch | 21 external/liblangtag/liblangtag-0.5.1-redefinition-of-typedef.patch | 31 external/liblangtag/liblangtag-0.5.1-scope-declaration.patch | 13 external/liblangtag/liblangtag-0.5.1-undefined-have-sys-param-h.patch | 14 external/liblangtag/liblangtag-0.5.1-unistd.patch | 12 external/liblangtag/liblangtag-0.5.1-vsnprintf.patch | 18 external/liblangtag/liblangtag-0.5.1-windows-do-not-prepend-dir-separator.patch | 16 external/liblangtag/liblangtag-leak.patch.0 | 10 filter/qa/cppunit/data/pict/fail/exception-1.pct |binary filter/source/graphicfilter/ipict/ipict.cxx | 5 fpicker/source/office/RemoteFilesDialog.cxx | 14 fpicker/source/office/RemoteFilesDialog.hxx | 52 fpicker/source/office/iodlg.hxx | 42 framework/source/uielement/saveasmenucontroller.cxx | 4 include/editeng/AccessibleEditableTextPara.hxx | 3 include/editeng/AccessibleParaManager.hxx | 2 include/editeng/acorrcfg.hxx | 4 include/editeng/borderline.hxx | 4 include/editeng/charsetcoloritem.hxx | 1 include/editeng/cmapitem.hxx | 2 include/editeng/crossedoutitem.hxx | 2 include/editeng/editeng.hxx | 3 include/editeng/editstat.hxx | 4 include/editeng/editview.hxx | 4 include/editeng/edtdlg.hxx | 4 include/editeng/emphasismarkitem.hxx | 2 include/editeng/flditem.hxx | 11 include/editeng/formatbreakitem.hxx | 6 include/editeng/fwdtitem.hxx | 7 include/editeng/langitem.hxx | 2 include/editeng/lrspitem.hxx | 5 include/editeng/measfld.hxx | 1 include/editeng/numitem.hxx | 7 include/editeng/outliner.hxx | 24 include/editeng/pmdlitem.hxx | 1 include/editeng/postitem.hxx | 2 include/editeng/scripttypeitem.hxx | 5 include/editeng/splwrap.hxx | 17 include/editeng/svxacorr.hxx | 5 include/editeng/svxrtf.hxx | 18 include/editeng/tstpitem.hxx | 4 include/editeng/txtrange.hxx | 2 include/editeng/unofield.hxx | 1 include/editeng/unoforou.hxx | 4 include/editeng/unoipset.hxx | 3 include/editeng/unopracc.hxx | 3 include/editeng/unotext.hxx | 1 include/editeng/wghtitem.hxx | 2 include/sfx2/event.hxx | 19 include/sfx2/sfxhelp.hxx | 2 include/sfx2/templateviewitem.hxx | 2 include/svtools/ServerDetailsControls.hxx | 4 include/svtools/breadcrumb.hxx | 4 include/svtools/foldertree.hxx | 4 sfx2/source/doc/guisaveas.cxx | 2 svtools/source/contnr/foldertree.cxx | 4 svtools/source/control/breadcrumb.cxx | 6 svtools/source/dialogs/ServerDetailsControls.cxx | 2 svtools/source/misc/dialogcontrolling.cxx | 18 svx/source/tbxctrls/lboxctrl.cxx | 2 vcl/source/window/event.cxx | 6 writerfilter/source/dmapper/DomainMapper.cxx | 34 writerfilter/source/dmapper/DomainMapper_Impl.cxx | 364 ++-- writerfilter/source/dmapper/DomainMapper_Impl.hxx | 7 writerfilter/source/dmapper/GraphicHelpers.cxx | 6 writerfilter/source/dmapper/GraphicImport.cxx | 91 - writerfilter/source/dmapper/ModelEventListener.cxx | 6 writerfilter/source/dmapper/NumberingManager.cxx | 34 writerfilter/source/dmapper/OLEHandler.cxx | 6 writerfilter/source/dmapper/PropertyIds.cxx | 761 ++++------ writerfilter/source/dmapper/PropertyIds.hxx | 15 writerfilter/source/dmapper/PropertyMap.cxx | 88 - writerfilter/source/dmapper/StyleSheetTable.cxx | 36 writerfilter/source/dmapper/TrackChangesHandler.cxx | 18 100 files changed, 796 insertions(+), 1555 deletions(-)
New commits: commit 43cf7b9abf1262011b22a7b1a355569585f76b16 Author: Eike Rathke <er...@redhat.com> Date: Thu Jul 16 11:50:27 2015 +0200 rm unused patch Change-Id: Ie33bc6a6ef5c1a638a138e4989bd885b6b678ab4 diff --git a/external/liblangtag/liblangtag-0.5.1-redefinition-of-typedef.patch b/external/liblangtag/liblangtag-0.5.1-redefinition-of-typedef.patch deleted file mode 100644 index 1c9ac14..0000000 --- a/external/liblangtag/liblangtag-0.5.1-redefinition-of-typedef.patch +++ /dev/null @@ -1,31 +0,0 @@ -diff -ru langtag.orig/liblangtag/lt-trie.c langtag/liblangtag/lt-trie.c ---- UnpackedTarball/langtag.orig/liblangtag/lt-trie.c 2013-04-30 04:37:30.000000000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-trie.c 2013-04-30 14:57:50.777932196 +0200 -@@ -33,11 +33,6 @@ - lt_iter_tmpl_t parent; - lt_trie_node_t *root; - }; --typedef struct _lt_trie_iter_t { -- lt_iter_t parent; -- lt_list_t *stack; -- lt_string_t *pos_str; --} lt_trie_iter_t; - - /*< private >*/ - static lt_trie_node_t * -diff -ru langtag.orig/liblangtag/lt-trie.h langtag/liblangtag/lt-trie.h ---- UnpackedTarball/langtag.orig/liblangtag/lt-trie.h 2013-04-30 04:44:36.000000000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-trie.h 2013-04-30 14:57:57.746969291 +0200 -@@ -21,7 +21,11 @@ - LT_BEGIN_DECLS - - typedef struct _lt_trie_t lt_trie_t; --typedef struct _lt_trie_iter_t lt_trie_iter_t; -+typedef struct _lt_trie_iter_t { -+ lt_iter_t parent; -+ lt_list_t *stack; -+ lt_string_t *pos_str; -+} lt_trie_iter_t; - - lt_trie_t *lt_trie_new (void); - lt_trie_t *lt_trie_ref (lt_trie_t *trie); commit 67bded9f0b63f722ff44bd2e653841cce389119b Author: Eike Rathke <er...@redhat.com> Date: Wed Jul 15 16:54:41 2015 +0200 update to liblangtag-0.5.7 Change-Id: I46bf74efb52435313eb17e0db8b1cf103a329004 Reviewed-on: https://gerrit.libreoffice.org/17078 Tested-by: Jenkins <c...@libreoffice.org> Reviewed-by: Eike Rathke <er...@redhat.com> diff --git a/configure.ac b/configure.ac index 87d7990..e7f6872 100644 --- a/configure.ac +++ b/configure.ac @@ -12047,8 +12047,6 @@ if test "$enable_liblangtag" = "yes" -o \( "$enable_liblangtag" = "" -a $_os != SYSTEM_LIBLANGTAG= AC_MSG_RESULT([no]) BUILD_TYPE="$BUILD_TYPE LIBLANGTAG" - dnl TODO: remove when liblangtag is updated to 0.5.5 - AC_DEFINE([LIBLANGTAG_INLINE_FIX]) if test "$COM" = "MSC"; then LIBLANGTAG_LIBS="${WORKDIR}/UnpackedTarball/langtag/liblangtag/.libs/liblangtag.lib" else diff --git a/download.lst b/download.lst index 07959ae..ceee956 100644 --- a/download.lst +++ b/download.lst @@ -95,7 +95,7 @@ export LIBEOT_TARBALL := libeot-0.01.tar.bz2 export LIBEXTTEXTCAT_TARBALL := 10d61fbaa6a06348823651b1bd7940fe-libexttextcat-3.4.4.tar.bz2 export LIBGLTF_MD5SUM := d63a9f47ab048f5009d90693d6aa6424 export LIBGLTF_TARBALL := libgltf-0.0.2.tar.bz2 -export LIBLANGTAG_TARBALL := 36271d3fa0d9dec1632029b6d7aac925-liblangtag-0.5.1.tar.bz2 +export LIBLANGTAG_TARBALL := 80d063d6db4c010e18c606af8aed6231-liblangtag-0.5.7.tar.bz2 export LIBXMLSEC_TARBALL := 1f24ab1d39f4a51faf22244c94a6203f-xmlsec1-1.2.14.tar.gz export LIBXML_TARBALL := 9c0cfef285d5c4a5c80d00904ddab380-libxml2-2.9.1.tar.gz export LIBXSLT_TARBALL := 9667bf6f9310b957254fdcf6596600b7-libxslt-1.1.28.tar.gz diff --git a/external/liblangtag/UnpackedTarball_langtag.mk b/external/liblangtag/UnpackedTarball_langtag.mk index 2951368..541baa3 100644 --- a/external/liblangtag/UnpackedTarball_langtag.mk +++ b/external/liblangtag/UnpackedTarball_langtag.mk @@ -15,22 +15,10 @@ $(eval $(call gb_UnpackedTarball_set_pre_action,langtag,\ $(GNUTAR) -x -j -f $(gb_UnpackedTarget_TARFILE_LOCATION)/$(LANGTAGREG_TARBALL) \ )) -# external/liblangtag/liblangtag-leak.patch.0 upstream: -# <https://bitbucket.org/tagoh/liblangtag/pull-request/8/fix-memory-leak/diff> -$(eval $(call gb_UnpackedTarball_add_patches,langtag,\ - external/liblangtag/liblangtag-0.5.1-msvc-warning.patch \ - external/liblangtag/liblangtag-0.5.1-vsnprintf.patch \ - external/liblangtag/liblangtag-0.5.1-msvc-ssize_t.patch \ - external/liblangtag/liblangtag-0.5.1-msvc-snprintf.patch \ - external/liblangtag/liblangtag-0.5.1-msvc-strtoull.patch \ - external/liblangtag/liblangtag-0.5.1-scope-declaration.patch \ - external/liblangtag/liblangtag-0.5.1-redefinition-of-typedef.patch \ - external/liblangtag/liblangtag-0.5.1-undefined-have-sys-param-h.patch \ - external/liblangtag/liblangtag-0.5.1-windows-do-not-prepend-dir-separator.patch \ - external/liblangtag/liblangtag-0.5.1-unistd.patch \ - external/liblangtag/liblangtag-0.5.1-include-last-record-in-language-subtag-registry.patch \ - external/liblangtag/liblangtag-leak.patch.0 \ -)) +# Currently no patches applied, if there were it would be: +#$(eval $(call gb_UnpackedTarball_add_patches,langtag,\ +# external/liblangtag/your-modification.patch \ +#)) ifeq ($(OS),WNT) ifeq ($(COM),GCC) diff --git a/external/liblangtag/liblangtag-0.5.1-include-last-record-in-language-subtag-registry.patch b/external/liblangtag/liblangtag-0.5.1-include-last-record-in-language-subtag-registry.patch deleted file mode 100644 index c28727c3b..0000000 --- a/external/liblangtag/liblangtag-0.5.1-include-last-record-in-language-subtag-registry.patch +++ /dev/null @@ -1,49 +0,0 @@ -Upstream 6564b87c2f6dac2482e530bc43b038a3e93a0e07 -resolves #6, include last record in language-subtag-registry.xml - -The '%%' in language-subtag-registry is a record delimiter, not record -end marker, it is not present on the last record at file end. - -(cherry picked from commit 4fff73a834f94a7ca256c086d99ea0a654a6c287) - -diff --git a/data/reg2xml.c b/data/reg2xml.c ---- UnpackedTarball/langtag.orig/data/reg2xml.c -+++ UnpackedTarball/langtag/data/reg2xml.c -@@ -48,6 +48,7 @@ _parse(const char *filename, - FILE *fp; - char buffer[1024], *range = NULL, *begin = NULL, *end = NULL; - lt_bool_t in_entry = FALSE; -+ lt_bool_t file_end = FALSE; - xmlNodePtr ent = NULL; - - if ((fp = fopen(filename, "rb")) == NULL) { -@@ -56,10 +57,13 @@ _parse(const char *filename, - } - while (1) { - fgets(buffer, 1024, fp); -- if (feof(fp)) -- break; -+ if (feof(fp)) { -+ if (!in_entry) -+ break; -+ file_end = TRUE; -+ } - _drop_crlf(buffer); -- if (lt_strcmp0(buffer, "%%") == 0) { -+ if (lt_strcmp0(buffer, "%%") == 0 || file_end) { - if (in_entry) { - if (ent) { - if (range) { -@@ -102,7 +106,7 @@ _parse(const char *filename, - ent = NULL; - range = NULL; - } -- in_entry = TRUE; -+ in_entry = !file_end; - } else { - if (!in_entry) { - /* ignore it */ - ---------------erAck-patch-parts-- - - diff --git a/external/liblangtag/liblangtag-0.5.1-msvc-snprintf.patch b/external/liblangtag/liblangtag-0.5.1-msvc-snprintf.patch deleted file mode 100644 index 61067e2..0000000 --- a/external/liblangtag/liblangtag-0.5.1-msvc-snprintf.patch +++ /dev/null @@ -1,25 +0,0 @@ -diff -ru langtag.orig/liblangtag/lt-messages.c langtag/liblangtag/lt-messages.c ---- UnpackedTarball/langtag.orig/liblangtag/lt-messages.c 2013-04-22 10:06:57.000000000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-messages.c 2013-04-30 11:38:04.116532065 +0200 -@@ -85,13 +85,21 @@ - clen = strlen(cs); - if (clen > 0) { - catstring = malloc(sizeof (char) * (clen + 6)); -+#ifdef _WIN32 -+ _snprintf(catstring, clen + 6, "[%s]: ", cs); -+#else - snprintf(catstring, clen + 6, "[%s]: ", cs); -+#endif - clen = strlen(catstring); - } - len = tlen + clen + 1; - retval = malloc(sizeof (char) * len); - if (retval) { -+#ifdef _WIN32 -+ _snprintf(retval, len, "%s%s ", ts, catstring ? catstring : ""); -+#else - snprintf(retval, len, "%s%s ", ts, catstring ? catstring : ""); -+#endif - } - if (catstring) - free(catstring); diff --git a/external/liblangtag/liblangtag-0.5.1-msvc-ssize_t.patch b/external/liblangtag/liblangtag-0.5.1-msvc-ssize_t.patch deleted file mode 100644 index 84774c1..0000000 --- a/external/liblangtag/liblangtag-0.5.1-msvc-ssize_t.patch +++ /dev/null @@ -1,12 +0,0 @@ -diff -ru UnpackedTarball/langtag.orig/liblangtag/lt-macros.h UnpackedTarball/langtag/liblangtag/lt-macros.h ---- UnpackedTarball/langtag.orig/liblangtag/lt-macros.h 2013-04-22 10:06:57.000000000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-macros.h 2013-04-30 16:07:00.619638050 +0200 -@@ -261,7 +261,7 @@ - - LT_BEGIN_DECLS - --#ifdef _MSC_VER -+#if defined(_MSC_VER) && !defined(ssize_t) - # ifdef _M_AMD64 - typedef signed long long ssize_t; - # else diff --git a/external/liblangtag/liblangtag-0.5.1-msvc-strtoull.patch b/external/liblangtag/liblangtag-0.5.1-msvc-strtoull.patch deleted file mode 100644 index e50dddb..0000000 --- a/external/liblangtag/liblangtag-0.5.1-msvc-strtoull.patch +++ /dev/null @@ -1,15 +0,0 @@ -diff -ru langtag.orig/extensions/lt-ext-ldml-u.c langtag/extensions/lt-ext-ldml-u.c ---- UnpackedTarball/langtag.orig/extensions/lt-ext-ldml-u.c 2013-04-22 10:06:56.000000000 +0200 -+++ UnpackedTarball/langtag/extensions/lt-ext-ldml-u.c 2013-04-30 12:56:19.557490167 +0200 -@@ -137,7 +137,11 @@ - if (!strchr(hexdigit, subtag[j])) - goto bail2; - } -+#ifdef _WIN32 -+ x = _strtoui64(subtag, &p, 16); -+#else - x = strtoull(subtag, &p, 16); -+#endif - if (p && p[0] == 0 && x <= 0x10ffff) { - retval = TRUE; - xmlFree(name); diff --git a/external/liblangtag/liblangtag-0.5.1-msvc-warning.patch b/external/liblangtag/liblangtag-0.5.1-msvc-warning.patch deleted file mode 100644 index 2d34fe8..0000000 --- a/external/liblangtag/liblangtag-0.5.1-msvc-warning.patch +++ /dev/null @@ -1,21 +0,0 @@ ---- misc/liblangtag-0.4.0/liblangtag/lt-ext-module.h 2012-02-02 06:02:45.000000000 +0100 -+++ misc/build/liblangtag-0.4.0/liblangtag/lt-ext-module.h 2012-08-08 08:46:57.810860044 +0200 -@@ -143,12 +143,12 @@ - * and #lt_ext_module_t and extend features. - */ - struct _lt_ext_module_funcs_t { -- const lt_ext_module_singleton_func_t get_singleton; -- const lt_ext_module_data_new_func_t create_data; -- const lt_ext_module_precheck_func_t precheck_tag; -- const lt_ext_module_parse_func_t parse_tag; -- const lt_ext_module_get_tag_func_t get_tag; -- const lt_ext_module_validate_func_t validate_tag; -+ lt_ext_module_singleton_func_t get_singleton; -+ lt_ext_module_data_new_func_t create_data; -+ lt_ext_module_precheck_func_t precheck_tag; -+ lt_ext_module_parse_func_t parse_tag; -+ lt_ext_module_get_tag_func_t get_tag; -+ lt_ext_module_validate_func_t validate_tag; - }; - - diff --git a/external/liblangtag/liblangtag-0.5.1-scope-declaration.patch b/external/liblangtag/liblangtag-0.5.1-scope-declaration.patch deleted file mode 100644 index 69bc434..0000000 --- a/external/liblangtag/liblangtag-0.5.1-scope-declaration.patch +++ /dev/null @@ -1,13 +0,0 @@ -diff -ru langtag.orig/tests/variant-db.c langtag/tests/variant-db.c ---- UnpackedTarball/langtag.orig/tests/variant-db.c 2013-04-22 10:06:57.000000000 +0200 -+++ UnpackedTarball/langtag/tests/variant-db.c 2013-04-30 14:12:18.399306545 +0200 -@@ -53,8 +53,8 @@ - } - lt_iter_finish(iter); - } else if (lt_strcmp0(argv[1], "lookup") == 0) { -- variant = lt_variant_db_lookup(variantdb, argv[2]); - const lt_list_t *prefix, *l; -+ variant = lt_variant_db_lookup(variantdb, argv[2]); - - if (variant) { - printf("desc: %s\n", lt_variant_get_name(variant)); diff --git a/external/liblangtag/liblangtag-0.5.1-undefined-have-sys-param-h.patch b/external/liblangtag/liblangtag-0.5.1-undefined-have-sys-param-h.patch deleted file mode 100644 index 3cee145..0000000 --- a/external/liblangtag/liblangtag-0.5.1-undefined-have-sys-param-h.patch +++ /dev/null @@ -1,14 +0,0 @@ -# liblangtag/lt-macros.h:17:5: error: "HAVE_SYS_PARAM_H" is not defined [-Werror=undef] - -diff -ru UnpackedTarball/langtag.orig/liblangtag/lt-macros.h UnpackedTarball/langtag/liblangtag/lt-macros.h ---- UnpackedTarball/langtag.orig/liblangtag/lt-macros.h 2013-04-30 16:09:19.362392416 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-macros.h 2013-04-30 18:01:13.275188759 +0200 -@@ -14,7 +14,7 @@ - #error "Only <liblangtag/langtag.h> can be included directly." - #endif - --#if HAVE_SYS_PARAM_H -+#if defined(HAVE_SYS_PARAM_H) && HAVE_SYS_PARAM_H - #include <sys/param.h> - #endif - diff --git a/external/liblangtag/liblangtag-0.5.1-unistd.patch b/external/liblangtag/liblangtag-0.5.1-unistd.patch deleted file mode 100644 index 0aebe0b..0000000 --- a/external/liblangtag/liblangtag-0.5.1-unistd.patch +++ /dev/null @@ -1,12 +0,0 @@ ---- UnpackedTarball/langtag/liblangtag/lt-utils.c 2013-05-10 14:36:54.445200000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-utils.c 2013-05-10 14:38:31.446000000 +0200 -@@ -23,7 +23,9 @@ - #if HAVE_STRING_H - #include <string.h> - #endif -+#if HAVE_UNISTD_H - #include <unistd.h> -+#endif - #include <sys/types.h> - #include "lt-messages.h" - #include "lt-utils.h" diff --git a/external/liblangtag/liblangtag-0.5.1-vsnprintf.patch b/external/liblangtag/liblangtag-0.5.1-vsnprintf.patch deleted file mode 100644 index 599c47f..0000000 --- a/external/liblangtag/liblangtag-0.5.1-vsnprintf.patch +++ /dev/null @@ -1,18 +0,0 @@ -diff -ru UnpackedTarball/orig.langtag/liblangtag/lt-utils.c UnpackedTarball/langtag/liblangtag/lt-utils.c ---- UnpackedTarball/orig.langtag/liblangtag/lt-utils.c 2013-07-19 16:47:20.310734230 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-utils.c 2013-07-19 16:53:10.839914271 +0200 -@@ -173,9 +173,11 @@ - - va_end(ap); - -- retval = malloc(sizeof (char) * size); -- if (retval) { -- vsprintf(retval, format, args); -+ if (size > 0) { -+ retval = malloc(sizeof (char) * size); -+ if (retval) { -+ vsprintf(retval, format, args); -+ } - } - } LT_STMT_END; - #elif HAVE_VSNPRINTF diff --git a/external/liblangtag/liblangtag-0.5.1-windows-do-not-prepend-dir-separator.patch b/external/liblangtag/liblangtag-0.5.1-windows-do-not-prepend-dir-separator.patch deleted file mode 100644 index cc0e4ab..0000000 --- a/external/liblangtag/liblangtag-0.5.1-windows-do-not-prepend-dir-separator.patch +++ /dev/null @@ -1,16 +0,0 @@ -diff -ru UnpackedTarball/langtag.orig/liblangtag/lt-string.c UnpackedTarball/langtag/liblangtag/lt-string.c ---- UnpackedTarball/langtag.orig/liblangtag/lt-string.c 2013-04-22 10:35:07.000000000 +0200 -+++ UnpackedTarball/langtag/liblangtag/lt-string.c 2013-04-30 18:34:14.422037274 +0200 -@@ -308,8 +308,12 @@ - lt_return_val_if_fail (string != NULL, NULL); - lt_return_val_if_fail (path != NULL, string); - -+#ifdef _WIN32 -+ /* This simply does not work on Windows with "D:\..." */ -+#else - if (lt_string_length(string) == 0 && path[0] != LT_DIR_SEPARATOR) - lt_string_append(string, LT_DIR_SEPARATOR_S); -+#endif - - va_start(ap, path); - p = path; diff --git a/external/liblangtag/liblangtag-leak.patch.0 b/external/liblangtag/liblangtag-leak.patch.0 deleted file mode 100644 index 4a74d68..0000000 --- a/external/liblangtag/liblangtag-leak.patch.0 +++ /dev/null @@ -1,10 +0,0 @@ ---- liblangtag/lt-tag.c -+++ liblangtag/lt-tag.c -@@ -1408,6 +1408,7 @@ - lt_variant_ref(lt_list_value(l)), - (lt_destroy_func_t)lt_variant_unref); - } -+ lt_mem_add_ref(&retval->parent, retval->variants, lt_list_free); - } - if (tag->extension) { - lt_tag_set_extension(retval, lt_extension_copy(tag->extension)); commit b70acbc198fdd570785bc6d4dec992d8474024dc Author: Noel Grandin <n...@peralex.com> Date: Thu Jul 16 11:40:44 2015 +0200 fix windows build warning C4245: '=' : conversion from 'int' to 'sal_uInt8', signed/unsigned mismatch Change-Id: I4856a096a647ef47cdb208211f588f98fab71290 diff --git a/sfx2/source/doc/guisaveas.cxx b/sfx2/source/doc/guisaveas.cxx index 6a204fa..3d98457 100644 --- a/sfx2/source/doc/guisaveas.cxx +++ b/sfx2/source/doc/guisaveas.cxx @@ -161,7 +161,7 @@ static sal_uInt8 getStoreModeFromSlotName( const OUString& aSlotName ) else if ( aSlotName == "SaveAs" ) nResult = SAVEAS_REQUESTED; else if ( aSlotName == "SaveAsRemote" ) - nResult = SAVEASREMOTE_REQUESTED; + nResult = static_cast<sal_uInt8>(SAVEASREMOTE_REQUESTED); else throw task::ErrorCodeIOException( ("getStoreModeFromSlotName(\"" + aSlotName commit cf4159e16c13a13d0bedccebb50bb08f1662bc1c Author: Caolán McNamara <caol...@redhat.com> Date: Thu Jul 16 10:01:24 2015 +0100 exception on div by 0 Change-Id: Id33d6a5e3df5812babd28ebfc65b95ce97219ad3 diff --git a/filter/qa/cppunit/data/pict/fail/exception-1.pct b/filter/qa/cppunit/data/pict/fail/exception-1.pct new file mode 100644 index 0000000..f9cd85a Binary files /dev/null and b/filter/qa/cppunit/data/pict/fail/exception-1.pct differ diff --git a/filter/source/graphicfilter/ipict/ipict.cxx b/filter/source/graphicfilter/ipict/ipict.cxx index e6b7fd0..0c1960b 100644 --- a/filter/source/graphicfilter/ipict/ipict.cxx +++ b/filter/source/graphicfilter/ipict/ipict.cxx @@ -1859,6 +1859,7 @@ sal_uLong PictReader::ReadData(sal_uInt16 nOpcode) void PictReader::ReadPict( SvStream & rStreamPict, GDIMetaFile & rGDIMetaFile ) { + try { sal_uInt16 nOpcode; sal_uInt8 nOneByteOpcode; sal_uLong nSize, nPercent, nLastPercent; @@ -1950,6 +1951,10 @@ void PictReader::ReadPict( SvStream & rStreamPict, GDIMetaFile & rGDIMetaFile ) pPict->SetEndian(nOrigNumberFormat); if (pPict->GetError()) pPict->Seek(nOrigPos); + } catch (...) + { + rStreamPict.SetError(SVSTREAM_FILEFORMAT_ERROR); + } } namespace pict { commit 091f6c382394390206f784a5ec79842709e0f3bc Author: Matthew Pottage <matthewpott...@invincitech.com> Date: Wed Jul 1 16:51:45 2015 +0100 Removed singleton PropertyNameSupplier and replaced it with single function. Measurements showed that the "optimisation" of caching PropertyIds and their string equivalent leads to an increase of approx. 6 times in the total overall time spent in PropertyNameSupplier::getName(eId), when running the unit tests. PropertyNameSupplier was the only PropertyNameSupplier (no derived classes). This means that getPropertyName can easily provide the same functionality. Change-Id: I933b67c11d4cc35395a0c70e15f1c24ac9842ab0 Reviewed-on: https://gerrit.libreoffice.org/16665 Tested-by: Jenkins <c...@libreoffice.org> Reviewed-by: Noel Grandin <noelgran...@gmail.com> diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx index becde28..8a875ae 100644 --- a/writerfilter/source/dmapper/DomainMapper.cxx +++ b/writerfilter/source/dmapper/DomainMapper.cxx @@ -104,13 +104,13 @@ DomainMapper::DomainMapper( const uno::Reference< uno::XComponentContext >& xCon { // #i24363# tab stops relative to indent m_pImpl->SetDocumentSettingsProperty( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_TABS_RELATIVE_TO_INDENT ), + getPropertyName( PROP_TABS_RELATIVE_TO_INDENT ), uno::makeAny( false ) ); m_pImpl->SetDocumentSettingsProperty( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_SURROUND_TEXT_WRAP_SMALL ), + getPropertyName( PROP_SURROUND_TEXT_WRAP_SMALL ), uno::makeAny( true ) ); m_pImpl->SetDocumentSettingsProperty( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_APPLY_PARAGRAPH_MARK_FORMAT_TO_NUMBERING ), + getPropertyName( PROP_APPLY_PARAGRAPH_MARK_FORMAT_TO_NUMBERING ), uno::makeAny( true ) ); // Don't load the default style definitions to avoid weird mix @@ -271,7 +271,7 @@ void DomainMapper::lcl_attribute(Id nName, Value & val) { uno::Reference< beans::XPropertySet > xAnchorProps( m_pImpl->GetTopContext()->GetFootnote()->getAnchor(), uno::UNO_QUERY ); xAnchorProps->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_CHAR_FONT_NAME), + getPropertyName( PROP_CHAR_FONT_NAME), uno::makeAny( sStringValue )); } else //a real symbol @@ -1166,7 +1166,6 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) Value::Pointer_t pValue = rSprm.getValue(); sal_Int32 nIntValue = pValue->getInt(); const OUString sStringValue = pValue->getString(); - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); switch(nSprmId) { @@ -1525,7 +1524,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) uno::Reference<beans::XPropertySet> xCharStyle(m_pImpl->GetCurrentNumberingCharStyle()); if (xCharStyle.is()) - xCharStyle->setPropertyValue(rPropNameSupplier.GetName(PROP_CHAR_WEIGHT), aBold); + xCharStyle->setPropertyValue(getPropertyName(PROP_CHAR_WEIGHT), aBold); if (nSprmId == NS_ooxml::LN_EG_RPrBase_b) m_pImpl->appendGrabBag(m_pImpl->m_aInteropGrabBag, "b", OUString::number(nIntValue)); else if (nSprmId == NS_ooxml::LN_EG_RPrBase_bCs) @@ -1605,7 +1604,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) uno::Reference<beans::XPropertySet> xCharStyle(m_pImpl->GetCurrentNumberingCharStyle()); if (xCharStyle.is()) - xCharStyle->setPropertyValue(rPropNameSupplier.GetName(PROP_CHAR_HEIGHT), aVal); + xCharStyle->setPropertyValue(getPropertyName(PROP_CHAR_HEIGHT), aVal); } } // Make sure char sizes defined in the stylesheets don't affect char props from direct formatting. @@ -1865,13 +1864,12 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) { uno::Reference< text::XLineNumberingProperties > xLineNumberingProperties( m_pImpl->GetTextDocument(), uno::UNO_QUERY_THROW ); uno::Reference< beans::XPropertySet > xLineNumberingPropSet = xLineNumberingProperties->getLineNumberingProperties(); - PropertyNameSupplier& rNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); - xLineNumberingPropSet->setPropertyValue(rNameSupplier.GetName( PROP_IS_ON ), uno::makeAny(true) ); + xLineNumberingPropSet->setPropertyValue(getPropertyName( PROP_IS_ON ), uno::makeAny(true) ); if( aSettings.nInterval ) - xLineNumberingPropSet->setPropertyValue(rNameSupplier.GetName( PROP_INTERVAL ), uno::makeAny((sal_Int16)aSettings.nInterval) ); + xLineNumberingPropSet->setPropertyValue(getPropertyName( PROP_INTERVAL ), uno::makeAny((sal_Int16)aSettings.nInterval) ); if( aSettings.nDistance ) - xLineNumberingPropSet->setPropertyValue(rNameSupplier.GetName( PROP_DISTANCE ), uno::makeAny(aSettings.nDistance) ); - xLineNumberingPropSet->setPropertyValue(rNameSupplier.GetName( PROP_RESTART_AT_EACH_PAGE ), uno::makeAny(aSettings.bRestartAtEachPage) ); + xLineNumberingPropSet->setPropertyValue(getPropertyName( PROP_DISTANCE ), uno::makeAny(aSettings.nDistance) ); + xLineNumberingPropSet->setPropertyValue(getPropertyName( PROP_RESTART_AT_EACH_PAGE ), uno::makeAny(aSettings.bRestartAtEachPage) ); } catch( const uno::Exception& ) { @@ -1978,13 +1976,13 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) if( xTOC.is() ) { uno::Reference<text::XTextColumns> xTextColumns; - xTOC->getPropertyValue(rPropNameSupplier.GetName( PROP_TEXT_COLUMNS )) >>= xTextColumns; + xTOC->getPropertyValue(getPropertyName( PROP_TEXT_COLUMNS )) >>= xTextColumns; if (xTextColumns.is()) { uno::Reference< beans::XPropertySet > xColumnPropSet( xTextColumns, uno::UNO_QUERY_THROW ); if ( xColumnPropSet.is() ) - xColumnPropSet->setPropertyValue( rPropNameSupplier.GetName( PROP_AUTOMATIC_DISTANCE ), uno::makeAny( pSectHdl->GetSpace() )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_TEXT_COLUMNS ), uno::makeAny( xTextColumns ) ); + xColumnPropSet->setPropertyValue( getPropertyName( PROP_AUTOMATIC_DISTANCE ), uno::makeAny( pSectHdl->GetSpace() )); + xTOC->setPropertyValue( getPropertyName( PROP_TEXT_COLUMNS ), uno::makeAny( xTextColumns ) ); } } } @@ -2194,7 +2192,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) if( NS_ooxml::LN_EG_FtnEdnNumProps_numStart == nSprmId && xFtnEdnSettings.is()) { xFtnEdnSettings->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_START_AT), + getPropertyName( PROP_START_AT), uno::makeAny( sal_Int16( nIntValue - 1 ))); } else if( NS_ooxml::LN_EG_FtnEdnNumProps_numRestart == nSprmId && xFtnEdnSettings.is()) @@ -2208,14 +2206,14 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, PropertyMapPtr rContext ) default: break; } xFtnEdnSettings->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_FOOTNOTE_COUNTING ), + getPropertyName( PROP_FOOTNOTE_COUNTING ), uno::makeAny( nFootnoteCounting )); } else if (xFtnEdnSettings.is()) { sal_Int16 nNumType = ConversionHelper::ConvertNumberingType( nIntValue ); xFtnEdnSettings->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_NUMBERING_TYPE), + getPropertyName( PROP_NUMBERING_TYPE), uno::makeAny( nNumType )); } } diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx index f8ae16d..cfa1b83 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx @@ -88,7 +88,7 @@ namespace writerfilter { namespace dmapper{ //line numbering for header/footer -void lcl_linenumberingHeaderFooter( PropertyNameSupplier& rPropNameSupplier, uno::Reference<container::XNameContainer> xStyles, const OUString& rname, DomainMapper_Impl* dmapper ) +void lcl_linenumberingHeaderFooter( uno::Reference<container::XNameContainer> xStyles, const OUString& rname, DomainMapper_Impl* dmapper ) { const StyleSheetEntryPtr pEntry = dmapper->GetStyleSheetTable()->FindStyleSheetByISTD( rname ); if (!pEntry) @@ -106,7 +106,7 @@ void lcl_linenumberingHeaderFooter( PropertyNameSupplier& rPropNameSupplier, uno if( !xStyle.is() ) return; uno::Reference<beans::XPropertySet> xPropertySet( xStyle, uno::UNO_QUERY ); - xPropertySet->setPropertyValue( rPropNameSupplier.GetName( PROP_PARA_LINE_NUMBER_COUNT ), uno::makeAny( ( bool )( nListId >= 0 ) ) ); + xPropertySet->setPropertyValue( getPropertyName( PROP_PARA_LINE_NUMBER_COUNT ), uno::makeAny( ( bool )( nListId >= 0 ) ) ); } } } @@ -133,18 +133,18 @@ void lcl_handleDropdownField( const uno::Reference< beans::XPropertySet >& rxFie } } -void lcl_handleTextField( const uno::Reference< beans::XPropertySet >& rxFieldProps, FFDataHandler::Pointer_t pFFDataHandler, PropertyNameSupplier& rPropNameSupplier ) +void lcl_handleTextField( const uno::Reference< beans::XPropertySet >& rxFieldProps, FFDataHandler::Pointer_t pFFDataHandler ) { if ( rxFieldProps.is() && pFFDataHandler ) { rxFieldProps->setPropertyValue - (rPropNameSupplier.GetName(PROP_HINT), + (getPropertyName(PROP_HINT), uno::makeAny(pFFDataHandler->getStatusText())); rxFieldProps->setPropertyValue - (rPropNameSupplier.GetName(PROP_HELP), + (getPropertyName(PROP_HELP), uno::makeAny(pFFDataHandler->getHelpText())); rxFieldProps->setPropertyValue - (rPropNameSupplier.GetName(PROP_CONTENT), + (getPropertyName(PROP_CONTENT), uno::makeAny(pFFDataHandler->getTextDefault())); } } @@ -776,10 +776,9 @@ void lcl_MoveBorderPropertiesToFrame(std::vector<beans::PropertyValue>& rFramePr sal_uInt32 nBorderPropertyCount = sizeof( aBorderProperties ) / sizeof(PropertyIds); - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); for( sal_uInt32 nProperty = 0; nProperty < nBorderPropertyCount; ++nProperty) { - OUString sPropertyName = rPropNameSupplier.GetName(aBorderProperties[nProperty]); + OUString sPropertyName = getPropertyName(aBorderProperties[nProperty]); beans::PropertyValue aValue; aValue.Name = sPropertyName; aValue.Value = xTextRangeProperties->getPropertyValue(sPropertyName); @@ -826,7 +825,6 @@ void lcl_AddRangeAndStyle( void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) { - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); if (m_aTextAppendStack.empty()) return; TextAppendContext& rAppendContext = m_aTextAppendStack.top(); @@ -853,34 +851,34 @@ void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) bool bAutoWidth = nWidth < 1; if( bAutoWidth ) nWidth = DEFAULT_FRAME_MIN_WIDTH; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_WIDTH), nWidth)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_WIDTH), nWidth)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HEIGHT), + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HEIGHT), rAppendContext.pLastParagraphProperties->Geth() > 0 ? rAppendContext.pLastParagraphProperties->Geth() : pStyleProperties->Geth() > 0 ? pStyleProperties->Geth() : DEFAULT_FRAME_MIN_HEIGHT)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_SIZE_TYPE), sal_Int16( + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_SIZE_TYPE), sal_Int16( rAppendContext.pLastParagraphProperties->GethRule() >= 0 ? rAppendContext.pLastParagraphProperties->GethRule() : pStyleProperties->GethRule() >=0 ? pStyleProperties->GethRule() : text::SizeType::VARIABLE))); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_WIDTH_TYPE), bAutoWidth ? text::SizeType::MIN : text::SizeType::FIX)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_WIDTH_TYPE), bAutoWidth ? text::SizeType::MIN : text::SizeType::FIX)); sal_Int16 nHoriOrient = sal_Int16( rAppendContext.pLastParagraphProperties->GetxAlign() >= 0 ? rAppendContext.pLastParagraphProperties->GetxAlign() : pStyleProperties->GetxAlign() >= 0 ? pStyleProperties->GetxAlign() : text::HoriOrientation::NONE ); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HORI_ORIENT), nHoriOrient)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HORI_ORIENT), nHoriOrient)); //set a non negative default value - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HORI_ORIENT_POSITION), + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HORI_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->IsxValid() ? rAppendContext.pLastParagraphProperties->Getx() : pStyleProperties->IsxValid() ? pStyleProperties->Getx() : DEFAULT_VALUE)); //Default the anchor in case FramePr_hAnchor is missing ECMA 17.3.1.11 - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HORI_ORIENT_RELATION), sal_Int16( + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HORI_ORIENT_RELATION), sal_Int16( rAppendContext.pLastParagraphProperties->GethAnchor() >= 0 ? rAppendContext.pLastParagraphProperties->GethAnchor() : pStyleProperties->GethAnchor() >=0 ? pStyleProperties->GethAnchor() : text::RelOrientation::FRAME ))); @@ -889,21 +887,21 @@ void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) rAppendContext.pLastParagraphProperties->GetyAlign() >= 0 ? rAppendContext.pLastParagraphProperties->GetyAlign() : pStyleProperties->GetyAlign() >= 0 ? pStyleProperties->GetyAlign() : text::VertOrientation::NONE ); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_VERT_ORIENT), nVertOrient)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_VERT_ORIENT), nVertOrient)); //set a non negative default value - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_VERT_ORIENT_POSITION), + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_VERT_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->IsyValid() ? rAppendContext.pLastParagraphProperties->Gety() : pStyleProperties->IsyValid() ? pStyleProperties->Gety() : DEFAULT_VALUE)); //Default the anchor in case FramePr_vAnchor is missing ECMA 17.3.1.11 - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_VERT_ORIENT_RELATION), sal_Int16( + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_VERT_ORIENT_RELATION), sal_Int16( rAppendContext.pLastParagraphProperties->GetvAnchor() >= 0 ? rAppendContext.pLastParagraphProperties->GetvAnchor() : pStyleProperties->GetvAnchor() >= 0 ? pStyleProperties->GetvAnchor() : text::RelOrientation::FRAME ))); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_SURROUND), text::WrapTextMode( + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_SURROUND), text::WrapTextMode( rAppendContext.pLastParagraphProperties->GetWrap() >= 0 ? rAppendContext.pLastParagraphProperties->GetWrap() : pStyleProperties->GetWrap() >= 0 ? pStyleProperties->GetWrap() : 0 ))); @@ -918,8 +916,8 @@ void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) rAppendContext.pLastParagraphProperties->GethSpace() : pStyleProperties->GethSpace() >= 0 ? pStyleProperties->GethSpace() : 0; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_LEFT_MARGIN), nHoriOrient == text::HoriOrientation::LEFT ? 0 : nLeftDist)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_RIGHT_MARGIN), nHoriOrient == text::HoriOrientation::RIGHT ? 0 : nRightDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_LEFT_MARGIN), nHoriOrient == text::HoriOrientation::LEFT ? 0 : nLeftDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_RIGHT_MARGIN), nHoriOrient == text::HoriOrientation::RIGHT ? 0 : nRightDist)); sal_Int32 nBottomDist; sal_Int32 nTopDist = nBottomDist = @@ -927,12 +925,12 @@ void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) rAppendContext.pLastParagraphProperties->GetvSpace() : pStyleProperties->GetvSpace() >= 0 ? pStyleProperties->GetvSpace() : 0; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_TOP_MARGIN), nVertOrient == text::VertOrientation::TOP ? 0 : nTopDist)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_BOTTOM_MARGIN), nVertOrient == text::VertOrientation::BOTTOM ? 0 : nBottomDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_TOP_MARGIN), nVertOrient == text::VertOrientation::TOP ? 0 : nTopDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_BOTTOM_MARGIN), nVertOrient == text::VertOrientation::BOTTOM ? 0 : nBottomDist)); // If there is no fill, the Word default is 100% transparency. // Otherwise CellColorHandler has priority, and this setting // will be ignored. - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_BACK_COLOR_TRANSPARENCY), sal_Int32(100))); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_BACK_COLOR_TRANSPARENCY), sal_Int32(100))); beans::PropertyValue aRet; uno::Sequence<beans::PropertyValue> aGrabBag(1); @@ -951,50 +949,50 @@ void DomainMapper_Impl::CheckUnregisteredFrameConversion( ) bool bAutoWidth = nWidth < 1; if( bAutoWidth ) nWidth = DEFAULT_FRAME_MIN_WIDTH; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_WIDTH), nWidth)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_WIDTH), nWidth)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_SIZE_TYPE), sal_Int16( + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_SIZE_TYPE), sal_Int16( rAppendContext.pLastParagraphProperties->GethRule() >= 0 ? rAppendContext.pLastParagraphProperties->GethRule() : text::SizeType::VARIABLE))); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_WIDTH_TYPE), bAutoWidth ? text::SizeType::MIN : text::SizeType::FIX)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_WIDTH_TYPE), bAutoWidth ? text::SizeType::MIN : text::SizeType::FIX)); sal_Int16 nHoriOrient = sal_Int16( rAppendContext.pLastParagraphProperties->GetxAlign() >= 0 ? rAppendContext.pLastParagraphProperties->GetxAlign() : text::HoriOrientation::NONE ); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HORI_ORIENT), nHoriOrient)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HORI_ORIENT), nHoriOrient)); sal_Int16 nVertOrient = sal_Int16( rAppendContext.pLastParagraphProperties->GetyAlign() >= 0 ? rAppendContext.pLastParagraphProperties->GetyAlign() : text::VertOrientation::NONE ); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_VERT_ORIENT), nVertOrient)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_VERT_ORIENT), nVertOrient)); sal_Int32 nVertDist = rAppendContext.pLastParagraphProperties->GethSpace(); if( nVertDist < 0 ) nVertDist = 0; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_LEFT_MARGIN), nVertOrient == text::VertOrientation::TOP ? 0 : nVertDist)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_RIGHT_MARGIN), nVertOrient == text::VertOrientation::BOTTOM ? 0 : nVertDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_LEFT_MARGIN), nVertOrient == text::VertOrientation::TOP ? 0 : nVertDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_RIGHT_MARGIN), nVertOrient == text::VertOrientation::BOTTOM ? 0 : nVertDist)); sal_Int32 nHoriDist = rAppendContext.pLastParagraphProperties->GetvSpace(); if( nHoriDist < 0 ) nHoriDist = 0; - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_TOP_MARGIN), nHoriOrient == text::HoriOrientation::LEFT ? 0 : nHoriDist)); - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_BOTTOM_MARGIN), nHoriOrient == text::HoriOrientation::RIGHT ? 0 : nHoriDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_TOP_MARGIN), nHoriOrient == text::HoriOrientation::LEFT ? 0 : nHoriDist)); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_BOTTOM_MARGIN), nHoriOrient == text::HoriOrientation::RIGHT ? 0 : nHoriDist)); if( rAppendContext.pLastParagraphProperties->Geth() > 0 ) - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HEIGHT), rAppendContext.pLastParagraphProperties->Geth())); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HEIGHT), rAppendContext.pLastParagraphProperties->Geth())); if( rAppendContext.pLastParagraphProperties->IsxValid() ) - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_HORI_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->Getx())); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_HORI_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->Getx())); if( rAppendContext.pLastParagraphProperties->GethAnchor() >= 0 ) aFrameProperties.push_back(comphelper::makePropertyValue("HoriOrientRelation", sal_Int16(rAppendContext.pLastParagraphProperties->GethAnchor()))); if( rAppendContext.pLastParagraphProperties->IsyValid() ) - aFrameProperties.push_back(comphelper::makePropertyValue(rPropNameSupplier.GetName(PROP_VERT_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->Gety())); + aFrameProperties.push_back(comphelper::makePropertyValue(getPropertyName(PROP_VERT_ORIENT_POSITION), rAppendContext.pLastParagraphProperties->Gety())); if( rAppendContext.pLastParagraphProperties->GetvAnchor() >= 0 ) aFrameProperties.push_back(comphelper::makePropertyValue("VertOrientRelation", sal_Int16(rAppendContext.pLastParagraphProperties->GetvAnchor()))); @@ -1032,7 +1030,6 @@ void DomainMapper_Impl::finishParagraph( PropertyMapPtr pPropertyMap ) uno::Reference< text::XTextAppend > xTextAppend; if (!m_aTextAppendStack.empty()) xTextAppend = rAppendContext.xTextAppend; - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); #ifdef DEBUG_WRITERFILTER TagLogger::getInstance().attribute("isTextAppend", sal_uInt32(xTextAppend.is())); @@ -1071,8 +1068,8 @@ void DomainMapper_Impl::finishParagraph( PropertyMapPtr pPropertyMap ) //select paragraph xParaCursor->gotoStartOfParagraph( true ); uno::Reference< beans::XPropertyState > xParaProperties( xParaCursor, uno::UNO_QUERY_THROW ); - xParaProperties->setPropertyToDefault(rPropNameSupplier.GetName(PROP_CHAR_ESCAPEMENT)); - xParaProperties->setPropertyToDefault(rPropNameSupplier.GetName(PROP_CHAR_HEIGHT)); + xParaProperties->setPropertyToDefault(getPropertyName(PROP_CHAR_ESCAPEMENT)); + xParaProperties->setPropertyToDefault(getPropertyName(PROP_CHAR_HEIGHT)); //handles (2) and part of (6) pToBeSavedProperties.reset( new ParagraphProperties(*pParaContext) ); sal_Int32 nCount = xParaCursor->getString().getLength(); @@ -1130,7 +1127,7 @@ void DomainMapper_Impl::finishParagraph( PropertyMapPtr pPropertyMap ) if( aDrop.Lines > 1 ) { beans::PropertyValue aValue; - aValue.Name = rPropNameSupplier.GetName(PROP_DROP_CAP_FORMAT); + aValue.Name = getPropertyName(PROP_DROP_CAP_FORMAT); aValue.Value <<= aDrop; aProperties.push_back(aValue); } @@ -1339,7 +1336,7 @@ void DomainMapper_Impl::appendOLE( const OUString& rStreamName, OLEHandlerPtr pO OUString aCLSID = pOLEHandler->getCLSID(m_xComponentContext); if (aCLSID.isEmpty()) - xOLEProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_STREAM_NAME ), + xOLEProperties->setPropertyValue(getPropertyName( PROP_STREAM_NAME ), uno::makeAny( rStreamName )); else xOLEProperties->setPropertyValue("CLSID", uno::makeAny(OUString(aCLSID))); @@ -1349,13 +1346,13 @@ void DomainMapper_Impl::appendOLE( const OUString& rStreamName, OLEHandlerPtr pO aSize.Width = 1000; if( !aSize.Height ) aSize.Height = 1000; - xOLEProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_WIDTH ), + xOLEProperties->setPropertyValue(getPropertyName( PROP_WIDTH ), uno::makeAny(aSize.Width)); - xOLEProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_HEIGHT ), + xOLEProperties->setPropertyValue(getPropertyName( PROP_HEIGHT ), uno::makeAny(aSize.Height)); uno::Reference< graphic::XGraphic > xGraphic = pOLEHandler->getReplacement(); - xOLEProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_GRAPHIC ), + xOLEProperties->setPropertyValue(getPropertyName( PROP_GRAPHIC ), uno::makeAny(xGraphic)); uno::Reference<beans::XPropertySet> xReplacementProperties(pOLEHandler->getShape(), uno::UNO_QUERY); if (xReplacementProperties.is()) @@ -1374,7 +1371,7 @@ void DomainMapper_Impl::appendOLE( const OUString& rStreamName, OLEHandlerPtr pO else // mimic the treatment of graphics here.. it seems anchoring as character // gives a better ( visually ) result - xOLEProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_ANCHOR_TYPE ), uno::makeAny( text::TextContentAnchorType_AS_CHARACTER ) ); + xOLEProperties->setPropertyValue(getPropertyName( PROP_ANCHOR_TYPE ), uno::makeAny( text::TextContentAnchorType_AS_CHARACTER ) ); // remove ( if valid ) associated shape ( used for graphic replacement ) SAL_WARN_IF(m_aAnchoredStack.empty(), "writerfilter.dmapper", "no anchor stack"); if (!m_aAnchoredStack.empty()) @@ -1408,20 +1405,20 @@ void DomainMapper_Impl::appendStarMath( const Value& val ) uno::Reference< text::XTextContent > xStarMath( m_xTextFactory->createInstance(sEmbeddedService), uno::UNO_QUERY_THROW ); uno::Reference< beans::XPropertySet > xStarMathProperties(xStarMath, uno::UNO_QUERY_THROW); - xStarMathProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_EMBEDDED_OBJECT ), + xStarMathProperties->setPropertyValue(getPropertyName( PROP_EMBEDDED_OBJECT ), val.getAny()); uno::Reference< uno::XInterface > xInterface( formula->getComponent(), uno::UNO_QUERY ); Size size( 1000, 1000 ); if( oox::FormulaImportBase* formulaimport = dynamic_cast< oox::FormulaImportBase* >( xInterface.get())) size = formulaimport->getFormulaSize(); - xStarMathProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_WIDTH ), + xStarMathProperties->setPropertyValue(getPropertyName( PROP_WIDTH ), uno::makeAny( sal_Int32(size.Width()))); - xStarMathProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_HEIGHT ), + xStarMathProperties->setPropertyValue(getPropertyName( PROP_HEIGHT ), uno::makeAny( sal_Int32(size.Height()))); // mimic the treatment of graphics here.. it seems anchoring as character // gives a better ( visually ) result - xStarMathProperties->setPropertyValue(PropertyNameSupplier::GetPropertyNameSupplier().GetName( PROP_ANCHOR_TYPE ), + xStarMathProperties->setPropertyValue(getPropertyName( PROP_ANCHOR_TYPE ), uno::makeAny( text::TextContentAnchorType_AS_CHARACTER ) ); appendTextContent( xStarMath, uno::Sequence< beans::PropertyValue >() ); } @@ -1498,11 +1495,9 @@ void DomainMapper_Impl::PushPageHeaderFooter(bool bHeader, SectionPropertyMap::P bool bFirst = eType == SectionPropertyMap::PAGE_FIRST; if ((!bLeft && !GetSettingsTable()->GetEvenAndOddHeaders()) || (GetSettingsTable()->GetEvenAndOddHeaders())) { - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); - //switch on header/footer use xPageStyle->setPropertyValue( - rPropNameSupplier.GetName(ePropIsOn), + getPropertyName(ePropIsOn), uno::makeAny(sal_True)); if (bFirst) @@ -1512,18 +1507,18 @@ void DomainMapper_Impl::PushPageHeaderFooter(bool bHeader, SectionPropertyMap::P { // This is a first page and has a follow style, then enable the header/footer there as well to be consistent. uno::Reference<beans::XPropertySet> xFollowStyle(GetPageStyles()->getByName(aFollowStyle), uno::UNO_QUERY); - xFollowStyle->setPropertyValue(rPropNameSupplier.GetName(ePropIsOn), uno::makeAny(sal_True)); + xFollowStyle->setPropertyValue(getPropertyName(ePropIsOn), uno::makeAny(sal_True)); } } // If the 'Different Even & Odd Pages' flag is turned on - do not ignore it // Even if the 'Even' header/footer is blank - the flag should be imported (so it would look in LO like in Word) if (!bFirst && GetSettingsTable()->GetEvenAndOddHeaders()) - xPageStyle->setPropertyValue(rPropNameSupplier.GetName(ePropShared), uno::makeAny(false)); + xPageStyle->setPropertyValue(getPropertyName(ePropShared), uno::makeAny(false)); //set the interface uno::Reference< text::XText > xText; - xPageStyle->getPropertyValue(rPropNameSupplier.GetName(bLeft? ePropTextLeft: ePropText)) >>= xText; + xPageStyle->getPropertyValue(getPropertyName(bLeft? ePropTextLeft: ePropText)) >>= xText; m_aTextAppendStack.push(TextAppendContext(uno::Reference< text::XTextAppend >(xText, uno::UNO_QUERY_THROW), m_bIsNewDoc? uno::Reference<text::XTextCursor>(): m_xBodyText->createTextCursorByRange(xText->getStart()))); @@ -1636,20 +1631,19 @@ void DomainMapper_Impl::CreateRedline(uno::Reference<text::XTextRange> const& xR try { OUString sType; - PropertyNameSupplier & rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier( ); switch ( pRedline->m_nToken & 0xffff ) { case XML_mod: - sType = rPropNameSupplier.GetName( PROP_FORMAT ); + sType = getPropertyName( PROP_FORMAT ); break; case XML_ins: - sType = rPropNameSupplier.GetName( PROP_INSERT ); + sType = getPropertyName( PROP_INSERT ); break; case XML_del: - sType = rPropNameSupplier.GetName( PROP_DELETE ); + sType = getPropertyName( PROP_DELETE ); break; case XML_ParagraphFormat: - sType = rPropNameSupplier.GetName( PROP_PARAGRAPH_FORMAT ); + sType = getPropertyName( PROP_PARAGRAPH_FORMAT ); break; default: throw lang::IllegalArgumentException("illegal redline token type", nullptr, 0); @@ -1657,11 +1651,11 @@ void DomainMapper_Impl::CreateRedline(uno::Reference<text::XTextRange> const& xR uno::Reference < text::XRedline > xRedline( xRange, uno::UNO_QUERY_THROW ); beans::PropertyValues aRedlineProperties( 3 ); beans::PropertyValue * pRedlineProperties = aRedlineProperties.getArray( ); - pRedlineProperties[0].Name = rPropNameSupplier.GetName( PROP_REDLINE_AUTHOR ); + pRedlineProperties[0].Name = getPropertyName( PROP_REDLINE_AUTHOR ); pRedlineProperties[0].Value <<= pRedline->m_sAuthor; - pRedlineProperties[1].Name = rPropNameSupplier.GetName( PROP_REDLINE_DATE_TIME ); + pRedlineProperties[1].Name = getPropertyName( PROP_REDLINE_DATE_TIME ); pRedlineProperties[1].Value <<= ConversionHelper::ConvertDateStringToDateTime( pRedline->m_sDate ); - pRedlineProperties[2].Name = rPropNameSupplier.GetName( PROP_REDLINE_REVERT_PROPERTIES ); + pRedlineProperties[2].Name = getPropertyName( PROP_REDLINE_REVERT_PROPERTIES ); pRedlineProperties[2].Value <<= pRedline->m_aRevertProperties; xRedline->makeRedline( sType, aRedlineProperties ); } @@ -1862,12 +1856,10 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape m_aAnchoredStack.push(xTextContent); uno::Reference<beans::XPropertySet> xShapePropertySet(xShape, uno::UNO_QUERY); - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); - m_xEmbedded.set(m_xTextFactory->createInstance("com.sun.star.text.TextEmbeddedObject"), uno::UNO_QUERY_THROW); uno::Reference<beans::XPropertySet> xEmbeddedProperties(m_xEmbedded, uno::UNO_QUERY_THROW); - xEmbeddedProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_EMBEDDED_OBJECT), xShapePropertySet->getPropertyValue(rPropNameSupplier.GetName(PROP_EMBEDDED_OBJECT))); - xEmbeddedProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_ANCHOR_TYPE), uno::makeAny(text::TextContentAnchorType_AS_CHARACTER)); + xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_EMBEDDED_OBJECT), xShapePropertySet->getPropertyValue(getPropertyName(PROP_EMBEDDED_OBJECT))); + xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_ANCHOR_TYPE), uno::makeAny(text::TextContentAnchorType_AS_CHARACTER)); // So that the original bitmap-only shape will be replaced by the embedded object. m_aAnchoredStack.top().bToRemove = true; m_aTextAppendStack.pop(); @@ -1884,14 +1876,12 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape uno::Reference< text::XTextContent > xTxtContent( xShape, uno::UNO_QUERY_THROW ); m_aAnchoredStack.push( xTxtContent ); - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); - uno::Reference< beans::XPropertySet > xProps( xShape, uno::UNO_QUERY_THROW ); #ifdef DEBUG_WRITERFILTER TagLogger::getInstance().unoPropertySet(xProps); #endif text::TextContentAnchorType nAnchorType(text::TextContentAnchorType_AT_PARAGRAPH); - xProps->getPropertyValue(rPropNameSupplier.GetName( PROP_ANCHOR_TYPE )) >>= nAnchorType; + xProps->getPropertyValue(getPropertyName( PROP_ANCHOR_TYPE )) >>= nAnchorType; bool checkZOrderStatus = false; if (xSInfo->supportsService("com.sun.star.text.TextFrame")) { @@ -1947,7 +1937,7 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape PropertyMapPtr paragraphContext = GetTopContextOfType( CONTEXT_PARAGRAPH ); boost::optional<PropertyMap::Property> aPropMargin = paragraphContext->getProperty(PROP_PARA_BOTTOM_MARGIN); if(aPropMargin) - xProps->setPropertyValue( rPropNameSupplier.GetName( PROP_BOTTOM_MARGIN ), aPropMargin->second ); + xProps->setPropertyValue( getPropertyName( PROP_BOTTOM_MARGIN ), aPropMargin->second ); } else { @@ -1963,7 +1953,7 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape aGrabBag[i].Value >>= zOrder; xShapePropertySet->setPropertyValue( "ZOrder", uno::makeAny(pZOrderHelper->findZOrder(zOrder))); pZOrderHelper->addItem(xShapePropertySet, zOrder); - xShapePropertySet->setPropertyValue(rPropNameSupplier.GetName( PROP_OPAQUE ), uno::makeAny( false ) ); + xShapePropertySet->setPropertyValue(getPropertyName( PROP_OPAQUE ), uno::makeAny( false ) ); checkZOrderStatus = true; } else if ( aGrabBag[i].Name == "TxbxHasLink" ) @@ -1996,7 +1986,7 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape } if (!m_bInHeaderFooterImport && !checkZOrderStatus) xProps->setPropertyValue( - rPropNameSupplier.GetName( PROP_OPAQUE ), + getPropertyName( PROP_OPAQUE ), uno::makeAny( true ) ); } m_bParaChanged = true; @@ -2015,11 +2005,10 @@ void DomainMapper_Impl::UpdateEmbeddedShapeProps(const uno::Reference< drawing:: if (!xShape.is()) return; - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); uno::Reference<beans::XPropertySet> xEmbeddedProperties(m_xEmbedded, uno::UNO_QUERY_THROW); awt::Size aSize = xShape->getSize( ); - xEmbeddedProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_WIDTH), uno::makeAny(sal_Int32(aSize.Width))); - xEmbeddedProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_HEIGHT), uno::makeAny(sal_Int32(aSize.Height))); + xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_WIDTH), uno::makeAny(sal_Int32(aSize.Width))); + xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_HEIGHT), uno::makeAny(sal_Int32(aSize.Height))); } @@ -2439,10 +2428,10 @@ void DomainMapper_Impl::SetNumberFormat( const OUString& rCommand, nKey = xNumberSupplier->getNumberFormats()->addNewConverted( sFormat, aUSLocale, aCurrentLocale ); } xPropertySet->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName(PROP_NUMBER_FORMAT), + getPropertyName(PROP_NUMBER_FORMAT), uno::makeAny( nKey )); xPropertySet->getPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName(PROP_NUMBER_FORMAT ) ) >>= nKey; + getPropertyName(PROP_NUMBER_FORMAT ) ) >>= nKey; } catch(const uno::Exception&) { @@ -2634,7 +2623,7 @@ uno::Reference<beans::XPropertySet> DomainMapper_Impl::FindOrCreateFieldMaster(c m_xTextFactory->createInstance(sFieldMasterService), uno::UNO_QUERY_THROW); //set the master's name xMaster->setPropertyValue( - PropertyNameSupplier::GetPropertyNameSupplier().GetName(PROP_NAME), + getPropertyName(PROP_NAME), uno::makeAny(rFieldMasterName)); } return xMaster; @@ -2905,7 +2894,6 @@ const FieldConversionMap_t & lcl_GetEnhancedFieldConversion() void DomainMapper_Impl::handleFieldAsk (FieldContextPtr pContext, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & xFieldInterface, uno::Reference< beans::XPropertySet > const& xFieldProperties) { @@ -2921,7 +2909,7 @@ void DomainMapper_Impl::handleFieldAsk FindOrCreateFieldMaster ("com.sun.star.text.FieldMaster.SetExpression", sVariable ); // An ASK field is always a string of characters - xMaster->setPropertyValue(rPropNameSupplier.GetName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); + xMaster->setPropertyValue(getPropertyName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); // attach the master to the field uno::Reference< text::XDependentTextField > xDependentField @@ -2930,14 +2918,14 @@ void DomainMapper_Impl::handleFieldAsk // set input flag at the field xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_INPUT), uno::makeAny( true )); + getPropertyName(PROP_IS_INPUT), uno::makeAny( true )); // set the prompt xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_HINT), + getPropertyName(PROP_HINT), uno::makeAny( sHint )); - xFieldProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); + xFieldProperties->setPropertyValue(getPropertyName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); // The ASK has no field value to display - xFieldProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_IS_VISIBLE), uno::makeAny(sal_False)); + xFieldProperties->setPropertyValue(getPropertyName(PROP_IS_VISIBLE), uno::makeAny(sal_False)); } else { @@ -2949,7 +2937,6 @@ void DomainMapper_Impl::handleFieldAsk void DomainMapper_Impl::handleAutoNum (FieldContextPtr pContext, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & xFieldInterface, uno::Reference< beans::XPropertySet > const& xFieldProperties) { @@ -2959,12 +2946,12 @@ void DomainMapper_Impl::handleAutoNum ("com.sun.star.text.FieldMaster.SetExpression", "AutoNr"); - xMaster->setPropertyValue( rPropNameSupplier.GetName(PROP_SUB_TYPE), + xMaster->setPropertyValue( getPropertyName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::SEQUENCE)); //apply the numbering type xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny( lcl_ParseNumberingType(pContext->GetCommand()) )); // attach the master to the field uno::Reference< text::XDependentTextField > xDependentField @@ -2974,19 +2961,18 @@ void DomainMapper_Impl::handleAutoNum void DomainMapper_Impl::handleAuthor (OUString const& rFirstParam, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & /*xFieldInterface*/, uno::Reference< beans::XPropertySet > const& xFieldProperties, FieldId eFieldId ) { if ( eFieldId != FIELD_USERINITIALS ) xFieldProperties->setPropertyValue - ( rPropNameSupplier.GetName(PROP_FULL_NAME), uno::makeAny( true )); + ( getPropertyName(PROP_FULL_NAME), uno::makeAny( true )); if (!rFirstParam.isEmpty()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_FIXED ), + getPropertyName( PROP_IS_FIXED ), uno::makeAny( true )); //PROP_CURRENT_PRESENTATION is set later anyway } @@ -2995,7 +2981,6 @@ void DomainMapper_Impl::handleAuthor void DomainMapper_Impl::handleDocProperty (FieldContextPtr pContext, OUString const& rFirstParam, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & xFieldInterface, uno::Reference< beans::XPropertySet > const&) { @@ -3074,23 +3059,23 @@ void DomainMapper_Impl::handleAuthor if( bIsCustomField ) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NAME), uno::makeAny(rFirstParam)); + getPropertyName(PROP_NAME), uno::makeAny(rFirstParam)); pContext->SetCustomField( xFieldProperties ); } else { if(0 != (aDocProperties[nMap].nFlags & SET_ARABIC)) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny( style::NumberingType::ARABIC )); else if(0 != (aDocProperties[nMap].nFlags & SET_FULL_NAME)) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_FULL_NAME), + getPropertyName(PROP_FULL_NAME), uno::makeAny( true )); else if(0 != (aDocProperties[nMap].nFlags & SET_DATE)) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_DATE), + getPropertyName(PROP_IS_DATE), uno::makeAny( true )); SetNumberFormat( pContext->GetCommand(), xFieldProperties ); } @@ -3103,8 +3088,7 @@ void DomainMapper_Impl::handleAuthor } uno::Sequence< beans::PropertyValues > lcl_createTOXLevelHyperlinks( bool bHyperlinks, const OUString& sChapterNoSeparator, - const uno::Sequence< beans::PropertyValues >& aLevel, - PropertyNameSupplier& rPropNameSupplier ) + const uno::Sequence< beans::PropertyValues >& aLevel ) { //create a copy of the level and add two new entries - hyperlink start and end bool bChapterNoSeparator = !sChapterNoSeparator.isEmpty(); @@ -3114,26 +3098,26 @@ uno::Sequence< beans::PropertyValues > lcl_createTOXLevelHyperlinks( bool bHyper if( bHyperlinks ) { beans::PropertyValues aHyperlink(1); - aHyperlink[0].Name = rPropNameSupplier.GetName( PROP_TOKEN_TYPE ); - aHyperlink[0].Value <<= rPropNameSupplier.GetName( PROP_TOKEN_HYPERLINK_START ); + aHyperlink[0].Name = getPropertyName( PROP_TOKEN_TYPE ); + aHyperlink[0].Value <<= getPropertyName( PROP_TOKEN_HYPERLINK_START ); pNewLevel[0] = aHyperlink; - aHyperlink[0].Value <<= rPropNameSupplier.GetName( PROP_TOKEN_HYPERLINK_END ); + aHyperlink[0].Value <<= getPropertyName( PROP_TOKEN_HYPERLINK_END ); pNewLevel[aNewLevel.getLength() -1] = aHyperlink; } if( bChapterNoSeparator ) { beans::PropertyValues aChapterNo(2); - aChapterNo[0].Name = rPropNameSupplier.GetName( PROP_TOKEN_TYPE ); - aChapterNo[0].Value <<= rPropNameSupplier.GetName( PROP_TOKEN_CHAPTER_INFO ); - aChapterNo[1].Name = rPropNameSupplier.GetName( PROP_CHAPTER_FORMAT ); + aChapterNo[0].Name = getPropertyName( PROP_TOKEN_TYPE ); + aChapterNo[0].Value <<= getPropertyName( PROP_TOKEN_CHAPTER_INFO ); + aChapterNo[1].Name = getPropertyName( PROP_CHAPTER_FORMAT ); //todo: is ChapterFormat::Number correct? aChapterNo[1].Value <<= (sal_Int16)text::ChapterFormat::NUMBER; pNewLevel[aNewLevel.getLength() - (bHyperlinks ? 4 : 2) ] = aChapterNo; beans::PropertyValues aChapterSeparator(2); - aChapterSeparator[0].Name = rPropNameSupplier.GetName( PROP_TOKEN_TYPE ); - aChapterSeparator[0].Value <<= rPropNameSupplier.GetName( PROP_TOKEN_TEXT ); - aChapterSeparator[1].Name = rPropNameSupplier.GetName( PROP_TEXT ); + aChapterSeparator[0].Name = getPropertyName( PROP_TOKEN_TYPE ); + aChapterSeparator[0].Value <<= getPropertyName( PROP_TOKEN_TEXT ); + aChapterSeparator[1].Name = getPropertyName( PROP_TEXT ); aChapterSeparator[1].Value <<= sChapterNoSeparator; pNewLevel[aNewLevel.getLength() - (bHyperlinks ? 3 : 1)] = aChapterSeparator; } @@ -3151,7 +3135,6 @@ uno::Sequence< beans::PropertyValues > lcl_createTOXLevelHyperlinks( bool bHyper void DomainMapper_Impl::handleToc (FieldContextPtr pContext, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & /*xFieldInterface*/, uno::Reference< beans::XPropertySet > const& /*xFieldProperties*/, const OUString & sTOCServiceName) @@ -3284,18 +3267,18 @@ void DomainMapper_Impl::handleToc : sTOCServiceName), uno::UNO_QUERY_THROW); if (xTOC.is()) - xTOC->setPropertyValue(rPropNameSupplier.GetName( PROP_TITLE ), uno::makeAny(OUString())); + xTOC->setPropertyValue(getPropertyName( PROP_TITLE ), uno::makeAny(OUString())); if (!aBookmarkName.isEmpty()) - xTOC->setPropertyValue(rPropNameSupplier.GetName(PROP_TOC_BOOKMARK), uno::makeAny(aBookmarkName)); + xTOC->setPropertyValue(getPropertyName(PROP_TOC_BOOKMARK), uno::makeAny(aBookmarkName)); if( !bTableOfFigures && xTOC.is() ) { - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_LEVEL ), uno::makeAny( nMaxLevel ) ); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_CREATE_FROM_OUTLINE ), uno::makeAny( bFromOutline )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_CREATE_FROM_MARKS ), uno::makeAny( bFromEntries )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_HIDE_TAB_LEADER_AND_PAGE_NUMBERS ), uno::makeAny( bHideTabLeaderPageNumbers )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_TAB_IN_TOC ), uno::makeAny( bIsTabEntry )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_TOC_NEW_LINE ), uno::makeAny( bNewLine )); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_TOC_PARAGRAPH_OUTLINE_LEVEL ), uno::makeAny( bParagraphOutlineLevel )); + xTOC->setPropertyValue( getPropertyName( PROP_LEVEL ), uno::makeAny( nMaxLevel ) ); + xTOC->setPropertyValue( getPropertyName( PROP_CREATE_FROM_OUTLINE ), uno::makeAny( bFromOutline )); + xTOC->setPropertyValue( getPropertyName( PROP_CREATE_FROM_MARKS ), uno::makeAny( bFromEntries )); + xTOC->setPropertyValue( getPropertyName( PROP_HIDE_TAB_LEADER_AND_PAGE_NUMBERS ), uno::makeAny( bHideTabLeaderPageNumbers )); + xTOC->setPropertyValue( getPropertyName( PROP_TAB_IN_TOC ), uno::makeAny( bIsTabEntry )); + xTOC->setPropertyValue( getPropertyName( PROP_TOC_NEW_LINE ), uno::makeAny( bNewLine )); + xTOC->setPropertyValue( getPropertyName( PROP_TOC_PARAGRAPH_OUTLINE_LEVEL ), uno::makeAny( bParagraphOutlineLevel )); if( !sTemplate.isEmpty() ) { //the string contains comma separated the names and related levels @@ -3316,7 +3299,7 @@ void DomainMapper_Impl::handleToc aMap.insert( TOCStyleMap::value_type(nLevel, sStyleName) ); } uno::Reference< container::XIndexReplace> xParaStyles; - xTOC->getPropertyValue(rPropNameSupplier.GetName(PROP_LEVEL_PARAGRAPH_STYLES)) >>= xParaStyles; + xTOC->getPropertyValue(getPropertyName(PROP_LEVEL_PARAGRAPH_STYLES)) >>= xParaStyles; for( nLevel = 1; nLevel < 10; ++nLevel) { sal_Int32 nLevelCount = aMap.count( nLevel ); @@ -3332,13 +3315,13 @@ void DomainMapper_Impl::handleToc xParaStyles->replaceByIndex(nLevel - 1, uno::makeAny(aStyles)); } } - xTOC->setPropertyValue(rPropNameSupplier.GetName(PROP_CREATE_FROM_LEVEL_PARAGRAPH_STYLES), uno::makeAny( true )); + xTOC->setPropertyValue(getPropertyName(PROP_CREATE_FROM_LEVEL_PARAGRAPH_STYLES), uno::makeAny( true )); } if(bHyperlinks || !sChapterNoSeparator.isEmpty()) { uno::Reference< container::XIndexReplace> xLevelFormats; - xTOC->getPropertyValue(rPropNameSupplier.GetName(PROP_LEVEL_FORMAT)) >>= xLevelFormats; + xTOC->getPropertyValue(getPropertyName(PROP_LEVEL_FORMAT)) >>= xLevelFormats; sal_Int32 nLevelCount = xLevelFormats->getCount(); //start with level 1, 0 is the header level for( sal_Int32 nLevel = 1; nLevel < nLevelCount; ++nLevel) @@ -3348,7 +3331,7 @@ void DomainMapper_Impl::handleToc uno::Sequence< beans::PropertyValues > aNewLevel = lcl_createTOXLevelHyperlinks( bHyperlinks, sChapterNoSeparator, - aLevel, rPropNameSupplier ); + aLevel ); xLevelFormats->replaceByIndex( nLevel, uno::makeAny( aNewLevel ) ); } } @@ -3356,19 +3339,19 @@ void DomainMapper_Impl::handleToc else if (bTableOfFigures && xTOC.is()) { if (!sFigureSequence.isEmpty()) - xTOC->setPropertyValue(rPropNameSupplier.GetName(PROP_LABEL_CATEGORY), + xTOC->setPropertyValue(getPropertyName(PROP_LABEL_CATEGORY), uno::makeAny(sFigureSequence)); if ( bHyperlinks ) { uno::Reference< container::XIndexReplace> xLevelFormats; - xTOC->getPropertyValue(rPropNameSupplier.GetName(PROP_LEVEL_FORMAT)) >>= xLevelFormats; + xTOC->getPropertyValue(getPropertyName(PROP_LEVEL_FORMAT)) >>= xLevelFormats; uno::Sequence< beans::PropertyValues > aLevel; xLevelFormats->getByIndex( 1 ) >>= aLevel; uno::Sequence< beans::PropertyValues > aNewLevel = lcl_createTOXLevelHyperlinks( bHyperlinks, sChapterNoSeparator, - aLevel, rPropNameSupplier ); + aLevel ); xLevelFormats->replaceByIndex( 1, uno::makeAny( aNewLevel ) ); } } @@ -3396,7 +3379,6 @@ void DomainMapper_Impl::handleToc void DomainMapper_Impl::handleBibliography (FieldContextPtr pContext, - PropertyNameSupplier& rPropNameSupplier, const OUString & sTOCServiceName) { uno::Reference< beans::XPropertySet > xTOC; @@ -3408,7 +3390,7 @@ void DomainMapper_Impl::handleBibliography sTOCServiceName), uno::UNO_QUERY_THROW); if (xTOC.is()) - xTOC->setPropertyValue(rPropNameSupplier.GetName( PROP_TITLE ), uno::makeAny(OUString())); + xTOC->setPropertyValue(getPropertyName( PROP_TITLE ), uno::makeAny(OUString())); pContext->SetTOC( xTOC ); m_bParaHadField = false; @@ -3419,7 +3401,6 @@ void DomainMapper_Impl::handleBibliography void DomainMapper_Impl::handleIndex (FieldContextPtr pContext, - PropertyNameSupplier& rPropNameSupplier, uno::Reference< uno::XInterface > & /*xFieldInterface*/, uno::Reference< beans::XPropertySet > const& /*xFieldProperties*/, const OUString & sTOCServiceName) @@ -3438,7 +3419,7 @@ void DomainMapper_Impl::handleIndex uno::UNO_QUERY_THROW); if (xTOC.is()) { - xTOC->setPropertyValue(rPropNameSupplier.GetName( PROP_TITLE ), uno::makeAny(OUString())); + xTOC->setPropertyValue(getPropertyName( PROP_TITLE ), uno::makeAny(OUString())); if( lcl_FindInCommand( pContext->GetCommand(), 'r', sValue )) { @@ -3452,7 +3433,7 @@ void DomainMapper_Impl::handleIndex { if(!sValue.isEmpty()) sIndexEntryType = sValue ; - xTOC->setPropertyValue(rPropNameSupplier.GetName( PROP_INDEX_ENTRY_TYPE ), uno::makeAny(sIndexEntryType)); + xTOC->setPropertyValue(getPropertyName( PROP_INDEX_ENTRY_TYPE ), uno::makeAny(sIndexEntryType)); } } pContext->SetTOC( xTOC ); @@ -3465,11 +3446,11 @@ void DomainMapper_Impl::handleIndex { sValue = sValue.replaceAll("\"", ""); uno::Reference<text::XTextColumns> xTextColumns; - xTOC->getPropertyValue(rPropNameSupplier.GetName( PROP_TEXT_COLUMNS )) >>= xTextColumns; + xTOC->getPropertyValue(getPropertyName( PROP_TEXT_COLUMNS )) >>= xTextColumns; if (xTextColumns.is()) { xTextColumns->setColumnCount( sValue.toInt32() ); - xTOC->setPropertyValue( rPropNameSupplier.GetName( PROP_TEXT_COLUMNS ), uno::makeAny( xTextColumns ) ); + xTOC->setPropertyValue( getPropertyName( PROP_TEXT_COLUMNS ), uno::makeAny( xTextColumns ) ); } } } @@ -3578,23 +3559,22 @@ void DomainMapper_Impl::CloseFieldCommand() xFieldProperties = uno::Reference< beans::XPropertySet >( xFieldInterface, uno::UNO_QUERY_THROW); } } - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); switch( aIt->second.eFieldId ) { case FIELD_ADDRESSBLOCK: break; case FIELD_ADVANCE : break; case FIELD_ASK : - handleFieldAsk(pContext, rPropNameSupplier, xFieldInterface, xFieldProperties); + handleFieldAsk(pContext, xFieldInterface, xFieldProperties); break; case FIELD_AUTONUM : case FIELD_AUTONUMLGL : case FIELD_AUTONUMOUT : - handleAutoNum(pContext, rPropNameSupplier, xFieldInterface, xFieldProperties); + handleAutoNum(pContext, xFieldInterface, xFieldProperties); break; case FIELD_AUTHOR : case FIELD_USERNAME : case FIELD_USERINITIALS : - handleAuthor(sFirstParam, rPropNameSupplier, + handleAuthor(sFirstParam, xFieldInterface, xFieldProperties, aIt->second.eFieldId); break; @@ -3605,17 +3585,17 @@ void DomainMapper_Impl::CloseFieldCommand() if (pContext->IsFieldLocked()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_FIXED), + getPropertyName(PROP_IS_FIXED), uno::makeAny( true )); m_bSetDateValue = true; } else xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_FIXED), + getPropertyName(PROP_IS_FIXED), uno::makeAny( false )); xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_DATE), + getPropertyName(PROP_IS_DATE), uno::makeAny( true )); SetNumberFormat( pContext->GetCommand(), xFieldProperties ); } @@ -3632,19 +3612,19 @@ void DomainMapper_Impl::CloseFieldCommand() // overwrite the Comments document property // #TODO implement the above too xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_FIXED ), uno::makeAny( false )); + getPropertyName( PROP_IS_FIXED ), uno::makeAny( false )); //PROP_CURRENT_PRESENTATION is set later anyway } break; case FIELD_CREATEDATE : { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_DATE ), uno::makeAny( true )); + getPropertyName( PROP_IS_DATE ), uno::makeAny( true )); SetNumberFormat( pContext->GetCommand(), xFieldProperties ); } break; case FIELD_DOCPROPERTY : - handleDocProperty(pContext, sFirstParam, rPropNameSupplier, + handleDocProperty(pContext, sFirstParam, xFieldInterface, xFieldProperties); break; case FIELD_DOCVARIABLE : @@ -3672,7 +3652,7 @@ void DomainMapper_Impl::CloseFieldCommand() xFieldProperties = uno::Reference< beans::XPropertySet >( xFieldInterface, uno::UNO_QUERY_THROW); - xFieldProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_CONTENT), uno::makeAny(aResult.sResult)); + xFieldProperties->setPropertyValue(getPropertyName(PROP_CONTENT), uno::makeAny(aResult.sResult)); } else { @@ -3711,7 +3691,7 @@ void DomainMapper_Impl::CloseFieldCommand() sal_Int32 nIndex = 0; if (xFieldProperties.is()) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_HINT), uno::makeAny( pContext->GetCommand().getToken( 1, '\"', nIndex))); + getPropertyName(PROP_HINT), uno::makeAny( pContext->GetCommand().getToken( 1, '\"', nIndex))); } break; case FIELD_FILENAME: @@ -3719,7 +3699,7 @@ void DomainMapper_Impl::CloseFieldCommand() sal_Int32 nNumberingTypeIndex = pContext->GetCommand().indexOf("\\p"); if (xFieldProperties.is()) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_FILE_FORMAT), + getPropertyName(PROP_FILE_FORMAT), uno::makeAny( nNumberingTypeIndex > 0 ? text::FilenameDisplayFormat::FULL : text::FilenameDisplayFormat::NAME_AND_EXT )); } break; @@ -3754,7 +3734,7 @@ void DomainMapper_Impl::CloseFieldCommand() if ( aIt->second.eFieldId == FIELD_FORMDROPDOWN ) lcl_handleDropdownField( xFieldProperties, pContext->getFFDataHandler() ); else - lcl_handleTextField( xFieldProperties, pContext->getFFDataHandler(), rPropNameSupplier ); + lcl_handleTextField( xFieldProperties, pContext->getFFDataHandler() ); } } break; @@ -3819,7 +3799,7 @@ void DomainMapper_Impl::CloseFieldCommand() if (!sFirstParam.isEmpty()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_FIXED ), uno::makeAny( true )); + getPropertyName( PROP_IS_FIXED ), uno::makeAny( true )); //PROP_CURRENT_PRESENTATION is set later anyway } } @@ -3832,13 +3812,13 @@ void DomainMapper_Impl::CloseFieldCommand() OUString sMacro = pContext->GetCommand().getToken( 0, ' ', nIndex); if (xFieldProperties.is()) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_MACRO_NAME), uno::makeAny( sMacro )); + getPropertyName(PROP_MACRO_NAME), uno::makeAny( sMacro )); //extract quick help text if(xFieldProperties.is() && pContext->GetCommand().getLength() > nIndex + 1) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_HINT), + getPropertyName(PROP_HINT), uno::makeAny( pContext->GetCommand().copy( nIndex ))); } } @@ -3865,10 +3845,10 @@ void DomainMapper_Impl::CloseFieldCommand() if (xFieldProperties.is()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny( lcl_ParseNumberingType(pContext->GetCommand()) )); xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_SUB_TYPE), + getPropertyName(PROP_SUB_TYPE), uno::makeAny( text::PageNumberType_CURRENT )); } @@ -3888,10 +3868,10 @@ void DomainMapper_Impl::CloseFieldCommand() + sFirstParam)) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_REFERENCE_FIELD_SOURCE), + getPropertyName(PROP_REFERENCE_FIELD_SOURCE), uno::makeAny( sal_Int16(text::ReferenceFieldSource::BOOKMARK)) ); xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_SOURCE_NAME), + getPropertyName(PROP_SOURCE_NAME), uno::makeAny(sFirstParam) ); sal_Int16 nFieldPart = (bPageRef ? text::ReferenceFieldPart::PAGE : text::ReferenceFieldPart::TEXT); OUString sValue; @@ -3916,16 +3896,16 @@ void DomainMapper_Impl::CloseFieldCommand() nFieldPart = text::ReferenceFieldPart::NUMBER_FULL_CONTEXT; } xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_REFERENCE_FIELD_PART ), uno::makeAny( nFieldPart )); + getPropertyName( PROP_REFERENCE_FIELD_PART ), uno::makeAny( nFieldPart )); } else { xFieldInterface = m_xTextFactory->createInstance("com.sun.star.text.TextField.GetExpression"); xFieldProperties.set(xFieldInterface, uno::UNO_QUERY); xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_CONTENT), + getPropertyName(PROP_CONTENT), uno::makeAny(sFirstParam)); - xFieldProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); + xFieldProperties->setPropertyValue(getPropertyName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::STRING)); } } break; @@ -3949,12 +3929,12 @@ void DomainMapper_Impl::CloseFieldCommand() sSeqName); xMaster->setPropertyValue( - rPropNameSupplier.GetName(PROP_SUB_TYPE), + getPropertyName(PROP_SUB_TYPE), uno::makeAny(text::SetVariableType::SEQUENCE)); // apply the numbering type xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny( lcl_ParseNumberingType(pContext->GetCommand()) )); // attach the master to the field @@ -3974,7 +3954,7 @@ void DomainMapper_Impl::CloseFieldCommand() // TODO \s isn't handled, but the spec isn't easy to understand without // an example for this one. xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_CONTENT), + getPropertyName(PROP_CONTENT), uno::makeAny(sFormula)); // Take care of the numeric formatting definition, default is Arabic @@ -3982,7 +3962,7 @@ void DomainMapper_Impl::CloseFieldCommand() if (nNumberingType == style::NumberingType::PAGE_DESCRIPTOR) nNumberingType = style::NumberingType::ARABIC; xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny(nNumberingType)); } break; @@ -3994,7 +3974,7 @@ void DomainMapper_Impl::CloseFieldCommand() if (!sFirstParam.isEmpty()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_FIXED ), uno::makeAny( true )); + getPropertyName( PROP_IS_FIXED ), uno::makeAny( true )); //PROP_CURRENT_PRESENTATION is set later anyway } } @@ -4006,7 +3986,7 @@ void DomainMapper_Impl::CloseFieldCommand() if (pContext->IsFieldLocked()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_IS_FIXED), + getPropertyName(PROP_IS_FIXED), uno::makeAny( true )); m_bSetDateValue = true; } @@ -4018,7 +3998,7 @@ void DomainMapper_Impl::CloseFieldCommand() if (!sFirstParam.isEmpty()) { xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName( PROP_IS_FIXED ), uno::makeAny( true )); + getPropertyName( PROP_IS_FIXED ), uno::makeAny( true )); //PROP_CURRENT_PRESENTATION is set later anyway } } @@ -4026,15 +4006,15 @@ void DomainMapper_Impl::CloseFieldCommand() case FIELD_USERADDRESS : //todo: user address collects street, city ... break; case FIELD_INDEX: - handleIndex(pContext, rPropNameSupplier, xFieldInterface, xFieldProperties, + handleIndex(pContext, xFieldInterface, xFieldProperties, OUString::createFromAscii(aIt->second.cFieldServiceName)); break; case FIELD_BIBLIOGRAPHY: - handleBibliography(pContext, rPropNameSupplier, + handleBibliography(pContext, OUString::createFromAscii(aIt->second.cFieldServiceName)); break; case FIELD_TOC: - handleToc(pContext, rPropNameSupplier, xFieldInterface, xFieldProperties, + handleToc(pContext, xFieldInterface, xFieldProperties, OUString::createFromAscii(aIt->second.cFieldServiceName)); break; case FIELD_XE: @@ -4095,7 +4075,7 @@ void DomainMapper_Impl::CloseFieldCommand() uno::UNO_QUERY_THROW); if (!sFirstParam.isEmpty()) { - xTC->setPropertyValue(rPropNameSupplier.GetName(PROP_ALTERNATIVE_TEXT), + xTC->setPropertyValue(getPropertyName(PROP_ALTERNATIVE_TEXT), uno::makeAny(sFirstParam)); } OUString sValue; @@ -4109,7 +4089,7 @@ void DomainMapper_Impl::CloseFieldCommand() { sal_Int32 nLevel = sValue.toInt32(); if( !sValue.isEmpty() && nLevel >= 0 && nLevel <= 10 ) - xTC->setPropertyValue(rPropNameSupplier.GetName(PROP_LEVEL), uno::makeAny( (sal_Int16)nLevel )); + xTC->setPropertyValue(getPropertyName(PROP_LEVEL), uno::makeAny( (sal_Int16)nLevel )); } // if( lcl_FindInCommand( pContext->GetCommand(), 'n', sValue )) // \n Suppress page numbers @@ -4124,7 +4104,7 @@ void DomainMapper_Impl::CloseFieldCommand() case FIELD_NUMPAGES: if (xFieldProperties.is()) xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(PROP_NUMBERING_TYPE), + getPropertyName(PROP_NUMBERING_TYPE), uno::makeAny( lcl_ParseNumberingType(pContext->GetCommand()) )); break; } @@ -4230,7 +4210,6 @@ void DomainMapper_Impl::SetFieldResult(OUString const& rResult) uno::Reference<text::XTextField> xTextField = pContext->GetTextField(); try { - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); OSL_ENSURE( xTextField.is() //||m_xTOC.is() ||m_xTC.is() //||m_sHyperlinkURL.getLength() @@ -4244,7 +4223,7 @@ void DomainMapper_Impl::SetFieldResult(OUString const& rResult) // user field content has to be set at the field master uno::Reference< text::XDependentTextField > xDependentField( xTextField, uno::UNO_QUERY_THROW ); xDependentField->getTextFieldMaster()->setPropertyValue( - rPropNameSupplier.GetName(PROP_CONTENT), + getPropertyName(PROP_CONTENT), uno::makeAny( rResult )); } else if ( m_bSetCitation ) @@ -4317,11 +4296,11 @@ void DomainMapper_Impl::SetFieldResult(OUString const& rResult) if (bIsSetExpression) { // this will throw for field types without Content uno::Any aValue(xFieldProperties->getPropertyValue( - rPropNameSupplier.GetName(PROP_CONTENT))); + getPropertyName(PROP_CONTENT))); aValue >>= sValue; } xFieldProperties->setPropertyValue( - rPropNameSupplier.GetName(bIsSetExpression && sValue.isEmpty()? PROP_CONTENT : PROP_CURRENT_PRESENTATION), + getPropertyName(bIsSetExpression && sValue.isEmpty()? PROP_CONTENT : PROP_CURRENT_PRESENTATION), uno::makeAny( rResult )); } } @@ -4467,11 +4446,10 @@ void DomainMapper_Impl::PopFieldContext() } else if(!pContext->GetHyperlinkURL().isEmpty()) { - PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); xCrsr->gotoEnd( true ); uno::Reference< beans::XPropertySet > xCrsrProperties( xCrsr, uno::UNO_QUERY_THROW ); - xCrsrProperties->setPropertyValue(rPropNameSupplier.GetName(PROP_HYPER_LINK_U_R_L), uno:: + xCrsrProperties->setPropertyValue(getPropertyName(PROP_HYPER_LINK_U_R_L), uno:: makeAny(pContext->GetHyperlinkURL())); if (m_bStartTOC) { @@ -4721,33 +4699,30 @@ void DomainMapper_Impl::SetLineNumbering( sal_Int32 nLnnMod, sal_uInt32 nLnc, sa { if( !m_bLineNumberingSet ) { - const PropertyNameSupplier& rPropNameSupplier = PropertyNameSupplier::GetPropertyNameSupplier(); - try { uno::Reference< text::XLineNumberingProperties > xLineProperties( m_xTextDocument, uno::UNO_QUERY_THROW ); uno::Reference< beans::XPropertySet > xProperties = xLineProperties->getLineNumberingProperties(); uno::Any aTrue( uno::makeAny( true )); - xProperties->setPropertyValue( rPropNameSupplier.GetName( PROP_IS_ON ), aTrue); - xProperties->setPropertyValue( rPropNameSupplier.GetName( PROP_COUNT_EMPTY_LINES ), aTrue ); ... etc. - the rest is truncated
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits