You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2017/05/27 01:54:41 UTC

svn commit: r1796363 - in /commons/proper/cli/trunk/src: changes/changes.xml main/java/org/apache/commons/cli/DefaultParser.java

Author: ggregory
Date: Sat May 27 01:54:41 2017
New Revision: 1796363

URL: http://svn.apache.org/viewvc?rev=1796363&view=rev
Log:
[CLI-276] Adjust access-modifier of checkRequiredOptions() to protected.

Modified:
    commons/proper/cli/trunk/src/changes/changes.xml
    commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/DefaultParser.java

Modified: commons/proper/cli/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/changes/changes.xml?rev=1796363&r1=1796362&r2=1796363&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/changes/changes.xml (original)
+++ commons/proper/cli/trunk/src/changes/changes.xml Sat May 27 01:54:41 2017
@@ -26,6 +26,9 @@
       <action type="add" dev="britter" due-to="Christoph Läubrich" issue="CLI-271">
         CommandLine.getXXX and CommandLine.hasXXX should accept an Option as a parameter
       </action>
+      <action type="add" dev="ggregory" due-to="Jason Dillon" issue="CLI-276">
+        Adjust access-modifier of checkRequiredOptions() to protected.
+      </action>
     </release>
 
     <release version="1.4" date="2017-03-09" description="New features and bug fixes">

Modified: commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/DefaultParser.java
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/DefaultParser.java?rev=1796363&r1=1796362&r2=1796363&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/DefaultParser.java (original)
+++ commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/DefaultParser.java Sat May 27 01:54:41 2017
@@ -191,7 +191,7 @@ public class DefaultParser implements Co
      * @throws MissingOptionException if any of the required Options
      * are not present.
      */
-    private void checkRequiredOptions() throws MissingOptionException
+    protected void checkRequiredOptions() throws MissingOptionException
     {
         // if there are required options that have not been processed
         if (!expectedOpts.isEmpty())