[PATCH] eu

2013-03-09 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2620 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/20/2620/1 eu Change-Id: Ib60a6dd29a89c3eb91520f9a6ced663a20596aac --- M connectivity/source/commontools/AutoRetrievingBase

[PATCH] use startsWith() instead of compareToAscii()

2013-03-09 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2621 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/2621/1 use startsWith() instead of compareToAscii() brain damage... Change-Id: I4dc63c7346f724eded9ac7b82cda25c2bb60be

[ABANDONED] eu

2013-03-09 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: eu .. Patch Set 1: Abandoned -- To view, visit https://gerrit.libreoffice.org/2620 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: ab

[PUSHED] Comment translation DE-EN in /sw/source/ui/web/

2013-03-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2619 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2619 To unsubscribe, vis

[PUSHED] Translate german comments in sw/source/filter/ww8/

2013-03-10 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2617 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2617 To unsubscribe, vis

[PUSHED] Comment translation german-english in sw/source/ui/table/

2013-03-10 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2625 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2625 To unsubscribe, vis

[PUSHED] tempfile: Creating tempfiles in a given folder is not used

2013-03-12 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2558 -- To view, visit https://gerrit.libreoffice.org/2558 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: mer

[PUSHED] Translate German comments, clean up some whitespace

2013-03-13 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2539 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2539 To unsubscribe, vis

[PUSHED] fdo#38838, String to OUString, in dbaccess/source/ui/browser

2013-03-14 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2538 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2538 To unsubscribe, vis

[PUSHED] fdo#38838, String to OUString in dbaccess

2013-03-14 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2386 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2386 To unsubscribe, vis

[PATCH] automated removal of RTL_CONSTASCII_USTRINGPARAM for quoted ...

