[PUSHED] add copy() to OUStringBuffer

2013-03-07 Thread Muthu Subramanian K (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/2125 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2125 To unsubscribe

[PUSHED] res. bug on numbered paragraph when .doc import

2013-03-05 Thread Muthu Subramanian K (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/2046 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved Tor Lillqvist: Looks good to me, but someone else must approve Libr

[PUSHED] fdo#38838, replacement of String with OUString

2013-03-05 Thread Muthu Subramanian K (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/2554 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2554 To unsubscribe

[PUSHED libreoffice-4-0-1] Resolves: fdo#61241 force area page to size itself for max v...

2013-03-04 Thread Muthu Subramanian K (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/2451 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved Miklos Vajna: Looks good to me, but someone else must approve Eike

[PUSHED libreoffice-4-0-1] Resolves: fdo#61571 cairo not linking against internal libpn...

2013-03-04 Thread Muthu Subramanian K (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/2494 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved Norbert Thiebaud: Looks good to me, but someone else must approve E

[PUSHED] Delete double condition check

2013-03-04 Thread Muthu Subramanian K (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/2453 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2453 To unsubscribe

[PUSHED] fdo#39468 translate German comments in vcl

2013-03-04 Thread Muthu Subramanian K (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/2534 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2534 To unsubscribe

[PUSHED] OUString cleanup in oox

2012-12-26 Thread Muthu Subramanian K (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/1485 Approvals: Muthu Subramanian K: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/1485 To unsubscribe,

[PUSHED] Review corrections in basctl/.../basobj3.cxx

2012-10-13 Thread Muthu Subramanian K (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/851 -- To view, visit https://gerrit.libreoffice.org/851 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] mork driver: add list tables and unit test

2012-10-10 Thread Muthu Subramanian K (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/841 -- To view, visit https://gerrit.libreoffice.org/841 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] ::rtl::OUString to OUString in basctl

2012-10-09 Thread Muthu Subramanian K (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/835 -- To view, visit https://gerrit.libreoffice.org/835 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] Some OUString and RTL_CONST refactoring

2012-10-09 Thread Muthu Subramanian K (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/836 -- To view, visit https://gerrit.libreoffice.org/836 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] More ::rtl::OUString to OUString in basctl

2012-10-09 Thread Muthu Subramanian K (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/834 -- To view, visit https://gerrit.libreoffice.org/834 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] rtl::OUString to OUString in basctl

2012-10-09 Thread Muthu Subramanian K (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/767 -- To view, visit https://gerrit.libreoffice.org/767 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] OUString cleanup in Chart2

2012-10-08 Thread Muthu Subramanian K (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/828 -- To view, visit https://gerrit.libreoffice.org/828 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] OUString cleanup in Chart2

2012-10-08 Thread Muthu Subramanian K (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/830 -- To view, visit https://gerrit.libreoffice.org/830 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] OUString cleanup in Chart2

2012-10-08 Thread Muthu Subramanian K (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/829 -- To view, visit https://gerrit.libreoffice.org/829 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] OUString cleanup in Chart2

2012-10-08 Thread Muthu Subramanian K (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/827 -- To view, visit https://gerrit.libreoffice.org/827 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] Fix fdo#47669: also check if we started the tag before endin...

2012-10-08 Thread Muthu Subramanian K (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/831 -- To view, visit https://gerrit.libreoffice.org/831 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] cppcheck: unreadVariable

2012-10-08 Thread Muthu Subramanian K (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/832 -- To view, visit https://gerrit.libreoffice.org/832 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me

[PUSHED] cppcheck: clarifyCalculation

2012-10-08 Thread Muthu Subramanian K (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/833 -- To view, visit https://gerrit.libreoffice.org/833 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: me