summaryrefslogtreecommitdiff
path: root/qadevOOo
AgeCommit message (Collapse)AuthorFilesLines
2014-10-30java: calling ready() on a BufferedReader is a dodgy way..Noel Grandin1-17/+12
..to detect end-of-file Change-Id: I3b1d5eca3a7424d72b5e3ab741f7407ad7e65193
2014-10-30java: Questionable use of non-short-circuit logicRobert Antoni Buj i Gelonch1-1/+2
Change-Id: Ied831cb5d6645e2696ff2d3d57843fc7a1f82f7e Reviewed-on: https://gerrit.libreoffice.org/12137 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-30runner: Dereference of the result of readLine() without nullcheckRobert Antoni Buj i Gelonch1-9/+13
Change-Id: I3d02436446a79137ce42b1a190c9e3985fbb9246 Reviewed-on: https://gerrit.libreoffice.org/12135 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-25java: prevent overflow by using 'long int' arithmetic in multiplicationRobert Antoni Buj i Gelonch5-5/+5
Change-Id: I8dda8f4621f265208c713c9edcfe725f1c9c5998 Reviewed-on: https://gerrit.libreoffice.org/12001 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-19runner: Iterate over each Entry in a MapRobert Antoni Buj i Gelonch3-18/+21
Change-Id: I48de54ea88e7fd9f2d903c172eb2b6e1a5b73edd Reviewed-on: https://gerrit.libreoffice.org/11918 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-18java: loss of precissionRobert Antoni Buj i Gelonch1-1/+1
Change-Id: I0563a76b1d2c838e6301bf7a9ed154c77a98c0d9 Reviewed-on: https://gerrit.libreoffice.org/12000 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-17java: dodgy &= operationNoel Grandin1-4/+2
Change-Id: Ia909797e08948a8ad0e6f536ce2af57880dc13ef
2014-10-17java: when rethrowing exceptions, store the originalNoel Grandin3-12/+5
Change-Id: I8a2a264597d0b1ae06b08136fea36003682380b5
2014-10-17java: final fields that can be staticNoel Grandin5-7/+7
Change-Id: I8c06be7bc0b8a38c662209f0de72a00550e25447
2014-10-17java: no need to call String.valueOf to append to a StringNoel Grandin13-31/+31
Change-Id: I5ef45b971d24dde814ab8cbb2e9503d9da4f0d73
2014-10-16java: when rethrowing, store the original exceptionNoel Grandin69-387/+237
Change-Id: Idfca83a2a646acab90886c0ef9c30dd7d2fc8b53
2014-10-16runner: if .. else if .. elseRobert Antoni Buj i Gelonch1-1/+1
Change-Id: If9c07074b60b14d0f4e947754d4bf34e4f2f9676 Reviewed-on: https://gerrit.libreoffice.org/11990 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-16java: always use braces for while loopsNoel Grandin8-9/+16
Change-Id: Iff896b0cace8b8305528b3b0140004ea856169ce
2014-10-16java: reduce the depth of some deeply nested if blocksNoel Grandin1-14/+15
Change-Id: I3c0c7f08d4d8ea594e72fc0d9b93d085d4ab4bf5
2014-10-15runner: replace '.size() == 0' with '.isEmpty()' (collections)Robert Antoni Buj i Gelonch1-1/+1
http://docs.oracle.com/javase/1.5.0/docs/api/java/util/ArrayList.html#isEmpty() Change-Id: I9a8f4feab9120d2d9fbf845f305380cda9b263bb Reviewed-on: https://gerrit.libreoffice.org/11968 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-14reformat ValueChangerNoel Grandin1-934/+980
so that it is at least marginally legible Change-Id: I20882ad77d6488bb1f2b9dc9e91dc47458596b4a
2014-10-14runner: Null pointer dereferenceRobert Antoni Buj i Gelonch1-5/+5
Change-Id: Id38a9e0bddbf3c9038abbc5de7137ee5d3f39db6 Reviewed-on: https://gerrit.libreoffice.org/11924 Reviewed-by: Matthew Francis <mjay.francis@gmail.com> Tested-by: Matthew Francis <mjay.francis@gmail.com>
2014-10-14runner: finally block to ensure that a resource is closed (Prior to Java SE 7)Robert Antoni Buj i Gelonch11-728/+873
http://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html Change-Id: I5ecefd3e5bf84fea2a8735a44236ed4c86b440c4 Reviewed-on: https://gerrit.libreoffice.org/11950 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-13java: Collections.addAll for adding an array of elements to a CollectionRobert Antoni Buj i Gelonch2-8/+6
Change-Id: I4689aa73e1e261c6461e04b9e814ffefa5817c09 Reviewed-on: https://gerrit.libreoffice.org/11953 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-13java: import from the same packageRobert Antoni Buj i Gelonch2-3/+0
Change-Id: I1bb0999783f365e20b682c3707e73c65724265c9 Reviewed-on: https://gerrit.libreoffice.org/11955 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-13runner: Object comparisonRobert Antoni Buj i Gelonch2-2/+2
Change-Id: I41b245f20652187dc36b4639272d0ddbc602afda Reviewed-on: https://gerrit.libreoffice.org/11921 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-12runner: code will never be executedRobert Antoni Buj i Gelonch1-4/+0
Change-Id: I0484a11b3511c1d7f70e9ae8407eb7d19bcc852d Reviewed-on: https://gerrit.libreoffice.org/11929 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-12runner: Concatenating null strings in Java (JDK 1.5+)Robert Antoni Buj i Gelonch1-23/+4
JLS, Section 15.18.1.1: If the reference is null, it is converted to the string "null". http://docs.oracle.com/javase/specs/jls/se5.0/html/expressions.html#15.18.1.1 Change-Id: Idae09c02b24da38ec6f3475fd0af35ef48a2405b Reviewed-on: https://gerrit.libreoffice.org/11930 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-12runner: remove import from same packageRobert Antoni Buj i Gelonch5-9/+5
Change-Id: I4aeb7f58034f8808325639d07aa2dc55959d97a8 Reviewed-on: https://gerrit.libreoffice.org/11932 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-12runner: remove import from java.lang PackageRobert Antoni Buj i Gelonch1-1/+0
Change-Id: Iedad24dfbdd60a05a09888f699f2467960f5cbe9 Reviewed-on: https://gerrit.libreoffice.org/11933 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-12runner: 'static final' to declare constants when it isn't an interfaceRobert Antoni Buj i Gelonch10-19/+19
Change-Id: Iea1e672e5468b66fd1ba5389bd781f03d0c7f419 Reviewed-on: https://gerrit.libreoffice.org/11931 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-11It is more efficient to use the integer implementations of String.indexOf()Robert Antoni Buj i Gelonch1-2/+2
Change-Id: Idb1e4945088b8443e9b507826e5289ee758a2bd2 Reviewed-on: https://gerrit.libreoffice.org/11919 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-10runner: The if statement is redundantRobert Antoni Buj i Gelonch12-167/+48
Change-Id: Ida40034bdfe6a44a936db1243ad6c71616caada3 Reviewed-on: https://gerrit.libreoffice.org/11895 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-09typo: proeprty -> propertyAndras Timar3-3/+3
Change-Id: Ib20ca28c2236fc99e1339b6e57e8641a213116c7
2014-10-09typo: Lable -> LabelAndras Timar1-2/+2
Change-Id: I8b9e9f75dd22ee5bff678c5bc0e1fa9381a103de
2014-10-07java: simplify some for loops to while loopsNoel Grandin3-4/+4
Change-Id: I323ab12c634d3baa4f624b63d7d483112c23192c
2014-10-07while loop that ends in a return is more accurately an if blockNoel Grandin1-1/+1
Change-Id: I94b74b46b185e1061b6b72acdf3d9dc0c1d122c8
2014-10-07java: use isEmpty() instead of "size() == 0"Noel Grandin3-4/+4
Change-Id: I23e1038246999b0744d8e9ae83b66fa1f7dafa99
2014-10-07java: use equalsIgnoreCase() instead of toLowerCase().equals()Noel Grandin11-51/+51
Change-Id: I8f4afc7e15a9d85d99ea80edbbbd2b507b254ddc
2014-10-07java: use equals() to compare Strings, not ==Noel Grandin3-3/+3
Change-Id: Ia41a4899fb82df7b580a020765404c4ef0cddfa2
2014-10-07java: simplify conditions involving logical negationNoel Grandin9-16/+16
Change-Id: Ib45e58273e650ef9a31dbdb5f71e995a5976f50e
2014-10-07java: remove some unnecessary intermediary object creationNoel Grandin3-8/+8
Change-Id: Id4949fa08546e710fbf9bd0c7e3bf62979f29c83
2014-10-07java: optimise calls to toArrayNoel Grandin1-2/+2
passing in a correctly sized array requires one less allocation Change-Id: I6198d226d9b630c99c4e8b6e3d0fdf890c495b44
2014-10-07java: remove unnecessary adding of empty stringsNoel Grandin6-10/+10
Change-Id: I3825ea5fb1eb34ca52659350e202f95abea945a9
2014-10-07java: no need to check for null before calling instanceofNoel Grandin3-3/+3
the instanceof check returns false when passed a null value Change-Id: I7742d0d9cf25ef23d9adee7328f701c7efeea8b5
2014-10-07java: remove useless overriding methodsNoel Grandin1-19/+0
that only call their superclass method Change-Id: I9be2f69be132eec7918964f504c50df0ae8401a2
2014-10-06fdo#73151 Always open Styles&Formatting dialog in the sidebarSamuel Mehrbrodt2-11/+11
Change-Id: I05b63dc00023945d92fd837bd64042135d3f9aed Reviewed-on: https://gerrit.libreoffice.org/11648 Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2014-10-02runner: Converting ArrayList to ArrayRobert Antoni Buj i Gelonch3-18/+4
Change-Id: Ifd5fe63cb00dd5b3987035201554731075d3bc00 Reviewed-on: https://gerrit.libreoffice.org/11755 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-01runner: Avoid a possible NullPointerException with tCase ObjectRobert Antoni Buj i Gelonch1-21/+14
Change-Id: Ie1202b7fa89306ea2a99031b9347b657039b839e Reviewed-on: https://gerrit.libreoffice.org/11750 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-09-25java: when rethrowing exceptions, store the original causeNoel Grandin7-7/+15
so that we get a nice complete stacktrace when it hits the final handler Change-Id: Iec4fcc15a2a25c55f591b5e069dce3d010197a90
2014-09-24fdo#39625 Make existing CppUnittests workTobias Madl1-1/+0
Just removed some useless comment. Change-Id: I3f43e245849b82f8a1257c6ac0bace673d0ca9d6 Reviewed-on: https://gerrit.libreoffice.org/11604 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-11qadevOOo: use String.length()==0 instead of String.equals(empty string)rbuj4-5/+5
Change-Id: I5cdd1fb4fe77c7fc9614f2a6de8995d98d60b551 Reviewed-on: https://gerrit.libreoffice.org/11368 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-10Typo: (N|n)ormaly->(N|n)ormallyJulien Nabet1-1/+1
Change-Id: I96d081f394b0b62d99ec1034bf5e99da9aedd9d9
2014-09-10qadevOOo: Convert a primitive type into a stringrbuj1-2/+2
Change-Id: I1a840346e75a3d7dd0de761660a0ea95c7ee034b Reviewed-on: https://gerrit.libreoffice.org/11370 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-10qadevOOo: Number Parsingrbuj3-6/+6
Change-Id: I284a1b2bcb61695b74ec51bb46cbb683abf83449 Reviewed-on: https://gerrit.libreoffice.org/11369 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>