2013-03-19 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2832 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/32/2832/1 automated removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStrings Done with a perl regex: s/OUString\s*\(\s

[PUSHED] automated removal of RTL_CONSTASCII_USTRINGPARAM for quoted ...

2013-03-19 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2832 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2832 To unsubscribe, vis

[PATCH] removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStrings ...

2013-03-19 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2835 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/35/2835/1 removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStrings declarations s/(OUString\s+[a-zA-Z_][A-Za-z0-9_]*\s*

[PUSHED] removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStrings ...

2013-03-19 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2835 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2835 To unsubscribe, vis

[PATCH] this looks better

2013-03-20 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2874 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/74/2874/1 this looks better Change-Id: I10af9ff8513f8f583867d0d6a18af03b3b5b1a28 --- M cppuhelper/inc/cppuhelper/propertys

[PATCH] basic: Remove non working AboutStarBasic

2013-03-21 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2899 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/99/2899/1 basic: Remove non working AboutStarBasic The content of this function was commented out since the first import.

[PATCH] clean up DBG_ERRORn

2013-03-21 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2900 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/00/2900/1 clean up DBG_ERRORn Those were DBG_ERRORn before and were falsely changed to OSL_TRACE by me. See: 1707a350af8a

[PUSHED] Comment translation GER-ENG in sw/source/ui/wrtsh/

2013-03-21 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2902 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2902 To unsubscribe, vis

[PUSHED] Comment translation - GER-ENG in sw/source/ui/wrtsh/wrtsh1.c...

2013-03-21 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2883 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2883 To unsubscribe, vis

[PUSHED] General comment cleanup for demo/test vcl apps

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2921 Approvals: Tor Lillqvist: Verified; Looks good to me, approved Thomas Arnhold: -- To view, visit https://gerrit.libreoffice.org/2921 T

[PUSHED] Improve size and positions of widgets for CustomAnimationDia...

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2932 -- To view, visit https://gerrit.libreoffice.org/2932 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: mer

[PUSHED] fdo#39468 Translate German comments - vcl/win/source/app

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2938 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2938 To unsubscribe, vis

[PATCH] wip

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2944 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/44/2944/1 wip Change-Id: I23427e1e3bfb583ef10123577ebec76af617f917 --- M vcl/inc/vcl/bitmap.hxx M vcl/source/gdi/bitmap.cx

[PATCH] fdo#62525: use cow_wrapper for SdrShadowAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2945 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/2945/1 fdo#62525: use cow_wrapper for SdrShadowAttribute Thanks to Thorsten. Change-Id: I2713bacd10538dfe177cb975a9633

[ABANDONED] wip

2013-03-23 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: wip .. Patch Set 1: Abandoned ... -- To view, visit https://gerrit.libreoffice.org/2944 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageTy

[PATCH] fdo#62525: use cow_wrapper for SdrShadowAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2946 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/46/2946/1 fdo#62525: use cow_wrapper for SdrShadowAttribute Thanks to Thorsten. Change-Id: I9b5435d2326e9ebf340e88025eeea

[ABANDONED] fdo#62525: use cow_wrapper for SdrShadowAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: fdo#62525: use cow_wrapper for SdrShadowAttribute .. Patch Set 1: Abandoned bla -- To view, visit https://gerrit.libreoffice.org/2945 To unsubscribe, visit https://ger

[PATCH] fdo#62525: use cow_wrapper for FillBitmapAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2948 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/48/2948/1 fdo#62525: use cow_wrapper for FillBitmapAttribute Change-Id: I0f666d4baaf6dc4e9f7ea30a518a8904f01d749f --- M dr

[PATCH] fdo#62525: use cow_wrapper for FillGradientAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2951 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/51/2951/1 fdo#62525: use cow_wrapper for FillGradientAttribute Change-Id: I85a9864820f49da8ebcc4a7d2c80a3c9d0c13b27 --- M

[PUSHED] Change duplicate short-cut Alt-I-U

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2943 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2943 To unsubscribe, vis

[PATCH] fdo#62525: use cow_wrapper for FillGradientAttribute

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2954 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/54/2954/1 fdo#62525: use cow_wrapper for FillGradientAttribute a xmlsecurity/workben/signaturetest.cxx.wo Change-Id

[PUSHED] fdo#39468 Translate German Comments - salframe.cxx

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2931 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2931 To unsubscribe, vis

[PUSHED] fdo#39468 Translate German comments - vcl/win/source/gdi/

2013-03-23 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2937 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2937 To unsubscribe, vis

[ABANDONED] [draft] use pragma once

2013-03-24 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: [draft] use pragma once .. Patch Set 3: Abandoned Not as easy as it seems. Clashes with Windows headers at least. Why ever... -- To view, visit https://gerrit.libreof

[PUSHED] Remarktranslations german - english

2013-02-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2058 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2058 To unsubscribe, vis

[PUSHED] Remark translations german - english

2013-02-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2059 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2059 To unsubscribe, vis

[PUSHED] Remark Translation german - english

2013-02-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2060 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2060 To unsubscribe, vis

[PUSHED] translate German comments, remove obsolete code

2013-02-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2054 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2054 To unsubscribe, vis

[PUSHED] translate German comments, remove obsolete code

2013-02-09 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2049 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2049 To unsubscribe, vis

[PUSHED] Comment translation german - english

2013-02-11 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2074 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2074 To unsubscribe, vis

[PUSHED] Comment translation german - english - correction

2013-02-11 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2083 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2083 To unsubscribe, vis

[PUSHED] Comment translation german - english

2013-02-11 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2084 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2084 To unsubscribe, vis

[PUSHED] Comment translation german - english

2013-02-11 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2085 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2085 To unsubscribe, vis

[PUSHED] sc_subsequent_filters-test: add bug number

2013-02-11 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2079 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2079 To unsubscribe, vis

[PUSHED] Translate German comments, fix some whitespace

2013-03-25 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2916 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2916 To unsubscribe, vis

[PATCH] Fix spelling errors and typos: 'unkown' change to 'unknown'

2013-03-25 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3030 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/30/3030/1 Fix spelling errors and typos: 'unkown' change to 'unknown' Change-Id: Ia3ff77d2b99c9ef6c4c9b510bb30f349592c0b94

[PUSHED] Cleanup comments and some translations in style.cxx

2013-03-26 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3062 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3062 To unsubscribe, vis

[PUSHED] Translate German comments, fix some ws

2013-03-26 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3063 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3063 To unsubscribe, vis

[PUSHED] fdo#39468 Translate German comments - vcl/source/control/

2013-03-26 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2966 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2966 To unsubscribe, vis

[PATCH] fdo#62525: use cow_wrapper for FontAttribute

2013-03-26 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3074 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/74/3074/1 fdo#62525: use cow_wrapper for FontAttribute Change-Id: Ic07da7c7cf225a910e6f0fa4f6d20c4700e7ec7a --- M drawingl

[PATCH] fdo#62525: use cow_wrapper for FillHatchAttribute

2013-03-26 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3073 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/73/3073/1 fdo#62525: use cow_wrapper for FillHatchAttribute Change-Id: I1581b4bca6f14bd27af12ff40a4f4c0fe08af133 --- M dra

[PUSHED] String::AppendAscii cleanup in dbaccess

2013-03-28 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3068 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3068 To unsubscribe, vis

[PATCH] OSL_FAIL -> SAL_WARN

2013-03-29 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3121 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/3121/1 OSL_FAIL -> SAL_WARN Change-Id: I1871e5e3a319b2d5177ae4bd470046c022c14eb7 --- M connectivity/source/commontools/

[PUSHED] Comment translation GER-ENG in sw/source/ui/utlui/ (Part1)

2013-03-29 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3101 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3101 To unsubscribe, vis

[PATCH] fdo#58807 bring Beep() for basic back

2013-03-30 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3124 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/3124/1 fdo#58807 bring Beep() for basic back The functionality was removed by fdo#48549. This partially reverts 0f610

[PATCH] drawinglayer depends on o3tl_inc

2013-03-30 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3125 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/25/3125/1 drawinglayer depends on o3tl_inc Change-Id: I9dab86e62eb34819be1564b3d7be50cf6acb6f73 --- M drawinglayer/Library

[ABANDONED] drawinglayer depends on o3tl_inc

2013-03-30 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: drawinglayer depends on o3tl_inc .. Patch Set 1: Abandoned Not needed. Thanks David :) -- To view, visit https://gerrit.libreoffice.org/3125 To unsubscribe, visit http

[PUSHED] Multiple typo fixes, small code changes

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3135 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3135 To unsubscribe, vis

[PUSHED] fdo#62096 Replace O(U)String compareTo with ==

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3136 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3136 To unsubscribe, vis

[PUSHED] Remove commented code in component.cxx (odk)

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3137 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3137 To unsubscribe, vis

[PUSHED] Comment translation GER-ENG in sw/source/ui/utlui/ PART2

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3129 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3129 To unsubscribe, vis

[PUSHED] Remove commented code in odk/examples

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3138 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3138 To unsubscribe, vis

[PATCH] vcl: convert some fprintf to SAL_INFO

2013-03-31 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3143 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/43/3143/1 vcl: convert some fprintf to SAL_INFO Change-Id: I4b9a20e62e8e137a1bfaa1c654307f15c33e3d3c --- M sal/inc/sal/log

[PUSHED] Comment translation GER-ENG in sw/source/ui/utlui/ (Part 3)

2013-04-01 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3144 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3144 To unsubscribe, vis

[PATCH] API CHANGE: drop rtl::O[U]StringBuffer::{charAt,setCharAt}

2013-04-01 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3155 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/55/3155/1 API CHANGE: drop rtl::O[U]StringBuffer::{charAt,setCharAt} As a follow up of fdo#41474. See: https://wiki.docum

[PATCH] Revert "Revert "reduce some uglyness""

2013-04-02 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3171 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/71/3171/1 Revert "Revert "reduce some uglyness"" The header guard seems to solve the problem. This reverts commit df8c41c

[PUSHED] vcl: convert some fprintf to SAL_INFO

2013-04-02 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3143 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3143 To unsubscribe, vis

[ABANDONED] API CHANGE: drop rtl::O[U]StringBuffer::{charAt, setCharAt}

2013-04-02 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: API CHANGE: drop rtl::O[U]StringBuffer::{charAt,setCharAt} .. Patch Set 1: Abandoned -- To view, visit https://gerrit.libreoffice.org/3155 To unsubscribe, visit https:/

[PATCH] Remove RTL_CONSTASCII_(U)STRINGPARAM in bridges

2013-04-02 Thread Thomas Arnhold (via Code Review)
Hello LibreOffice gerrit bot, Luboš Luňák, I'd like you to reexamine a rebased change. Please visit https://gerrit.libreoffice.org/3139 to look at the new rebased patch set (#2). Change subject: Remove RTL_CONSTASCII_(U)STRINGPARAM in bridges ...

[PATCH] Remove SvtInternalOptions

2013-04-02 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3180 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/80/3180/1 Remove SvtInternalOptions * only Get/SetCurrentTempURL were used, but only for compatibility reasons, to remove

[PUSHED] fdo#39468: Translated German comments in filter

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3178 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3178 To unsubscribe, vis

[PUSHED] fdo#39468 Translate German in DBG_ASSERT and OSL_ENSURE in t...

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3163 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3163 To unsubscribe, vis

[PATCH] fdo#62525: use cow_wrapper for ViewInformation2D

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3183 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/83/3183/1 fdo#62525: use cow_wrapper for ViewInformation2D Change-Id: Ide3bce3257c9fed7bbda7276dfb55fa179d74fdd --- M draw

[PATCH] fdo#58807 bring Beep() for basic back

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3184 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/84/3184/1 fdo#58807 bring Beep() for basic back The functionality was removed by fdo#48549. This partially reverts 0f610

[ABANDONED] fdo#58807 bring Beep() for basic back

2013-04-03 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: fdo#58807 bring Beep() for basic back .. Patch Set 1: Abandoned -- To view, visit https://gerrit.libreoffice.org/3184 To unsubscribe, visit https://gerrit.libreoffice.o

[PATCH libreoffice-4-0] fdo#58807 bring Beep() for basic back

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3185 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/85/3185/1 fdo#58807 bring Beep() for basic back The functionality was removed by fdo#48549. This partially reverts 0f610

[PUSHED libreoffice-4-0] Keep semantics, but improve readibility.

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3186 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3186 To unsubscribe, vis

[PATCH] unotools: remove some unused items

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3191 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/91/3191/1 unotools: remove some unused items Change-Id: I95bcac6a3ff8cf4644c9d1596c75cac2d9cd94fe --- M officecfg/registry

[PUSHED] fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT

2013-04-03 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3179 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3179 To unsubscribe, vis

[PUSHED] Changed compareTo to startsWith

2013-05-19 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3956 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3956 To unsubscribe, vis

[PUSHED] fdo#39468 Translate German comments - graphicfilter

2013-05-19 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3955 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3955 To unsubscribe, vis

[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/ribbar...

2013-05-19 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3924 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3924 To unsubscribe, vis

[PUSHED] fdo#39468 Comment translation GER-ENG

2013-05-20 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3965 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3965 To unsubscribe, vis

[PATCH] be case-insensitive for open/starsymbol comparison

2013-05-23 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/4015 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/15/4015/1 be case-insensitive for open/starsymbol comparison Change-Id: I7a342c809a723f5f9c03271c6145d9c367cea6de --- M ed

[PUSHED] fix OUString conversion

2013-05-25 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/4035 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/4035 To unsubscribe, vis

[PATCH] fdo#63351 make use of ZoomSlider in StarMath

2013-05-27 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/4053 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/53/4053/1 fdo#63351 make use of ZoomSlider in StarMath In collaboration with Marcos Souza :) Includes some reordering of

[PATCH] Make use of IsStarSymbol()

2013-05-28 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/4077 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/77/4077/1 Make use of IsStarSymbol() Change-Id: I7a2e3a6f11bbaaaff50e8163e4ce3e2583b1d03c --- M editeng/source/items/texti

[PATCH] fdo#60724 change spelling error REMOVEABLE -> REMOVABLE II

2013-05-28 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/4078 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/78/4078/1 fdo#60724 change spelling error REMOVEABLE -> REMOVABLE II Change-Id: I0057b9174af6a83f3fde0e27c3a91a4aeca95873

[PATCH] fdo#39468 - translate German comments

2013-05-28 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/4079 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/79/4079/1 fdo#39468 - translate German comments Change-Id: I1324c2682873112fa7702b94fcaceab6de0b94d1 --- M rsc/inc/rscrsc.

[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/uno

2013-06-02 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/4129 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/4129 To unsubscribe, vis

[PUSHED] Replace invalid doxygen commands: thrown -> throws

2013-06-03 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/4136 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/4136 To unsubscribe, vis

[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/wrtsh/...

2013-04-10 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3319 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3319 To unsubscribe, vis

[PATCH] fdo#63350 math: move stardard toolbar to the left

2013-04-10 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3323 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/23/3323/1 fdo#63350 math: move stardard toolbar to the left Change-Id: I8f57d46737315caa41f7488e82262067a35f5253 --- M off

[ABANDONED] fdo#63350 math: move stardard toolbar to the left

2013-04-10 Thread Thomas Arnhold (via Code Review)
Thomas Arnhold has abandoned this change. Change subject: fdo#63350 math: move stardard toolbar to the left .. Patch Set 1: Abandoned hrm this should be libreoffice-4-0 -- To view, visit https://gerrit.libreoffice.org/3323 To

[PATCH libreoffice-4-0] fdo#63350 math: move stardard toolbar to the left

2013-04-10 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3324 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/3324/1 fdo#63350 math: move stardard toolbar to the left Change-Id: Ibadd1dff90383fb0a285a9cb94cc0050906e3f24 --- M off

[PUSHED] remove dead code

2013-04-12 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3361 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3361 To unsubscribe, vis

[PUSHED] remove dead class SfxResourceManager

2013-04-15 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3393 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3393 To unsubscribe, vis

[PUSHED] Removal of Dead code

2013-04-15 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3392 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3392 To unsubscribe, vis

[PUSHED] Simple string cleanup in backingwindow.cxx

2013-04-15 Thread Thomas Arnhold (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3377 Approvals: Thomas Arnhold: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3377 To unsubscribe, vis

[PATCH] scp: remove unused macros

2013-04-16 Thread Thomas Arnhold (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3410 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/10/3410/1 scp: remove unused macros Change-Id: I48ac97bd6ac8cbd534e50c617f3d54ff10bd2908 --- M scp2/macros/macro.pl M scp2

  1   2   >