jfreechart-commit — Receives automatic notification of all CVS commits

[Jfreechart-commit] jcommon/source/org/jfree/date SerialDate.java,1.6,1.7

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/date
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv21128/source/org/jfree/date

Modified Files:
	SerialDate.java 
Log Message:
Fixed some FindBugs warnings.

Index: SerialDate.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/date/SerialDate.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -C2 -d -r1.6 -r1.7
*** SerialDate.java	18 Oct 2005 13:15:08 -0000	1.6
--- SerialDate.java	3 Nov 2005 09:25:17 -0000	1.7
***************
*** 138,154 ****
  
      /** The number of days in each month in non leap years. */
!     public static final int[] LAST_DAY_OF_MONTH =
          {0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
  
      /** The number of days in a (non-leap) year up to the end of each month. */
!     public static final int[] AGGREGATE_DAYS_TO_END_OF_MONTH =
          {0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
  
      /** The number of days in a year up to the end of the preceding month. */
!     public static final int[] AGGREGATE_DAYS_TO_END_OF_PRECEDING_MONTH =
          {0, 0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
  
      /** The number of days in a leap year up to the end of each month. */
!     public static final int[] LEAP_YEAR_AGGREGATE_DAYS_TO_END_OF_MONTH =
          {0, 31, 60, 91, 121, 152, 182, 213, 244, 274, 305, 335, 366};
  
--- 138,154 ----
  
      /** The number of days in each month in non leap years. */
!     static final int[] LAST_DAY_OF_MONTH =
          {0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
  
      /** The number of days in a (non-leap) year up to the end of each month. */
!     static final int[] AGGREGATE_DAYS_TO_END_OF_MONTH =
          {0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
  
      /** The number of days in a year up to the end of the preceding month. */
!     static final int[] AGGREGATE_DAYS_TO_END_OF_PRECEDING_MONTH =
          {0, 0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
  
      /** The number of days in a leap year up to the end of each month. */
!     static final int[] LEAP_YEAR_AGGREGATE_DAYS_TO_END_OF_MONTH =
          {0, 31, 60, 91, 121, 152, 182, 213, 244, 274, 305, 335, 366};
  
***************
*** 156,160 ****
       * The number of days in a leap year up to the end of the preceding month. 
       */
!     public static final int[] 
          LEAP_YEAR_AGGREGATE_DAYS_TO_END_OF_PRECEDING_MONTH =
              {0, 0, 31, 60, 91, 121, 152, 182, 213, 244, 274, 305, 335, 366};
--- 156,160 ----
       * The number of days in a leap year up to the end of the preceding month. 
       */
!     static final int[] 
          LEAP_YEAR_AGGREGATE_DAYS_TO_END_OF_PRECEDING_MONTH =
              {0, 0, 31, 60, 91, 121, 152, 182, 213, 244, 274, 305, 335, 366};
***************
*** 215,237 ****
  
      /**
!      * Returns true if the supplied integer code represents a valid 
!      * day-of-the-week, and false otherwise.
       *
       * @param code  the code being checked for validity.
       *
!      * @return true if the supplied integer code represents a valid 
!      *         day-of-the-week, and false otherwise.
       */
      public static boolean isValidWeekdayCode(final int code) {
  
          switch(code) {
!             case SUNDAY: return true;
!             case MONDAY: return true;
!             case TUESDAY: return true;
!             case WEDNESDAY: return true;
!             case THURSDAY: return true;
!             case FRIDAY: return true;
!             case SATURDAY: return true;
!             default: return false;
          }
  
--- 215,239 ----
  
      /**
!      * Returns <code>true</code> if the supplied integer code represents a 
!      * valid day-of-the-week, and <code>false</code> otherwise.
       *
       * @param code  the code being checked for validity.
       *
!      * @return <code>true</code> if the supplied integer code represents a 
!      *         valid day-of-the-week, and <code>false</code> otherwise.
       */
      public static boolean isValidWeekdayCode(final int code) {
  
          switch(code) {
!             case SUNDAY: 
!             case MONDAY: 
!             case TUESDAY: 
!             case WEDNESDAY: 
!             case THURSDAY: 
!             case FRIDAY: 
!             case SATURDAY: 
!                 return true;
!             default: 
!                 return false;
          }
  
***************
*** 325,341 ****
  
          switch(code) {
!             case JANUARY: return true;
!             case FEBRUARY: return true;
!             case MARCH: return true;
!             case APRIL: return true;
!             case MAY: return true;
!             case JUNE: return true;
!             case JULY: return true;
!             case AUGUST: return true;
!             case SEPTEMBER: return true;
!             case OCTOBER: return true;
!             case NOVEMBER: return true;
!             case DECEMBER: return true;
!             default: return false;
          }
  
--- 327,345 ----
  
          switch(code) {
!             case JANUARY: 
!             case FEBRUARY: 
!             case MARCH: 
!             case APRIL: 
!             case MAY: 
!             case JUNE: 
!             case JULY: 
!             case AUGUST: 
!             case SEPTEMBER: 
!             case OCTOBER: 
!             case NOVEMBER: 
!             case DECEMBER: 
!                 return true;
!             default: 
!                 return false;
          }
  
***************
*** 352,366 ****
  
          switch(code) {
!             case JANUARY: return 1;
!             case FEBRUARY: return 1;
              case MARCH: return 1;
!             case APRIL: return 2;
!             case MAY: return 2;
              case JUNE: return 2;
!             case JULY: return 3;
!             case AUGUST: return 3;
              case SEPTEMBER: return 3;
!             case OCTOBER: return 4;
!             case NOVEMBER: return 4;
              case DECEMBER: return 4;
              default: throw new IllegalArgumentException(
--- 356,370 ----
  
          switch(code) {
!             case JANUARY: 
!             case FEBRUARY: 
              case MARCH: return 1;
!             case APRIL: 
!             case MAY: 
              case JUNE: return 2;
!             case JULY: 
!             case AUGUST: 
              case SEPTEMBER: return 3;
!             case OCTOBER: 
!             case NOVEMBER: 
              case DECEMBER: return 4;
              default: throw new IllegalArgumentException(
***************
*** 477,484 ****
  
          switch(code) {
!             case FIRST_WEEK_IN_MONTH: return true;
!             case SECOND_WEEK_IN_MONTH: return true;
!             case THIRD_WEEK_IN_MONTH: return true;
!             case FOURTH_WEEK_IN_MONTH: return true;
              case LAST_WEEK_IN_MONTH: return true;
              default: return false;
--- 481,488 ----
  
          switch(code) {
!             case FIRST_WEEK_IN_MONTH: 
!             case SECOND_WEEK_IN_MONTH: 
!             case THIRD_WEEK_IN_MONTH: 
!             case FOURTH_WEEK_IN_MONTH: 
              case LAST_WEEK_IN_MONTH: return true;
              default: return false;

[Jfreechart-commit] jcommon/source/org/jfree/date RelativeDayOfWeekRule.java,1.4,1.5

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/date
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv21027/source/org/jfree/date

Modified Files:
	RelativeDayOfWeekRule.java 
Log Message:
Fixed minor issue reported by FindBugs

Index: RelativeDayOfWeekRule.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/date/RelativeDayOfWeekRule.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -C2 -d -r1.4 -r1.5
*** RelativeDayOfWeekRule.java	18 Oct 2005 13:15:08 -0000	1.4
--- RelativeDayOfWeekRule.java	3 Nov 2005 09:24:45 -0000	1.5
***************
*** 49,62 ****
   * (SerialDate.PRECEDING, SerialDate.NEAREST, SerialDate.FOLLOWING).
   * <P>
!  * For example, Good Friday can be specified as 'the Friday PRECEDING Easter Sunday'.
   *
   * @author David Gilbert
   */
! public class RelativeDayOfWeekRule extends AnnualDateRule implements Cloneable {
  
      /** A reference to the annual date rule on which this rule is based. */
      private AnnualDateRule subrule;
  
!     /** The day of the week (SerialDate.MONDAY, SerialDate.TUESDAY, and so on). */
      private int dayOfWeek;
  
--- 49,65 ----
   * (SerialDate.PRECEDING, SerialDate.NEAREST, SerialDate.FOLLOWING).
   * <P>
!  * For example, Good Friday can be specified as 'the Friday PRECEDING Easter 
!  * Sunday'.
   *
   * @author David Gilbert
   */
! public class RelativeDayOfWeekRule extends AnnualDateRule {
  
      /** A reference to the annual date rule on which this rule is based. */
      private AnnualDateRule subrule;
  
!     /** 
!      * The day of the week (SerialDate.MONDAY, SerialDate.TUESDAY, and so on). 
!      */
      private int dayOfWeek;
  
***************
*** 76,82 ****
       * @param subrule  the rule that determines the reference date.
       * @param dayOfWeek  the day-of-the-week relative to the reference date.
!      * @param relative  indicates *which* day-of-the-week (preceding, nearest or following).
       */
!     public RelativeDayOfWeekRule(final AnnualDateRule subrule, final int dayOfWeek, final int relative) {
          this.subrule = subrule;
          this.dayOfWeek = dayOfWeek;
--- 79,87 ----
       * @param subrule  the rule that determines the reference date.
       * @param dayOfWeek  the day-of-the-week relative to the reference date.
!      * @param relative  indicates *which* day-of-the-week (preceding, nearest 
!      *                  or following).
       */
!     public RelativeDayOfWeekRule(final AnnualDateRule subrule, 
!             final int dayOfWeek, final int relative) {
          this.subrule = subrule;
          this.dayOfWeek = dayOfWeek;
***************
*** 87,91 ****
       * Returns the sub-rule (also called the reference rule).
       *
!      * @return the annual date rule that determines the reference date for this rule.
       */
      public AnnualDateRule getSubrule() {
--- 92,97 ----
       * Returns the sub-rule (also called the reference rule).
       *
!      * @return The annual date rule that determines the reference date for this 
!      *         rule.
       */
      public AnnualDateRule getSubrule() {
***************
*** 96,100 ****
       * Sets the sub-rule.
       *
!      * @param subrule  the annual date rule that determines the reference date for this rule.
       */
      public void setSubrule(final AnnualDateRule subrule) {
--- 102,107 ----
       * Sets the sub-rule.
       *
!      * @param subrule  the annual date rule that determines the reference date 
!      *                 for this rule.
       */
      public void setSubrule(final AnnualDateRule subrule) {
***************
*** 114,118 ****
       * Sets the day-of-the-week for this rule.
       *
!      * @param dayOfWeek  the day-of-the-week (SerialDate.MONDAY, SerialDate.TUESDAY, and so on).
       */
      public void setDayOfWeek(final int dayOfWeek) {
--- 121,126 ----
       * Sets the day-of-the-week for this rule.
       *
!      * @param dayOfWeek  the day-of-the-week (SerialDate.MONDAY, 
!      *                   SerialDate.TUESDAY, and so on).
       */
      public void setDayOfWeek(final int dayOfWeek) {
***************
*** 121,128 ****
  
      /**
!      * Returns the 'relative' attribute, that determines *which* day-of-the-week we are
!      * interested in (SerialDate.PRECEDING, SerialDate.NEAREST or SerialDate.FOLLOWING).
       *
!      * @return the 'relative' attribute.
       */
      public int getRelative() {
--- 129,137 ----
  
      /**
!      * Returns the 'relative' attribute, that determines *which* 
!      * day-of-the-week we are interested in (SerialDate.PRECEDING, 
!      * SerialDate.NEAREST or SerialDate.FOLLOWING).
       *
!      * @return The 'relative' attribute.
       */
      public int getRelative() {
***************
*** 134,138 ****
       * SerialDate.FOLLOWING).
       *
!      * @param relative  determines *which* day-of-the-week is selected by this rule.
       */
      public void setRelative(final int relative) {
--- 143,148 ----
       * SerialDate.FOLLOWING).
       *
!      * @param relative  determines *which* day-of-the-week is selected by this 
!      *                  rule.
       */
      public void setRelative(final int relative) {
***************
*** 148,152 ****
       */
      public Object clone() throws CloneNotSupportedException {
!         final RelativeDayOfWeekRule duplicate = (RelativeDayOfWeekRule) super.clone();
          duplicate.subrule = (AnnualDateRule) duplicate.getSubrule().clone();
          return duplicate;
--- 158,163 ----
       */
      public Object clone() throws CloneNotSupportedException {
!         final RelativeDayOfWeekRule duplicate 
!             = (RelativeDayOfWeekRule) super.clone();
          duplicate.subrule = (AnnualDateRule) duplicate.getSubrule().clone();
          return duplicate;
***************
*** 158,162 ****
       * @param year  the year (1900 &lt;= year &lt;= 9999).
       *
!      * @return the date generated by the rule for the given year (null possible).
       */
      public SerialDate getDate(final int year) {
--- 169,174 ----
       * @param year  the year (1900 &lt;= year &lt;= 9999).
       *
!      * @return The date generated by the rule for the given year (possibly 
!      *         <code>null</code>).
       */
      public SerialDate getDate(final int year) {
***************
*** 174,199 ****
  
          if (base != null) {
- 
              switch (this.relative) {
- 
                  case(SerialDate.PRECEDING):
!                     result = SerialDate.getPreviousDayOfWeek(this.dayOfWeek, base);
                      break;
- 
                  case(SerialDate.NEAREST):
!                     result = SerialDate.getNearestDayOfWeek(this.dayOfWeek, base);
                      break;
- 
                  case(SerialDate.FOLLOWING):
!                     result = SerialDate.getFollowingDayOfWeek(this.dayOfWeek, base);
                      break;
- 
                  default:
                      break;
- 
              }
- 
          }
- 
          return result;
  
--- 186,206 ----
  
          if (base != null) {
              switch (this.relative) {
                  case(SerialDate.PRECEDING):
!                     result = SerialDate.getPreviousDayOfWeek(this.dayOfWeek, 
!                             base);
                      break;
                  case(SerialDate.NEAREST):
!                     result = SerialDate.getNearestDayOfWeek(this.dayOfWeek, 
!                             base);
                      break;
                  case(SerialDate.FOLLOWING):
!                     result = SerialDate.getFollowingDayOfWeek(this.dayOfWeek, 
!                             base);
                      break;
                  default:
                      break;
              }
          }
          return result;
  

[Jfreechart-commit] jcommon/source/org/jfree/xml/factory/objects DecimalFormatObjectDescription.java,1.2,1.3

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/factory/objects
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv16323/source/org/jfree/xml/factory/objects

Modified Files:
	DecimalFormatObjectDescription.java 
Log Message:
Removed deprecated method in preparation for 1.0.0 release.

Index: DecimalFormatObjectDescription.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/xml/factory/objects/DecimalFormatObjectDescription.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -C2 -d -r1.2 -r1.3
*** DecimalFormatObjectDescription.java	18 Oct 2005 13:31:58 -0000	1.2
--- DecimalFormatObjectDescription.java	2 Nov 2005 17:01:05 -0000	1.3
***************
*** 90,104 ****
  
      /**
-      * Creates a new object description.
-      *
-      * @param className  the class.
-      * @param init  initialise
-      * @deprecated should no longer be used...
-      */
-     public DecimalFormatObjectDescription(final Class className, final boolean init) {
-         this(className);
-     }
- 
-     /**
       * Sets the parameters of this description object to match the supplied object.
       *
--- 90,93 ----

[Jfreechart-commit] jcommon/source/org/jfree/xml/parser Base64ReadHandler.java,1.2,NONE

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/parser
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv16139/source/org/jfree/xml/parser

Removed Files:
	Base64ReadHandler.java 
Log Message:
Removed deprecated class in preparation for 1.0.0 release.

--- Base64ReadHandler.java DELETED ---

[Jfreechart-commit] jcommon/source/org/jfree/xml/writer XMLWriterSupport.java,1.4,1.5

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/writer
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv15971/source/org/jfree/xml/writer

Modified Files:
	XMLWriterSupport.java 
Log Message:
Removed deprecated method in preparation for 1.0.0 release.

Index: XMLWriterSupport.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/xml/writer/XMLWriterSupport.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -C2 -d -r1.4 -r1.5
*** XMLWriterSupport.java	18 Oct 2005 13:35:06 -0000	1.4
--- XMLWriterSupport.java	2 Nov 2005 17:00:23 -0000	1.5
***************
*** 46,52 ****
  import java.io.IOException;
  import java.io.Writer;
- import java.util.Enumeration;
  import java.util.Iterator;
- import java.util.Properties;
  
  /**
--- 46,50 ----
***************
*** 247,272 ****
       *
       * @throws java.io.IOException if there is an I/O problem.
-      * @deprecated use the attribute list instead of the properties.
-      */
-     public void writeTag(final Writer w, final String name, final Properties attributes, final boolean close)
-         throws IOException {
-         final AttributeList attList = new AttributeList();
-         final Enumeration keys = attributes.keys();
-         while (keys.hasMoreElements()) {
-             final String key = (String) keys.nextElement();
-             attList.setAttribute(key, attributes.getProperty(key));
-         }
-         writeTag(w, name, attList, close);
-     }
- 
-     /**
-      * Writes an opening XML tag along with a list of attribute/value pairs.
-      *
-      * @param w  the writer.
-      * @param name  the tag name.
-      * @param attributes  the attributes.
-      * @param close  controls whether the tag is closed.
-      *
-      * @throws java.io.IOException if there is an I/O problem.
       */
      public void writeTag(final Writer w, final String name, final AttributeList attributes, final boolean close)
--- 245,248 ----

[Jfreechart-commit] jcommon/source/org/jfree/xml/writer XMLWriter.java,1.3,1.4

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/writer
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv15793/source/org/jfree/xml/writer

Modified Files:
	XMLWriter.java 
Log Message:
Removed deprecated method in preparation for 1.0.0 release (also some reformatting for 80 cols).

Index: XMLWriter.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/xml/writer/XMLWriter.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -C2 -d -r1.3 -r1.4
*** XMLWriter.java	18 Oct 2005 13:35:06 -0000	1.3
--- XMLWriter.java	2 Nov 2005 16:59:43 -0000	1.4
***************
*** 45,49 ****
  import java.io.IOException;
  import java.io.Writer;
- import java.util.Properties;
  
  /**
--- 45,48 ----
***************
*** 58,63 ****
  
      /**
!      * Creates a new XML writer for the specified character stream.  By default, four spaces are
!      * used for indentation.
       *
       * @param writer the character stream.
--- 57,62 ----
  
      /**
!      * Creates a new XML writer for the specified character stream.  By 
!      * default, four spaces are used for indentation.
       *
       * @param writer the character stream.
***************
*** 71,76 ****
       *
       * @param writer       the character stream.
!      * @param indentString the string used for indentation (should contain white space, for
!      *                     example four spaces).
       */
      public XMLWriter(final Writer writer, final String indentString) {
--- 70,75 ----
       *
       * @param writer       the character stream.
!      * @param indentString the string used for indentation (should contain 
!      *                     white space, for example four spaces).
       */
      public XMLWriter(final Writer writer, final String indentString) {
***************
*** 84,90 ****
  
      /**
!      * Writes the XML declaration that usually appears at the top of every XML file.
       * 
!      * @throws IOException if there is a problem writing to the character stream.
       */
      public void writeXmlDeclaration() throws IOException {
--- 83,91 ----
  
      /**
!      * Writes the XML declaration that usually appears at the top of every XML 
!      * file.
       * 
!      * @throws IOException if there is a problem writing to the character 
!      *                     stream.               
       */
      public void writeXmlDeclaration() throws IOException {
***************
*** 97,104 ****
       *
       * @param name  the tag name.
!      * @param close a flag that controls whether or not the tag is closed immediately.
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final boolean close) throws IOException {
          if (close) {
              writeTag(this.writer, name, new AttributeList(), close);
--- 98,107 ----
       *
       * @param name  the tag name.
!      * @param close a flag that controls whether or not the tag is closed 
!      *              immediately.
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final boolean close) 
!             throws IOException {
          if (close) {
              writeTag(this.writer, name, new AttributeList(), close);
***************
*** 128,133 ****
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final String attributeName, final String attributeValue,
!                          final boolean close) throws IOException {
          writeTag(this.writer, name, attributeName, attributeValue, close);
      }
--- 131,137 ----
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final String attributeName, 
!                          final String attributeValue, final boolean close) 
!             throws IOException {
          writeTag(this.writer, name, attributeName, attributeValue, close);
      }
***************
*** 141,158 ****
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final AttributeList attributes, final boolean close) throws IOException {
!         writeTag(this.writer, name, attributes, close);
!     }
! 
!     /**
!      * Writes an opening XML tag along with a list of attribute/value pairs.
!      *
!      * @param name       the tag name.
!      * @param attributes the attributes.
!      * @param close      controls whether the tag is closed.
!      * @throws java.io.IOException if there is an I/O problem.
!      * @deprecated use the attribute list instead ...
!      */
!     public void writeTag(final String name, final Properties attributes, final boolean close) throws IOException {
          writeTag(this.writer, name, attributes, close);
      }
--- 145,150 ----
       * @throws java.io.IOException if there is an I/O problem.
       */
!     public void writeTag(final String name, final AttributeList attributes, 
!                          final boolean close) throws IOException {
          writeTag(this.writer, name, attributes, close);
      }
***************
*** 162,166 ****
       *
       * @param text the text.
!      * @throws IOException if there is a problem writing to the character stream.
       */
      public void writeText(final String text) throws IOException {
--- 154,159 ----
       *
       * @param text the text.
!      * @throws IOException if there is a problem writing to the character 
!      *                     stream.
       */
      public void writeText(final String text) throws IOException {

[Jfreechart-commit] jcommon/source/org/jfree/xml/factory/objects ClassFactoryCollector.java,1.4,1.5

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/factory/objects
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv11664/source/org/jfree/xml/factory/objects

Modified Files:
	ClassFactoryCollector.java 
Log Message:
Removed unused import.

Index: ClassFactoryCollector.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/xml/factory/objects/ClassFactoryCollector.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -C2 -d -r1.4 -r1.5
*** ClassFactoryCollector.java	18 Oct 2005 13:31:58 -0000	1.4
--- ClassFactoryCollector.java	2 Nov 2005 16:37:32 -0000	1.5
***************
*** 46,50 ****
  
  import java.util.ArrayList;
- import org.jfree.util.ClassComparator;
  import java.util.Iterator;
  
--- 46,49 ----

[Jfreechart-commit] jcommon/source/org/jfree/xml/factory/objects ClassComparator.java,1.3,NONE

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/factory/objects
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv11600/source/org/jfree/xml/factory/objects

Removed Files:
	ClassComparator.java 
Log Message:
Removed deprecated class in preparation for 1.0.0 release.

--- ClassComparator.java DELETED ---

[Jfreechart-commit] jcommon/source/org/jfree/ui UIUtilities.java,1.5,1.6

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv11538/source/org/jfree/ui

Modified Files:
	UIUtilities.java 
Log Message:
Fixed minor error in header.

Index: UIUtilities.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/ui/UIUtilities.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -C2 -d -r1.5 -r1.6
*** UIUtilities.java	18 Oct 2005 13:18:34 -0000	1.5
--- UIUtilities.java	2 Nov 2005 16:36:50 -0000	1.6
***************
*** 25,31 ****
   * in the United States and other countries.]
   *
!  * ---------------
!  * TextAnchor.java
!  * ---------------
   * (C) Copyright 2002-2005, by Object Refinery Limited.
   *
--- 25,31 ----
   * in the United States and other countries.]
   *
!  * ----------------
!  * UIUtilities.java
!  * ----------------
   * (C) Copyright 2002-2005, by Object Refinery Limited.
   *
***************
*** 39,42 ****
--- 39,43 ----
   * 25-Jul-2002 : Version 1 (TM);
   */
+ 
  package org.jfree.ui;
  

[Jfreechart-commit] jcommon/source/org/jfree/ui SortableTableModel.java,1.3,1.4

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv11490/source/org/jfree/ui

Modified Files:
	SortableTableModel.java 
Log Message:
Minor API doc updates and reformatting for 80 cols.

Index: SortableTableModel.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/ui/SortableTableModel.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -C2 -d -r1.3 -r1.4
*** SortableTableModel.java	18 Oct 2005 13:18:34 -0000	1.3
--- SortableTableModel.java	2 Nov 2005 16:36:28 -0000	1.4
***************
*** 28,32 ****
   * SortableTableModel.java
   * -----------------------
!  * (C) Copyright 2000-2004, by Object Refinery Limited;
   *
   * Original Author:  David Gilbert (for Object Refinery Limited);
--- 28,32 ----
   * SortableTableModel.java
   * -----------------------
!  * (C) Copyright 2000-2005, by Object Refinery Limited;
   *
   * Original Author:  David Gilbert (for Object Refinery Limited);
***************
*** 69,73 ****
  
      /**
!      * Returns the index of the sorting column, or -1 if the data is not sorted on any column.
       *
       * @return the column used for sorting.
--- 69,74 ----
  
      /**
!      * Returns the index of the sorting column, or -1 if the data is not sorted
!      * on any column.
       *
       * @return the column used for sorting.
***************
*** 78,94 ****
  
      /**
!      * Returns true if the data is sorted in ascending order, and false otherwise.
!      *
!      * @return true if the data is sorted in ascending order, and false otherwise.
!      * @deprecated use isAscending instead, as that method complies to the Beans naming convention.
!      */
!     public boolean getAscending() {
!         return this.ascending;
!     }
! 
!     /**
!      * Returns true if the data is sorted in ascending order, and false otherwise.
       *
!      * @return true if the data is sorted in ascending order, and false otherwise.
       */
      public boolean isAscending() {
--- 79,87 ----
  
      /**
!      * Returns <code>true</code> if the data is sorted in ascending order, and 
!      * <code>false</code> otherwise.
       *
!      * @return <code>true</code> if the data is sorted in ascending order, and 
!      *         <code>false</code> otherwise.
       */
      public boolean isAscending() {
***************
*** 97,101 ****
  
      /**
!      * Sets the flag that determines whether the sort order is ascending or descending.
       *
       * @param flag  the flag.
--- 90,95 ----
  
      /**
!      * Sets the flag that determines whether the sort order is ascending or 
!      * descending.
       *
       * @param flag  the flag.

[Jfreechart-commit] jcommon/source/org/jfree/xml/attributehandlers package.html,NONE,1.1

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/xml/attributehandlers
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10853/source/org/jfree/xml/attributehandlers

Added Files:
	package.html 
Log Message:
New package description for API docs.

--- NEW FILE: package.html ---
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<html>
<body bgcolor="white">
Provides attribute handlers for standard object types.
</body>
</html>

[Jfreechart-commit] jcommon/source/org/jfree/util ShapeUtilities.java,1.16,1.17

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/util
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10754/source/org/jfree/util

Modified Files:
	ShapeUtilities.java 
Log Message:
Fixed some Checkstyle warnings.

Index: ShapeUtilities.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/util/ShapeUtilities.java,v
retrieving revision 1.16
retrieving revision 1.17
diff -C2 -d -r1.16 -r1.17
*** ShapeUtilities.java	18 Oct 2005 13:24:19 -0000	1.16
--- ShapeUtilities.java	2 Nov 2005 16:31:50 -0000	1.17
***************
*** 110,114 ****
                  return (Shape) ObjectUtilities.clone(shape);
              }
!             catch(CloneNotSupportedException cnse) {
              }
          }
--- 110,114 ----
                  return (Shape) ObjectUtilities.clone(shape);
              }
!             catch (CloneNotSupportedException cnse) {
              }
          }
***************
*** 581,587 ****
          final double h = rect2.getHeight();
  
!         return ((x >= x0) && (y >= y0) &&
!                 ((x + w) <= (x0 + rect1.getWidth())) &&
!                 ((y + h) <= (y0 + rect1.getHeight())));
      
      }
--- 581,587 ----
          final double h = rect2.getHeight();
  
!         return ((x >= x0) && (y >= y0) 
!                 && ((x + w) <= (x0 + rect1.getWidth())) 
!                 && ((y + h) <= (y0 + rect1.getHeight())));
      
      }
***************
*** 597,601 ****
       * @return A boolean.
       */
!     public static boolean intersects (final Rectangle2D rect1, final Rectangle2D rect2) {
  
        final double x0 = rect1.getX();
--- 597,602 ----
       * @return A boolean.
       */
!     public static boolean intersects (final Rectangle2D rect1, 
!                                       final Rectangle2D rect2) {
  
        final double x0 = rect1.getX();
***************
*** 606,613 ****
        final double y = rect2.getY();
        final double height = rect2.getHeight();
!       return (x + width >= x0 &&
!               y + height >= y0 &&
!               x <= x0 + rect1.getWidth() &&
!               y <= y0 + rect1.getHeight());
      }
  }
--- 607,612 ----
        final double y = rect2.getY();
        final double height = rect2.getHeight();
!       return (x + width >= x0 && y + height >= y0 && x <= x0 + rect1.getWidth()
!               && y <= y0 + rect1.getHeight());
      }
  }

[Jfreechart-commit] jcommon/source/org/jfree/ui/tabbedui package.html,NONE,1.1

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui/tabbedui
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10670/source/org/jfree/ui/tabbedui

Added Files:
	package.html 
Log Message:
New package description for API docs.

--- NEW FILE: package.html ---
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<html>
<body bgcolor="white">
Provides classes for tabbed user interfaces.
</body>
</html>

[Jfreechart-commit] jcommon/source/org/jfree/ui/action package.html,NONE,1.1

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui/action
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10588/source/org/jfree/ui/action

Added Files:
	package.html 
Log Message:
New package description for API docs.

--- NEW FILE: package.html ---
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<html>
<body bgcolor="white">
Provides an Action facility matching that introduced in JDK 1.3 (JCommon aims to run on JDK 1.2.2 or later).
</body>
</html>

[Jfreechart-commit] jcommon/source/org/jfree/ui/about Library.java,1.3,NONE

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui/about
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10517/source/org/jfree/ui/about

Removed Files:
	Library.java 
Log Message:
Removed deprecated class in preparation for 1.0.0 release.

--- Library.java DELETED ---

[Jfreechart-commit] jcommon/source/org/jfree/ui Spacer.java,1.10,NONE

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10388/source/org/jfree/ui

Removed Files:
	Spacer.java 
Log Message:
Removed unused Spacer class.

--- Spacer.java DELETED ---

[Jfreechart-commit] jcommon/source/org/jfree/ui/junit UIPackageTests.java,1.3,1.4 SpacerTests.java,1.3,NONE

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui/junit
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10388/source/org/jfree/ui/junit

Modified Files:
	UIPackageTests.java 
Removed Files:
	SpacerTests.java 
Log Message:
Removed unused Spacer class.

Index: UIPackageTests.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/ui/junit/UIPackageTests.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -C2 -d -r1.3 -r1.4
*** UIPackageTests.java	18 Oct 2005 13:22:36 -0000	1.3
--- UIPackageTests.java	2 Nov 2005 16:30:23 -0000	1.4
***************
*** 67,71 ****
          suite.addTestSuite(RectangleInsetsTests.class);
          suite.addTestSuite(Size2DTests.class);
-         suite.addTestSuite(SpacerTests.class);
          suite.addTestSuite(StandardGradientPaintTransformerTests.class);
          suite.addTestSuite(TextAnchorTests.class);
--- 67,70 ----

--- SpacerTests.java DELETED ---

[Jfreechart-commit] jcommon/source/org/jfree/io SerialUtilities.java,1.11,1.12

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/io
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10135/source/org/jfree/io

Modified Files:
	SerialUtilities.java 
Log Message:
Fixed some Checkstyle warnings.

Index: SerialUtilities.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/io/SerialUtilities.java,v
retrieving revision 1.11
retrieving revision 1.12
diff -C2 -d -r1.11 -r1.12
*** SerialUtilities.java	18 Oct 2005 13:16:02 -0000	1.11
--- SerialUtilities.java	2 Nov 2005 16:29:35 -0000	1.12
***************
*** 322,334 ****
                  while (!hasNext) {
                      final int type = stream.readInt();
!                     for(int i=0; i<6; i++) {
                          args[i] = stream.readFloat();
                      }
                      switch (type) { 
                          case PathIterator.SEG_MOVETO :  
!                             gp.moveTo(args[0],args[1]);
                              break;
                          case PathIterator.SEG_LINETO :                           
!                             gp.lineTo(args[0],args[1]);
                              break; 
                          case PathIterator.SEG_CUBICTO :
--- 322,334 ----
                  while (!hasNext) {
                      final int type = stream.readInt();
!                     for(int i = 0; i < 6; i++) {
                          args[i] = stream.readFloat();
                      }
                      switch (type) { 
                          case PathIterator.SEG_MOVETO :  
!                             gp.moveTo(args[0], args[1]);
                              break;
                          case PathIterator.SEG_LINETO :                           
!                             gp.lineTo(args[0], args[1]);
                              break; 
                          case PathIterator.SEG_CUBICTO :
***************
*** 419,423 ****
                  final float[] args = new float[6];
                  stream.writeBoolean(pi.isDone());
!                 while(!pi.isDone()) {
                      final int type = pi.currentSegment(args);
                      stream.writeInt(type);
--- 419,423 ----
                  final float[] args = new float[6];
                  stream.writeBoolean(pi.isDone());
!                 while (!pi.isDone()) {
                      final int type = pi.currentSegment(args);
                      stream.writeInt(type);

[Jfreechart-commit] jcommon/source/org/jfree/base package.html,NONE,1.1

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/base
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10013/source/org/jfree/base

Added Files:
	package.html 
Log Message:
New package description for API docs.

--- NEW FILE: package.html ---
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<html>
<body bgcolor="white">
Base level classes.
</body>
</html>

[Jfreechart-commit] jcommon/source/org/jfree/base Library.java,1.3,1.4

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/base
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv9937/source/org/jfree/base

Modified Files:
	Library.java 
Log Message:
Minor API doc updates and reformatting for 80 cols.

Index: Library.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/base/Library.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -C2 -d -r1.3 -r1.4
*** Library.java	18 Oct 2005 13:13:58 -0000	1.3
--- Library.java	2 Nov 2005 16:28:49 -0000	1.4
***************
*** 39,42 ****
--- 39,43 ----
   * 21-Feb-2002 : Version 1 (DG);
   * 25-Mar-2002 : Added a new constructor (DG);
+  * 02-Nov-2005 : Minor API doc updates (DG);
   *
   */
***************
*** 44,51 ****
  package org.jfree.base;
  
  /**
!  * A simple class representing a library in a software project.
!  * <P>
!  * Used in the AboutFrame class.
   *
   * @author David Gilbert
--- 45,53 ----
  package org.jfree.base;
  
+ import org.jfree.ui.about.AboutFrame;
+ 
  /**
!  * A simple class representing a library in a software project.  For use in 
!  * the {@link AboutFrame} class.
   *
   * @author David Gilbert
***************
*** 73,77 ****
       * @param info  the web address or other info.
       */
!     public Library(final String name, final String version, final String licence, final String info) {
  
          this.name = name;
--- 75,80 ----
       * @param info  the web address or other info.
       */
!     public Library(final String name, final String version, 
!                    final String licence, final String info) {
  
          this.name = name;
***************
*** 134,140 ****
  
      /**
!      * Sets the license name.
       * 
!      * @param licenceName  the license name.
       */
      protected void setLicenceName(final String licenceName) {
--- 137,143 ----
  
      /**
!      * Sets the licence name.
       * 
!      * @param licenceName  the licence name.
       */
      protected void setLicenceName(final String licenceName) {

[Jfreechart-commit] jcommon/source/com/keypoint PngEncoder.java,1.5,1.6

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/com/keypoint
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv9863/source/com/keypoint

Modified Files:
	PngEncoder.java 
Log Message:
Removed spurious API doc comments, they were causing a warning in the build.

Index: PngEncoder.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/com/keypoint/PngEncoder.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -C2 -d -r1.5 -r1.6
*** PngEncoder.java	18 Oct 2005 13:13:34 -0000	1.5
--- PngEncoder.java	2 Nov 2005 16:28:19 -0000	1.6
***************
*** 216,223 ****
      /**
       * Returns the image to be encoded.
-      *
-      * @param image A Java Image object which uses the DirectColorModel
-      * @see java.awt.Image
-      * @see java.awt.image.DirectColorModel
       */
      public Image getImage() {
--- 216,219 ----

[Jfreechart-commit] jcommon/source/com/keypoint package.html,1.1,1.2

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/com/keypoint
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv9769/source/com/keypoint

Modified Files:
	package.html 
Log Message:
Remove '.' it is garbling the API doc output.

Index: package.html
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/com/keypoint/package.html,v
retrieving revision 1.1
retrieving revision 1.2
diff -C2 -d -r1.1 -r1.2
*** package.html	15 Jul 2004 14:48:39 -0000	1.1
--- package.html	2 Nov 2005 16:27:39 -0000	1.2
***************
*** 2,6 ****
  <html>
  <body bgcolor="white">
! A package containing the PNG encoder written by J. David Eisenberg.
  </body>
  </html>
--- 2,6 ----
  <html>
  <body bgcolor="white">
! A package containing the PNG encoder written by J David Eisenberg.
  </body>
  </html>

[Jfreechart-commit] jcommon/source/org/jfree/util Configuration.java,1.5,1.6

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/util
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv27143/source/org/jfree/util

Modified Files:
	Configuration.java 
Log Message:
Small API doc addition to kill warning during build.

Index: Configuration.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/util/Configuration.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -C2 -d -r1.5 -r1.6
*** Configuration.java	18 Oct 2005 13:24:19 -0000	1.5
--- Configuration.java	2 Nov 2005 15:23:29 -0000	1.6
***************
*** 28,32 ****
   * Configuration.java
   * ------------------
!  * (C)opyright 2002-2004, by Thomas Morgner and Contributors.
   *
   * Original Author:  Thomas Morgner (ta...@sh...);
--- 28,32 ----
   * Configuration.java
   * ------------------
!  * (C)opyright 2002-2005, by Thomas Morgner and Contributors.
   *
   * Original Author:  Thomas Morgner (ta...@sh...);
***************
*** 64,72 ****
  
      /**
!      * Returns the configuration property with the specified key (or the specified default value
!      * if there is no such property).
       * <p>
!      * If the property is not defined in this configuration, the code will lookup the property in
!      * the parent configuration.
       *
       * @param key  the property key.
--- 64,72 ----
  
      /**
!      * Returns the configuration property with the specified key (or the 
!      * specified default value if there is no such property).
       * <p>
!      * If the property is not defined in this configuration, the code will 
!      * lookup the property in the parent configuration.
       *
       * @param key  the property key.
***************
*** 86,91 ****
  
      /**
       * 
!      * @return
       */
      public Enumeration getConfigProperties();
--- 86,92 ----
  
      /**
+      * Returns the configuration properties.
       * 
!      * @return The configuration properties.
       */
      public Enumeration getConfigProperties();

[Jfreechart-commit] jcommon/source/org/jfree/demo DateChooserPanelDemo.java,NONE,1.1

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/demo
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv12623/source/org/jfree/demo

Added Files:
	DateChooserPanelDemo.java 
Log Message:
New demo.

--- NEW FILE: DateChooserPanelDemo.java ---
/* ========================================================================
 * JCommon : a free general purpose class library for the Java(tm) platform
 * ========================================================================
 *
 * (C) Copyright 2000-2005, by Object Refinery Limited and Contributors.
 * 
 * Project Info:  http://www.jfree.org/jcommon/index.html
 *
 * This library is free software; you can redistribute it and/or modify it 
 * under the terms of the GNU Lesser General Public License as published by 
 * the Free Software Foundation; either version 2.1 of the License, or 
 * (at your option) any later version.
 *
 * This library is distributed in the hope that it will be useful, but 
 * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY 
 * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public 
 * License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this library; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, 
 * USA.  
 *
 * [Java is a trademark or registered trademark of Sun Microsystems, Inc. 
 * in the United States and other countries.]
 * 
 * -------------------------
 * DateChooserPanelDemo.java
 * -------------------------
 * (C) Copyright 2005, by Object Refinery Limited.
 *
 * Original Author:  David Gilbert (for Object Refinery Limited);
 * Contributor(s):   -;
 *
 * $Id: DateChooserPanelDemo.java,v 1.1 2005/11/02 14:11:22 mungady Exp $
 *
 * Changes
 * -------
 * 02-Nov-2005 : Version 1;
 *
 */


package org.jfree.demo;

import org.jfree.ui.ApplicationFrame;
import org.jfree.ui.DateChooserPanel;
import org.jfree.ui.RefineryUtilities;

/**
 * A simple demo showing the {@link DateChooserPanel}.
 */
public class DateChooserPanelDemo extends ApplicationFrame {

    /**
     * Creates a new demo.
     *
     * @param title  the frame title.
     */
    public DateChooserPanelDemo(String title) {
        super(title);
        setContentPane(new DateChooserPanel());
    }
    
    /**
     * Starting point for the demonstration application.
     *
     * @param args  ignored.
     */
    public static void main(String[] args) {
        DateChooserPanelDemo demo = new DateChooserPanelDemo(
            "DateChooserPanel Demo");
        demo.pack();
        RefineryUtilities.centerFrameOnScreen(demo);
        demo.setVisible(true);
    }

}

[Jfreechart-commit] jcommon/source/org/jfree/ui DateChooserPanel.java,1.7,1.8

[David G. <mu...@us...>]

Update of /cvsroot/jfreechart/jcommon/source/org/jfree/ui
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv12333/source/org/jfree/ui

Modified Files:
	DateChooserPanel.java 
Log Message:
Fixed bug 1344319.

Index: DateChooserPanel.java
===================================================================
RCS file: /cvsroot/jfreechart/jcommon/source/org/jfree/ui/DateChooserPanel.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -C2 -d -r1.7 -r1.8
*** DateChooserPanel.java	18 Oct 2005 13:18:34 -0000	1.7
--- DateChooserPanel.java	2 Nov 2005 14:10:32 -0000	1.8
***************
*** 40,43 ****
--- 40,46 ----
   * 08-Dec-2001 : Dropped the getMonths() method (DG);
   * 13-Oct-2002 : Fixed errors reported by Checkstyle (DG);
+  * 02-Nov-2005 : Fixed a bug where the current day-of-the-month is past
+  *               the end of the newly selected month when the month or year
+  *               combo boxes are changed - see bug id 1344319 (DG);
   *
   */
***************
*** 129,133 ****
  
      /**
!      * A flag that indicates whether or not we are currently refreshing the buttons.
       */
      private boolean refreshing = false;
--- 132,137 ----
  
      /**
!      * A flag that indicates whether or not we are currently refreshing the 
!      * buttons.
       */
      private boolean refreshing = false;
***************
*** 140,144 ****
  
      /**
!      * Constructs a new date chooser panel, using today's date as the initial selection.
       */
      public DateChooserPanel() {
--- 144,149 ----
  
      /**
!      * Constructs a new date chooser panel, using today's date as the initial 
!      * selection.
       */
      public DateChooserPanel() {
***************
*** 150,157 ****
       *
       * @param calendar     the calendar controlling the date.
!      * @param controlPanel a flag that indicates whether or not the 'today' button should
!      *                     appear on the panel.
       */
!     public DateChooserPanel(final Calendar calendar, final boolean controlPanel) {
  
          super(new BorderLayout());
--- 155,163 ----
       *
       * @param calendar     the calendar controlling the date.
!      * @param controlPanel a flag that indicates whether or not the 'today' 
!      *                     button should appear on the panel.
       */
!     public DateChooserPanel(final Calendar calendar, 
!                             final boolean controlPanel) {
  
          super(new BorderLayout());
***************
*** 185,189 ****
  
          this.chosenDate.setTime(theDate);
!         this.monthSelector.setSelectedIndex(this.chosenDate.get(Calendar.MONTH));
          refreshYearSelector();
          refreshButtons();
--- 191,196 ----
  
          this.chosenDate.setTime(theDate);
!         this.monthSelector.setSelectedIndex(this.chosenDate.get(
!                 Calendar.MONTH));
          refreshYearSelector();
          refreshButtons();
***************
*** 209,213 ****
--- 216,231 ----
          if (e.getActionCommand().equals("monthSelectionChanged")) {
              final JComboBox c = (JComboBox) e.getSource();
+             
+             // In most cases, changing the month will not change the selected
+             // day.  But if the selected day is 29, 30 or 31 and the newly
+             // selected month doesn't have that many days, we revert to the 
+             // last day of the newly selected month...
+             int dayOfMonth = this.chosenDate.get(Calendar.DAY_OF_MONTH);
+             this.chosenDate.set(Calendar.DAY_OF_MONTH, 1);
              this.chosenDate.set(Calendar.MONTH, c.getSelectedIndex());
+             int maxDayOfMonth = this.chosenDate.getActualMaximum(
+                     Calendar.DAY_OF_MONTH);
+             this.chosenDate.set(Calendar.DAY_OF_MONTH, Math.min(dayOfMonth, 
+                     maxDayOfMonth));
              refreshButtons();
          }
***************
*** 216,220 ****
--- 234,249 ----
                  final JComboBox c = (JComboBox) e.getSource();
                  final Integer y = (Integer) c.getSelectedItem();
+                 
+                 // in most cases, changing the year will not change the 
+                 // selected day.  But if the selected day is Feb 29, and the
+                 // newly selected year is not a leap year, we revert to 
+                 // Feb 28...
+                 int dayOfMonth = this.chosenDate.get(Calendar.DAY_OF_MONTH);
+                 this.chosenDate.set(Calendar.DAY_OF_MONTH, 1);
                  this.chosenDate.set(Calendar.YEAR, y.intValue());
+                 int maxDayOfMonth = this.chosenDate.getActualMaximum(
+                     Calendar.DAY_OF_MONTH);
+                 this.chosenDate.set(Calendar.DAY_OF_MONTH, Math.min(dayOfMonth, 
+                     maxDayOfMonth));
                  refreshYearSelector();
                  refreshButtons();
***************
*** 234,239 ****
  
      /**
!      * Returns a panel of buttons, each button representing a day in the month.  This is a
!      * sub-component of the DatePanel.
       *
       * @return the panel.
--- 263,268 ----
  
      /**
!      * Returns a panel of buttons, each button representing a day in the month.
!      * This is a sub-component of the DatePanel.
       *
       * @return the panel.
***************
*** 246,250 ****
  
          for (int i = 0; i < this.WEEK_DAYS.length; i++) {
!             p.add(new JLabel(weekDays[this.WEEK_DAYS[i]], SwingConstants.CENTER));
          }
  
--- 275,280 ----
  
          for (int i = 0; i < this.WEEK_DAYS.length; i++) {
!             p.add(new JLabel(weekDays[this.WEEK_DAYS[i]], 
!                     SwingConstants.CENTER));
          }
  
***************
*** 275,279 ****
              return this.chosenDateButtonColor;
          }
!         else if (theDate.get(Calendar.MONTH) == this.chosenDate.get(Calendar.MONTH)) {
              return this.chosenMonthButtonColor;
          }
--- 305,310 ----
              return this.chosenDateButtonColor;
          }
!         else if (theDate.get(Calendar.MONTH) == this.chosenDate.get(
!                 Calendar.MONTH)) {
              return this.chosenMonthButtonColor;
          }
***************
*** 302,307 ****
  
      /**
!      * Returns the first date that is visible in the grid.  This should always be in the month
!      * preceding the month of the selected date.
       *
       * @return the date.
--- 333,338 ----
  
      /**
!      * Returns the first date that is visible in the grid.  This should always 
!      * be in the month preceding the month of the selected date.
       *
       * @return the date.
***************
*** 309,313 ****
      private Calendar getFirstVisibleDate() {
          final Calendar c = Calendar.getInstance();
!         c.set(this.chosenDate.get(Calendar.YEAR), this.chosenDate.get(Calendar.MONTH), 1);
          c.add(Calendar.DATE, -1);
          while (c.get(Calendar.DAY_OF_WEEK) != getFirstDayOfWeek()) {
--- 340,345 ----
      private Calendar getFirstVisibleDate() {
          final Calendar c = Calendar.getInstance();
!         c.set(this.chosenDate.get(Calendar.YEAR), this.chosenDate.get(
!                 Calendar.MONTH), 1);
          c.add(Calendar.DATE, -1);
          while (c.get(Calendar.DAY_OF_WEEK) != getFirstDayOfWeek()) {
***************
*** 318,322 ****
  
      /**
!      * Returns the first day of the week (controls the labels in the date panel).
       *
       * @return the first day of the week.
--- 350,355 ----
  
      /**
!      * Returns the first day of the week (controls the labels in the date 
!      * panel).
       *
       * @return the first day of the week.
***************
*** 340,345 ****
  
      /**
!      * Changes the contents of the year selection JComboBox to reflect the chosen date and the
!      * year range.
       */
      private void refreshYearSelector() {
--- 373,378 ----
  
      /**
!      * Changes the contents of the year selection JComboBox to reflect the 
!      * chosen date and the year range.
       */
      private void refreshYearSelector() {
***************
*** 347,355 ****
              this.refreshing = true;
              this.yearSelector.removeAllItems();
!             final Integer[] years = getYears(this.chosenDate.get(Calendar.YEAR));
              for (int i = 0; i < years.length; i++) {
                  this.yearSelector.addItem(years[i]);
              }
!             this.yearSelector.setSelectedItem(new Integer(this.chosenDate.get(Calendar.YEAR)));
              this.refreshing = false;
          }
--- 380,390 ----
              this.refreshing = true;
              this.yearSelector.removeAllItems();
!             final Integer[] years = getYears(this.chosenDate.get(
!                     Calendar.YEAR));
              for (int i = 0; i < years.length; i++) {
                  this.yearSelector.addItem(years[i]);
              }
!             this.yearSelector.setSelectedItem(new Integer(this.chosenDate.get(
!                     Calendar.YEAR)));
              this.refreshing = false;
          }
***************
*** 357,362 ****
  
      /**
!      * Returns a vector of years preceding and following the specified year.  The number of
!      * years preceding and following is determined by the yearSelectionRange attribute.
       *
       * @param chosenYear the selected year.
--- 392,398 ----
  
      /**
!      * Returns a vector of years preceding and following the specified year.  
!      * The number of years preceding and following is determined by the 
!      * yearSelectionRange attribute.
       *
       * @param chosenYear the selected year.
***************
*** 375,380 ****
  
      /**
!      * Constructs a panel containing two JComboBoxes (for the month and year) and a button (to
!      * reset the date to TODAY).
       *
       * @return the panel.
--- 411,416 ----
  
      /**
!      * Constructs a panel containing two JComboBoxes (for the month and year) 
!      * and a button (to reset the date to TODAY).
       *
       * @return the panel.
***************
*** 386,390 ****
          final int maxMonth = this.chosenDate.getMaximum(Calendar.MONTH);
          final String[] months = new String[maxMonth - minMonth + 1];
!         System.arraycopy(SerialDate.getMonths(), minMonth, months, 0, months.length);
  
          this.monthSelector = new JComboBox(months);
--- 422,427 ----
          final int maxMonth = this.chosenDate.getMaximum(Calendar.MONTH);
          final String[] months = new String[maxMonth - minMonth + 1];
!         System.arraycopy(SerialDate.getMonths(), minMonth, months, 0, 
!                 months.length);
  
          this.monthSelector = new JComboBox(months);
***************
*** 402,407 ****
  
      /**
!      * Returns a panel that appears at the bottom of the calendar panel - contains a button for
!      * selecting today's date.
       *
       * @return the panel.
--- 439,444 ----
  
      /**
!      * Returns a panel that appears at the bottom of the calendar panel - 
!      * contains a button for selecting today's date.
       *
       * @return the panel.
***************
*** 440,444 ****
          this.chosenDateButtonColor = chosenDateButtonColor;
          refreshButtons();
!         firePropertyChange("chosenDateButtonColor", oldValue, chosenDateButtonColor);
      }
  
--- 477,482 ----
          this.chosenDateButtonColor = chosenDateButtonColor;
          refreshButtons();
!         firePropertyChange("chosenDateButtonColor", oldValue, 
!                 chosenDateButtonColor);
      }
  
***************
*** 464,468 ****
          this.chosenMonthButtonColor = chosenMonthButtonColor;
          refreshButtons();
!         firePropertyChange("chosenMonthButtonColor", oldValue, chosenMonthButtonColor);
      }
  
--- 502,507 ----
          this.chosenMonthButtonColor = chosenMonthButtonColor;
          refreshButtons();
!         firePropertyChange("chosenMonthButtonColor", oldValue, 
!                 chosenMonthButtonColor);
      }
  
***************
*** 488,492 ****
          this.chosenOtherButtonColor = chosenOtherButtonColor;
          refreshButtons();
!         firePropertyChange("chosenOtherButtonColor", oldValue, chosenOtherButtonColor);
      }
  
--- 527,532 ----
          this.chosenOtherButtonColor = chosenOtherButtonColor;
          refreshButtons();
!         firePropertyChange("chosenOtherButtonColor", oldValue, 
!                 chosenOtherButtonColor);
      }
  
***************
*** 509,513 ****
          this.yearSelectionRange = yearSelectionRange;
          refreshYearSelector();
!         firePropertyChange("yearSelectionRange", oldYearSelectionRange, yearSelectionRange);
      }
  }
\ No newline at end of file
--- 549,554 ----
          this.yearSelectionRange = yearSelectionRange;
          refreshYearSelector();
!         firePropertyChange("yearSelectionRange", oldYearSelectionRange, 
!                 yearSelectionRange);
      }
  }
\ No newline at end of file