summaryrefslogtreecommitdiff
path: root/qadevOOo/runner
AgeCommit message (Collapse)Author
2014-11-18java: make fields final where possibleNoel Grandin
found by PMD Change-Id: I87780366119c141cd2dafe6ca1bf2d9798b10aec
2014-11-18java: rather use @Deprecated annotationNoel Grandin
than an empty @deprecated tag Change-Id: I1646ce1c0c8823ac5be0153aeb8eb2b830e7ebfa Reviewed-on: https://gerrit.libreoffice.org/12525 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-12java: convert fields to local variables where possibleNoel Grandin
found by PMD Change-Id: I05b45382b8fb1b734657ce9421a20e6ef6fbe542 Reviewed-on: https://gerrit.libreoffice.org/12376 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-30java: calling ready() on a BufferedReader is a dodgy way..Noel Grandin
..to detect end-of-file Change-Id: I3b1d5eca3a7424d72b5e3ab741f7407ad7e65193
2014-10-30java: Questionable use of non-short-circuit logicRobert Antoni Buj i Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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: when rethrowing exceptions, store the originalNoel Grandin
Change-Id: I8a2a264597d0b1ae06b08136fea36003682380b5
2014-10-17java: no need to call String.valueOf to append to a StringNoel Grandin
Change-Id: I5ef45b971d24dde814ab8cbb2e9503d9da4f0d73
2014-10-16java: when rethrowing, store the original exceptionNoel Grandin
Change-Id: Idfca83a2a646acab90886c0ef9c30dd7d2fc8b53
2014-10-16runner: if .. else if .. elseRobert Antoni Buj i Gelonch
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 Grandin
Change-Id: Iff896b0cace8b8305528b3b0140004ea856169ce
2014-10-16java: reduce the depth of some deeply nested if blocksNoel Grandin
Change-Id: I3c0c7f08d4d8ea594e72fc0d9b93d085d4ab4bf5
2014-10-15runner: replace '.size() == 0' with '.isEmpty()' (collections)Robert Antoni Buj i Gelonch
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 Grandin
so that it is at least marginally legible Change-Id: I20882ad77d6488bb1f2b9dc9e91dc47458596b4a
2014-10-14runner: Null pointer dereferenceRobert Antoni Buj i Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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 Gelonch
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-07java: simplify some for loops to while loopsNoel Grandin
Change-Id: I323ab12c634d3baa4f624b63d7d483112c23192c
2014-10-07while loop that ends in a return is more accurately an if blockNoel Grandin
Change-Id: I94b74b46b185e1061b6b72acdf3d9dc0c1d122c8
2014-10-07java: use isEmpty() instead of "size() == 0"Noel Grandin
Change-Id: I23e1038246999b0744d8e9ae83b66fa1f7dafa99
2014-10-07java: use equalsIgnoreCase() instead of toLowerCase().equals()Noel Grandin
Change-Id: I8f4afc7e15a9d85d99ea80edbbbd2b507b254ddc
2014-10-07java: simplify conditions involving logical negationNoel Grandin
Change-Id: Ib45e58273e650ef9a31dbdb5f71e995a5976f50e
2014-10-07java: remove unnecessary adding of empty stringsNoel Grandin
Change-Id: I3825ea5fb1eb34ca52659350e202f95abea945a9
2014-10-07java: no need to check for null before calling instanceofNoel Grandin
the instanceof check returns false when passed a null value Change-Id: I7742d0d9cf25ef23d9adee7328f701c7efeea8b5
2014-10-06fdo#73151 Always open Styles&Formatting dialog in the sidebarSamuel Mehrbrodt
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 Gelonch
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 Gelonch
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 Grandin
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 Madl
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)rbuj
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-10qadevOOo: Convert a primitive type into a stringrbuj
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 Parsingrbuj
Change-Id: I284a1b2bcb61695b74ec51bb46cbb683abf83449 Reviewed-on: https://gerrit.libreoffice.org/11369 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-10qadevOOo: use a character literalrbuj
Change-Id: I6d3e592de64c63e66bb788b112e188ef217e7bf7 Reviewed-on: https://gerrit.libreoffice.org/11367 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-10qadevOOo: use System.arraycopyrbuj
Change-Id: Ibbac337b481e138ff9bfaa9d350b52e368ba5f2b Reviewed-on: https://gerrit.libreoffice.org/11355 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-08qadevOOo: reuse the value of entryName.indexOf(Left Parenthesis)rbuj
Change-Id: I1950275af2a063db74fef326ac0dd4e555e5170d Reviewed-on: https://gerrit.libreoffice.org/11323 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-08qadevOOo: remove import from the same packagerbuj
Change-Id: Id12bece6f1bba0591c3b311553df3fecec40e524 Reviewed-on: https://gerrit.libreoffice.org/11324 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-08qadevOOo: All classes in the java.lang package are imported by defaultrbuj
Change-Id: I10f87af6e8128354bfca3587dbd390e5d41f5ce7 Reviewed-on: https://gerrit.libreoffice.org/11325 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
generated by cgit (git 2.34.1) at 2025-01-30 21:04:50 +0000