]> source.dussan.org Git - poi.git/commitdiff
apply some spelling fixes
authorPJ Fanning <fanningpj@apache.org>
Tue, 27 Jun 2023 11:48:38 +0000 (11:48 +0000)
committerPJ Fanning <fanningpj@apache.org>
Tue, 27 Jun 2023 11:48:38 +0000 (11:48 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1910627 13f79535-47bb-0310-9956-ffa450edef68

31 files changed:
build.gradle
build.xml
jenkins/create_jobs.groovy
poi-examples/src/main/java/org/apache/poi/examples/ss/ConditionalFormats.java
poi-examples/src/main/java/org/apache/poi/examples/ss/ToCSV.java
poi-excelant/src/poi-ant-contrib/java/Junit5Progress.java
poi-ooxml/src/main/java/org/apache/poi/ooxml/POIXMLTypeLoader.java
poi-ooxml/src/main/java/org/apache/poi/ooxml/util/IdentifierManager.java
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/OPCPackage.java
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagePartName.java
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/PackagingURIHelper.java
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/internal/ContentTypeManager.java
poi-ooxml/src/main/java/org/apache/poi/poifs/crypt/dsig/SignatureConfig.java
poi-ooxml/src/main/java/org/apache/poi/poifs/crypt/dsig/facets/XAdESXLSignatureFacet.java
poi-ooxml/src/main/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java
poi-ooxml/src/main/java/org/apache/poi/xdgf/usermodel/XDGFShape.java
poi-ooxml/src/main/java/org/apache/poi/xslf/util/PPTX2PNG.java
poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/AbstractWordUtils.java
poi-scratchpad/src/main/java/org/apache/poi/hwpf/usermodel/HWPFList.java
poi/src/main/java/org/apache/poi/hssf/model/InternalWorkbook.java
poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFDataFormat.java
poi/src/main/java/org/apache/poi/poifs/crypt/CryptoFunctions.java
poi/src/main/java/org/apache/poi/sl/draw/DrawNothing.java
poi/src/main/java/org/apache/poi/sl/draw/DrawShape.java
poi/src/main/java/org/apache/poi/sl/draw/Drawable.java
poi/src/main/java/org/apache/poi/ss/formula/functions/MatrixFunction.java
poi/src/test/java/org/apache/poi/hssf/record/aggregates/TestFormulaRecordAggregate.java
poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java
poi/src/test/java/org/apache/poi/ss/usermodel/BaseTestRow.java
poi/src/test/java/org/apache/poi/ss/usermodel/BaseTestSheetAutosizeColumn.java

index 864f776af22b0b1168405cb7cfff9487ffdc0ca2..2c086de80741328b95c74efd5091dbe743980d7a 100644 (file)
@@ -405,7 +405,7 @@ subprojects {
         // https://docs.sonarqube.org/display/SONARQUBE52/Analyzing+with+SonarQube+Scanner+for+Gradle
         // for documentation of properties.
         //
-        // Some additional properties are currently set in the Jenkins-DSL, see jenksin/create_jobs.groovy
+        // Some additional properties are currently set in the Jenkins-DSL, see jenkins/create_jobs.groovy
         //
         sonarqube {
             properties {
index 85f43320fe8cb9e16fba6e1b80257734120bc913..55a97e86f3404cd8d5007449d86a64e0cb8e6741 100644 (file)
--- a/build.xml
+++ b/build.xml
@@ -59,7 +59,7 @@ under the License.
     <property name="util.lib" location="${basedir}/lib/util"/>
 
 
-    <!-- compiler options options -->
+    <!-- compiler options -->
     <property name="jdk.version.source" value="1.8" description="JDK version of source code"/>
     <property name="jdk.version.class" value="1.8" description="JDK version of generated class files"/>
     <property name="compile.debug" value="true"/>
@@ -482,7 +482,7 @@ under the License.
         <pathelement location="${main.output.dir}"/>
         <pathelement location="${ooxml.test.guava.jar}"/>
         <pathelement location="${ooxml.test.opczip.jar}"/>
-        <!-- classes are omitted on test cases outside the xml-dsign area to avoid classpath poisioning -->
+        <!-- classes are omitted on test cases outside the xml-dsign area to avoid classpath poisoning -->
         <!--path refid="ooxml.xmlsec.classpath"/-->
         <!-- Used only for ExtractorFactory, see #57963 -->
         <pathelement location="${scratchpad.output.dir}" unless:true="${scratchpad.ignore}"/>
@@ -2518,7 +2518,7 @@ under the License.
         <fail><condition><matches pattern="[1-9][0-9]* Unknown Licens" string="${rat.reportcontent}"/></condition></fail>
     </target>
 
-    <!-- Runs the Forbiddens APIs checker against the source code, to -->
+    <!-- Runs the Forbidden APIs checker against the source code, to -->
     <!--  spot any cases where we've accidently used methods we shouldn't -->
     <!-- See https://github.com/policeman-tools/forbidden-apis for details -->
     <!--  of the checks that this can do -->
@@ -2761,7 +2761,7 @@ under the License.
 
 
     <target name="test-scratchpad-download-resources">
-        <!-- disable font downloading until TestFontRendering works on all plattforms -->
+        <!-- disable font downloading until TestFontRendering works on all platforms -->
         <!-- ... eventually copy the files into the resource dirs ... -->
         <!--
         <mkdir dir="build/scratchpad-test-resources"/>
@@ -3076,7 +3076,7 @@ under the License.
 
         <tstamp>
             <!-- usually overwritten by a command line argument -->
-            <!-- for convience this is kept in the same format as in the release-prep call  -->
+            <!-- for convenience this is kept in the same format as in the release-prep call  -->
             <format property="file_date" pattern="yyyyMMdd" locale="US"/>
         </tstamp>
 
@@ -3192,7 +3192,7 @@ under the License.
         </sequential>
     </macrodef>
 
-    <!-- TODO: currently this only saves a copy and paste snipplet to a file ...
+    <!-- TODO: currently this only saves a copy and paste snippet to a file ...
          would be nice if it really changes changes.xml and download.xml
          Doesn't work with Java6 - the https urls can't be accessed via loadChecksum
      -->
index 022fa17ab3dc63c26ce3d8a81171b0a6e348549d..0a5bd6865778eac0685a27f11335f69b418dddf4 100644 (file)
@@ -626,7 +626,7 @@ Unfortunately we often see builds break because of changes/new machines...''')
                 'adoptopenjdk_hotspot_8u282',
                 'ibmjdk_1.8.0_261'
         )
-        // Note H50 is reserved according to its node-descripion
+        // Note H50 is reserved according to its node-description
         label('Nodes','H22','H23','H24','H25','H26','H27','H28','H29','H30','H31','H32','H33','H34','H35','H36','H37','H38','H39','H40','H41','H42','H43','H44','H48','lucene1','lucene2','master')
     }
     steps {
index 22eec9633af33973262e5bf3a02d534fd282ee09..db65f767874447792c53de4dde4de6869cbaff89 100644 (file)
@@ -285,7 +285,7 @@ public final class ConditionalFormats {
     /**
      * Use Excel conditional formatting to hide the duplicate values,
      * and make the list easier to read. In this example, when the table is sorted by Region,
-     * the second (and subsequent) occurences of each region name will have white font colour.
+     * the second (and subsequent) occurrences of each region name will have white font colour.
      */
     static void hideDupplicates(Sheet sheet) {
         sheet.createRow(0).createCell(0).setCellValue("City");
@@ -309,7 +309,7 @@ public final class ConditionalFormats {
         sheetCF.addConditionalFormatting(regions, rule1);
 
         sheet.getRow(1).createCell(1).setCellValue("<== the second (and subsequent) " +
-                "occurences of each region name will have white font colour.  " +
+                "occurrences of each region name will have white font colour.  " +
                 "Condition: Formula Is   =A2=A1   (White Font)");
     }
 
index 7053d3bba9df42ae02dd405a382c6e680687a14d..8c1058341d7eef783c128f49fb46072a034a2c84 100644 (file)
@@ -689,7 +689,7 @@ public class ToCSV {
         // It is not wise to have such a wide catch clause - Exception is very
         // close to being at the top of the inheritance hierarchy - though it
         // will suffice for this example as it is really not possible to recover
-        // easilly from an exceptional set of circumstances at this point in the
+        // easil from an exceptional set of circumstances at this point in the
         // program. It should however, ideally be replaced with one or more
         // catch clauses optimised to handle more specific problems.
         catch(Exception ex) {
index a4170b6154af9be5913acca921c4d85725ff8913..ad098b7e98972a3c2c2753fded3ccd18445e0c85 100644 (file)
@@ -28,7 +28,7 @@ import org.junit.platform.launcher.TestIdentifier;
 /**
  * Custom listener class for Ants junitlauncher, because it chomps the important running details
  *
- * @see <a href="https://bz.apache.org/bugzilla/show_bug.cgi?id=64836">Bug 64836 - junitlaucher poor summary</a>
+ * @see <a href="https://bz.apache.org/bugzilla/show_bug.cgi?id=64836">Bug 64836 - junitlauncher poor summary</a>
  **/
 public class Junit5Progress implements TestExecutionListener {
     private final AtomicInteger numSkippedInTestSet = new AtomicInteger();
index cdcd7576719da2d62130dbef56fb6c4c2cd52ba2..6e070b125250495057c4ce68ca3cf4297e15c882 100644 (file)
@@ -43,7 +43,7 @@ public class POIXMLTypeLoader {
         DEFAULT_XML_OPTIONS.setCharacterEncoding("UTF-8");
         DEFAULT_XML_OPTIONS.setDisallowDocTypeDeclaration(true);
         DEFAULT_XML_OPTIONS.setEntityExpansionLimit(1);
-        // Piccolo is disabled for POI builts, i.e. JAXP is used for parsing
+        // JAXP is used for parsing
         // so only user code using XmlObject/XmlToken.Factory.parse
         // directly can bypass the entity check, which is probably unlikely (... and not within our responsibility :)) 
         // DEFAULT_XML_OPTIONS.setLoadEntityBytesLimit(4096);
index 7121651f062c51bb4780c4458e7863a3496df622..57b3d920706f800ba8a7b6cb9e020db5cc845c2b 100644 (file)
@@ -34,7 +34,7 @@ public class IdentifierManager {
 
     /**
      * @param lowerbound the lower limit of the id-range to manage. Must be greater than or equal to {@link #MIN_ID}.
-     * @param upperbound the upper limit of the id-range to manage. Must be less then or equal {@link #MAX_ID}.
+     * @param upperbound the upper limit of the id-range to manage. Must be less than or equal {@link #MAX_ID}.
      */
     public IdentifierManager(long lowerbound, long upperbound) {
         if (lowerbound > upperbound) {
index 3f8d95ac33d93cab7b4508ebdd98a8ac3fb79535..11467117054957a5c030daa3483e54eb54ebfc09 100644 (file)
@@ -212,7 +212,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
            // pack.originalPackagePath = file.getAbsolutePath();
            return pack;
        } catch (InvalidFormatException | RuntimeException e) {
-           // use revert() to free resources when the packgae is opened read-only
+           // use revert() to free resources when the package is opened read-only
            pack.revert();
 
            throw e;
@@ -495,7 +495,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
     /**
      * Add a thumbnail to the package. This method is provided to make easier
      * the addition of a thumbnail in a package. You can do the same work by
-     * using the traditionnal relationship and part mechanism.
+     * using the traditional relationship and part mechanism.
      *
      * @param path The full path to the image file.
      */
@@ -513,7 +513,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
     /**
      * Add a thumbnail to the package. This method is provided to make easier
      * the addition of a thumbnail in a package. You can do the same work by
-     * using the traditionnal relationship and part mechanism.
+     * using the traditional relationship and part mechanism.
      *
      * @param filename The full path to the image file.
      * @param data the image data
@@ -950,7 +950,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
             // If the specified partis flagged as deleted, we make it
             // available
             part.setDeleted(false);
-            // and delete the old part to replace it thereafeter
+            // and delete the old part to replace it thereafter
             this.partList.remove(part._partName);
         }
         this.partList.put(part._partName, part);
@@ -1554,7 +1554,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
 
     /**
      * Replace a content type in this package.<p>
-     * A typical scneario to call this method is to rename a template file to the main format, e.g.
+     * A typical scenario to call this method is to rename a template file to the main format, e.g.
      * <ul>
      *     <li>".dotx" to ".docx"</li>
      *     <li>".dotm" to ".docm"</li>
index 63af3519eaae8ff8c24215fce160ff9314b3d171..7883c6d2aeed507a74e3c95223eb3ade5c17923d 100644 (file)
@@ -111,7 +111,7 @@ public final class PackagePartName implements Comparable<PackagePartName> {
             partURI = new URI(partName);
         } catch (URISyntaxException e) {
             throw new IllegalArgumentException(
-                    "partName argmument is not a valid OPC part name !");
+                    "partName argument is not a valid OPC part name !");
         }
 
         if (checkConformance) {
@@ -364,13 +364,13 @@ public final class PackagePartName implements Comparable<PackagePartName> {
     }
 
     /**
-     * Throws an exception if the specified part name ends with a forwar slash
+     * Throws an exception if the specified part name ends with a forward slash
      * character '/'. [M1.5]
      *
      * @param partUri
      *            The part name to check.
      * @throws InvalidFormatException
-     *             If the specified part name ends with a forwar slash character
+     *             If the specified part name ends with a forward slash character
      *             '/'.
      */
     private static void throwExceptionIfPartNameEndsWithForwardSlashChar(
index bece2c742ec616b48439284949365d1b7c603800..350e63e22d08520b1c8b8a65737ebd09d6a5308a 100644 (file)
@@ -139,7 +139,7 @@ public final class PackageRelationshipCollection implements Iterable<PackageRela
      *            The part that own this relationships collection. If <b>null</b>
      *            then this part is considered as the package root.
      * @throws InvalidFormatException
-     *             If an error occurs during the parsing of the relatinships
+     *             If an error occurs during the parsing of the relationships
      *             part fo the specified part.
      */
     public PackageRelationshipCollection(OPCPackage container, PackagePart part)
index 07a4962cb7d78977f4d554e96e66099006201586..82cf69b32df654c0b6724d11d4a778c9f795ebcd 100644 (file)
@@ -486,7 +486,7 @@ public final class PackagingURIHelper {
      *
      * @param partName
      *            The part name to validate.
-     * @return The correspondant part name if valid, else <code>null</code>.
+     * @return The correspondent part name if valid, else <code>null</code>.
      * @throws InvalidFormatException
      *             Throws if the specified part name is not OPC compliant.
      * @see #createPartName(URI)
@@ -509,7 +509,7 @@ public final class PackagingURIHelper {
      *            The part name to validate.
      * @param relativePart
      *            The relative base part.
-     * @return The correspondant part name if valid, else <code>null</code>.
+     * @return The correspondent part name if valid, else <code>null</code>.
      * @throws InvalidFormatException
      *             Throws if the specified part name is not OPC compliant.
      * @see #createPartName(URI)
@@ -533,7 +533,7 @@ public final class PackagingURIHelper {
      *            The part name URI to validate.
      * @param relativePart
      *            The relative base part.
-     * @return The correspondant part name if valid, else <code>null</code>.
+     * @return The correspondent part name if valid, else <code>null</code>.
      * @throws InvalidFormatException
      *             Throws if the specified part name is not OPC compliant.
      * @see #createPartName(URI)
@@ -625,7 +625,7 @@ public final class PackagingURIHelper {
      *            Source part URI
      * @return the full path (as URI) of the relation file
      * @throws InvalidOperationException
-     *             Throws if the specified URI is a relationshp part.
+     *             Throws if the specified URI is a relationship part.
      */
     public static PackagePartName getRelationshipPartName(
             PackagePartName partName) {
@@ -688,7 +688,7 @@ public final class PackagingURIHelper {
              value = value.replace('\\', '/');
         }
 
-        // URI fragemnts (those starting with '#') are not encoded
+        // URI fragments (those starting with '#') are not encoded
         // and may contain white spaces and raw unicode characters
         int fragmentIdx = value.indexOf('#');
         if(fragmentIdx != -1){
index 1d47f0b45a3786c2b823cb8f2b7d205aa107565d..072fc6e28354d3474b0974d7909db0c84d273ff8 100644 (file)
@@ -402,7 +402,7 @@ public abstract class ContentTypeManager {
                 addDefaultContentType(extension, contentType);
             }
 
-            // Overriden content types
+            // Overridden content types
             NodeList overrideTypes = xmlContentTypetDoc.getDocumentElement().getElementsByTagNameNS(TYPES_NAMESPACE_URI, OVERRIDE_TAG_NAME);
             int overrideTypeCount = overrideTypes.getLength();
             for (int i = 0; i < overrideTypeCount; i++) {
index c87fb8e7c8bc029fd9fa259e73ca388c4d68910e..eae8234b9f61d810ffad9067b9ceebbe93e45268 100644 (file)
@@ -255,7 +255,7 @@ public class SignatureConfig {
     private String commitmentType = "Created and approved this document";
 
     /**
-     * Swtich to enable/disable automatic CRL download - by default the download is with all https hostname
+     * Switch to enable/disable automatic CRL download - by default the download is with all https hostname
      * and certificate verifications disabled.
      *
      * @since POI 5.2.1
@@ -834,7 +834,7 @@ public class SignatureConfig {
     }
 
     /**
-     * @param xadesIssuerNameNoReverseOrder when true, the issuer DN instead of the issuer X500 prinicpal is used
+     * @param xadesIssuerNameNoReverseOrder when true, the issuer DN instead of the issuer X500 principal is used
      */
     public void setXadesIssuerNameNoReverseOrder(boolean xadesIssuerNameNoReverseOrder) {
         this.xadesIssuerNameNoReverseOrder = xadesIssuerNameNoReverseOrder;
@@ -1004,7 +1004,7 @@ public class SignatureConfig {
 
 
     /**
-     * @return the cannonicalization method for XAdES-XL signing.
+     * @return the canonicalization method for XAdES-XL signing.
      * Defaults to {@code EXCLUSIVE}
      * @see <a href="http://docs.oracle.com/javase/7/docs/api/javax/xml/crypto/dsig/CanonicalizationMethod.html">javax.xml.crypto.dsig.CanonicalizationMethod</a>
      */
@@ -1013,7 +1013,7 @@ public class SignatureConfig {
     }
 
     /**
-     * @param xadesCanonicalizationMethod the cannonicalization method for XAdES-XL signing
+     * @param xadesCanonicalizationMethod the canonicalization method for XAdES-XL signing
      * @see <a href="http://docs.oracle.com/javase/7/docs/api/javax/xml/crypto/dsig/CanonicalizationMethod.html">javax.xml.crypto.dsig.CanonicalizationMethod</a>
      */
     public void setXadesCanonicalizationMethod(String xadesCanonicalizationMethod) {
index 7e5bbcf5ac2e234aaffe65d2661f76aca6ea5f23..b3c849f4a5162dbcd7a746840ded4a53431eb7db 100644 (file)
@@ -142,7 +142,7 @@ public class XAdESXLSignatureFacet implements SignatureFacet {
                 insertXChild(unsignedSigProps, validationData);
             }
         } catch (CertificateEncodingException e) {
-            throw new MarshalException("unable to create XAdES signatrue", e);
+            throw new MarshalException("unable to create XAdES signature", e);
         }
 
 
index 2e845e20d70bc553d8543609f978c1d2b24330f9..e429c22d229c4b0375eb5e77776a2189cea62d20 100644 (file)
@@ -215,7 +215,7 @@ public class TSPTimeStampService implements TimeStampService {
 
     /**
      * Check if CRL is to be added, check cached CRLs in config and download if necessary.
-     * Can be overriden to suppress the logic
+     * Can be overridden to suppress the logic
      * @return empty list, if not found or suppressed, otherwise the list of CRLs as encoded bytes
      */
     protected List<byte[]> retrieveCRL(SignatureConfig signatureConfig, X509Certificate holder) throws IOException {
index 4d620e50725793079db762deb652664f6e437017..25b3af8f938ff2d07b52885490b1fe0702011eb3 100644 (file)
@@ -845,7 +845,7 @@ public class XDGFShape extends XDGFSheet {
 
     /**
      * @return returns bounds as a path in local coordinates, which is
-     *         userful if you need to transform to global coordinates
+     *         useful if you need to transform to global coordinates
      *
      * Warning: Don't use this for 1d objects, and will fail for
      *          infinite line objects
index a52356dd6af40f5ef9008473d6976a6ab28143d3..a589855795bcd78f49232d7c8d9ac28b70fd2603 100644 (file)
@@ -358,7 +358,7 @@ public final class PPTX2PNG {
                 try {
                     return new SVGFormat(textAsShapes);
                 } catch (Exception | NoClassDefFoundError e) {
-                    LOG.atError().withThrowable(e).log("Batik is not not added to/working on the module-path. Use classpath mode instead of JPMS. Fallback to PNG.");
+                    LOG.atError().withThrowable(e).log("Batik is not added to/working on the module-path. Use classpath mode instead of JPMS. Fallback to PNG.");
                     return new BitmapFormat("png");
                 }
             }
index 9aaa8920178641b2575ce3828c4b1b77d94bd041..048a288d6181272e57eb1191a1f120c74fda77a2 100644 (file)
@@ -228,7 +228,7 @@ public class AbstractWordUtils
                 final String key = lsid + "#" + ( (int) element );
                 int num;
 
-                if ( !list.isStartAtOverriden( element )
+                if ( !list.isStartAtOverridden( element )
                         && numberingState.levels.containsKey( key ) )
                 {
                     num = numberingState.levels.get( key );
index 8a2d6d1ba4efc2781a79e6f1fee56f165c3b397e..0ba89ec9faf3018c137cf07f8d66ba13c0f3493f 100644 (file)
@@ -29,6 +29,7 @@ import org.apache.poi.hwpf.model.ListLevel;
 import org.apache.poi.hwpf.model.StyleSheet;
 import org.apache.poi.hwpf.sprm.CharacterSprmCompressor;
 import org.apache.poi.hwpf.sprm.ParagraphSprmCompressor;
+import org.apache.poi.util.Removal;
 
 /**
  * This class is used to create a list in a Word document. It is used in
@@ -157,7 +158,7 @@ public final class HWPFList
 
     public int getStartAt( char level )
     {
-        if ( isStartAtOverriden( level ) )
+        if ( isStartAtOverridden( level ) )
         {
             return _lfoData.getRgLfoLvl()[level].getIStartAt();
         }
@@ -178,7 +179,23 @@ public final class HWPFList
         return _ignoreLogicalLeftIdentation;
     }
 
-    public boolean isStartAtOverriden( char level )
+    /**
+     * @since POI 5.2.4
+     */
+    public boolean isStartAtOverridden(char level )
+    {
+        ListFormatOverrideLevel lfolvl = _lfoData.getRgLfoLvl().length > level ? _lfoData
+                .getRgLfoLvl()[level] : null;
+
+        return lfolvl != null && lfolvl.getIStartAt() != 0
+                && !lfolvl.isFormatting();
+    }
+
+    /**
+     * @deprecated use {@link #isStartAtOverridden(char)}
+     */
+    @Removal(version = "POI 6.0.0")
+    public boolean isStartAtOverriden(char level )
     {
         ListFormatOverrideLevel lfolvl = _lfoData.getRgLfoLvl().length > level ? _lfoData
                 .getRgLfoLvl()[level] : null;
index 003ad45c6b58b1ad4c96ce1d0d3187df50e14efd..424ab6eda815e24016fd5c5e0226ce7345695b10 100644 (file)
@@ -1719,7 +1719,7 @@ public final class InternalWorkbook {
      */
     public int createFormat(String formatString) {
 
-        maxformatid = maxformatid >= 0xa4 ? maxformatid + 1 : 0xa4; //Starting value from M$ empircal study.
+        maxformatid = maxformatid >= 0xa4 ? maxformatid + 1 : 0xa4; //Starting value from M$ empirical  study.
         FormatRecord rec = new FormatRecord(maxformatid, formatString);
 
         int pos = 0;
index 2c2399445dc703847aca9d2c9c0e737939560c24..0badfb51d104d832dd9576ed20179b06fba7cdb6 100644 (file)
@@ -136,9 +136,9 @@ public final class HSSFDataFormat implements DataFormat {
 
         String fmt = _formats.size() > index ? _formats.get(index) : null;
         if (_builtinFormats.length > index && _builtinFormats[index] != null) {
-           // It's in the built in range
+           // It's in the built-in range
            if (fmt != null) {
-              // It's been overriden, use that value
+              // It's been overridden, use that value
               return fmt;
            } else {
               // Standard built in format
index 28babc0c0f01a9c5a3bb3749fdae4eaef0475ff0..587d72eca6c99349412ce1e1ff7343321204aca1 100644 (file)
@@ -277,7 +277,7 @@ public final class CryptoFunctions {
 
     /**
      * Returns a new byte array with a truncated to the given size.
-     * If the hash has less then size bytes, it will be filled with 0x36-bytes
+     * If the hash has less than size bytes, it will be filled with 0x36-bytes
      *
      * @param hash the to be truncated/filled hash byte array
      * @param size the size of the returned byte array
@@ -289,7 +289,7 @@ public final class CryptoFunctions {
 
     /**
      * Returns a new byte array with a truncated to the given size.
-     * If the hash has less then size bytes, it will be filled with 0-bytes
+     * If the hash has less than size bytes, it will be filled with 0-bytes
      *
      * @param hash the to be truncated/filled hash byte array
      * @param size the size of the returned byte array
index f875da7fc523a7349be9302e6b130655366d742c..b0a0d23dbe2a8bc87ad435fb9c9e9251d677440b 100644 (file)
@@ -33,7 +33,7 @@ public class DrawNothing implements Drawable {
     /**
      * Apply 2-D transforms before drawing this shape. This includes rotation and flipping.
      *
-     * @param graphics the graphics whos transform matrix will be modified
+     * @param graphics the graphics whose transform matrix will be modified
      */
     public void applyTransform(Graphics2D graphics) {
     }
index 05407e6a78189a0caca58eabdac6188c6faa82fb..e1ab3bf176bfce2c8c442c2760742cfe50b24a1f 100644 (file)
@@ -52,7 +52,7 @@ public class DrawShape implements Drawable {
     /**
      * Apply 2-D transforms before drawing this shape. This includes rotation and flipping.
      *
-     * @param graphics the graphics whos transform matrix will be modified
+     * @param graphics the graphics whose transform matrix will be modified
      */
     @Override
     public void applyTransform(Graphics2D graphics) {
index 7f0a1a6c07c6a32cec9bda8470d679de10d87957..b11397ca4ff33c91e450bb643808015d17dbef01 100644 (file)
@@ -177,7 +177,7 @@ public interface Drawable {
     /**
      * Apply 2-D transforms before drawing this shape. This includes rotation and flipping.
      *
-     * @param graphics the graphics whos transform matrix will be modified
+     * @param graphics the graphics whose transform matrix will be modified
      */
     void applyTransform(Graphics2D graphics);
 
index db7380d9e940ebfa36d14feedec098a8dbcb908b..e4bfc14e6da7489d64f4970954285bc3bcecef19 100644 (file)
@@ -253,7 +253,7 @@ public abstract class MatrixFunction implements Function{
         protected double[] collectValues(ValueEval arg) throws EvaluationException {
             double[] values = instance.collectValues(arg);
 
-            /* handle case where MDETERM is operating on an array that that is not completely filled*/
+            /* handle case where MDETERM is operating on an array that is not completely filled*/
             if (arg instanceof AreaEval && values.length == 1)
                 throw new EvaluationException(ErrorEval.VALUE_INVALID);
 
@@ -297,7 +297,7 @@ public abstract class MatrixFunction implements Function{
         protected double[] collectValues(ValueEval arg) throws EvaluationException {
             double[] values = instance.collectValues(arg);
 
-            /* handle case where MDETERM is operating on an array that that is not completely filled*/
+            /* handle case where MDETERM is operating on an array that is not completely filled*/
             if (arg instanceof AreaEval && values.length == 1)
                 throw new EvaluationException(ErrorEval.VALUE_INVALID);
 
index fb3301e89fb7ddd36c9ab6754f3c41223475a38e..f87fcb286b4935f2dbd24814e5e584c6a11cf42b 100644 (file)
@@ -62,7 +62,7 @@ final class TestFormulaRecordAggregate {
         StringRecord sr = new StringRecord();
         sr.setString("NA");
         SharedValueManager svm = SharedValueManager.createEmpty();
-        // bug 46213 -> String record was  supplied but formula record flag is not  set
+        // bug 46213 -> String record was  supplied but formula record flag is not set
         FormulaRecordAggregate fra = new FormulaRecordAggregate(fr, sr, svm);
         List<org.apache.poi.hssf.record.Record> vraRecs = new ArrayList<>();
         fra.visitContainedRecords(vraRecs::add);
index c7135da492fb4f2999a3593380d87d87dbb810b2..10e7759a06442a85a1066b4266d6627e26147f52 100644 (file)
@@ -130,7 +130,7 @@ final class TestPOIFSFileSystem {
      *  multiple of 512 bytes)
      *
      * As yet, this problem remains. One school of thought is
-     *  not not issue an EOF when we discover the last block
+     *  to not issue an EOF when we discover the last block
      *  is short, but this seems a bit wrong.
      * The other is to fix the handling of the last block in
      *  POIFS, since it seems to be slight wrong
index 878b5ab9e4fd062911d2486a72628fb7f4a0418e..8ad953b6eb1ddf92b4bed7df3aae1f5aa4fc1841 100644 (file)
@@ -282,8 +282,7 @@ public abstract class BaseTestRow {
         assertEquals(5, row.getCell(5, MissingCellPolicy.CREATE_NULL_AS_BLANK).getColumnIndex());
 
 
-        // Now change the cell policy on the workbook, check
-        //  that that is now used if no policy given
+        // Now change the cell policy on the workbook, check that is now used if no policy given
         workbook.setMissingCellPolicy(MissingCellPolicy.RETURN_BLANK_AS_NULL);
 
         assertEquals(CellType.STRING,  row.getCell(0).getCellType());
index 5e8a41c1c4923f13d585000e487e986760565110..35bb9fb4d714dc3ec9078b9b970ff01bb7c93a3a 100644 (file)
@@ -85,7 +85,7 @@ public abstract class BaseTestSheetAutosizeColumn {
         // autosize not-evaluated cells, formula cells are sized as if the result is 0
         for (int i = 0; i < 6; i++) sheet.autoSizeColumn(i);
 
-        assertTrue(sheet.getColumnWidth(0) < sheet.getColumnWidth(1));  // width of '0' is less then width of '10'
+        assertTrue(sheet.getColumnWidth(0) < sheet.getColumnWidth(1));  // width of '0' is less than width of '10'
         assertEquals(sheet.getColumnWidth(1), sheet.getColumnWidth(2)); // 10 and '10' should be sized equally
         assertEquals(sheet.getColumnWidth(3), sheet.getColumnWidth(0)); // formula result is unknown, the width is calculated  for '0'
         assertEquals(sheet.getColumnWidth(4), sheet.getColumnWidth(5)); // 10.0000 and '10.0000'
@@ -95,7 +95,7 @@ public abstract class BaseTestSheetAutosizeColumn {
 
         for (int i = 0; i < 6; i++) sheet.autoSizeColumn(i);
 
-        assertTrue(sheet.getColumnWidth(0) < sheet.getColumnWidth(1));  // width of '0' is less then width of '10'
+        assertTrue(sheet.getColumnWidth(0) < sheet.getColumnWidth(1));  // width of '0' is less than width of '10'
         assertEquals(sheet.getColumnWidth(1), sheet.getColumnWidth(2)); // columns 1, 2 and 3 should have the same width
         assertEquals(sheet.getColumnWidth(2), sheet.getColumnWidth(3)); // columns 1, 2 and 3 should have the same width
         assertEquals(sheet.getColumnWidth(4), sheet.getColumnWidth(5)); // 10.0000 and '10.0000'