Groovy
  1. Groovy
  2. GROOVY-2524

Javadoc typos for DefaultGroovyMethods

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.5.1
    • Fix Version/s: 1.6-rc-2
    • Component/s: groovy-jdk
    • Labels:
      None
    • Flags:
      Patch

      Description

      Looking at the javadoc for the Groovy enhanced java.lang.Object there are some typos.

      From the dist docs 1.5.1 open the docs (/docs/html/groovy-jdk/index.html) and click on java.lang.Object.
      It list all the new methods we have (they are fab BTW)

      However there are a few typos and some JDK1.5 printf methods that are not in the summary.
      I have attached a patch for it. It was from the SVN from the trunk.

        Activity

        Claus Ibsen created issue -
        Hide
        Paul King added a comment -

        Added thanks. I didn't apply the two 'i.e. -> ie' changes. It is probably an improvement but in some parts of the world perhaps the traditional 'i.e.' is still preferred. There were 52 usages of 'i.e.' in the codebase (in comments) and one usage of 'ie' - now changed to 'i.e.' for consistency. So I would prefer to change them all at once if we are to change. For future note, there were 84 usages of 'e.g.' and none of 'eg'.

        Show
        Paul King added a comment - Added thanks. I didn't apply the two 'i.e. -> ie' changes. It is probably an improvement but in some parts of the world perhaps the traditional 'i.e.' is still preferred. There were 52 usages of 'i.e.' in the codebase (in comments) and one usage of 'ie' - now changed to 'i.e.' for consistency. So I would prefer to change them all at once if we are to change. For future note, there were 84 usages of 'e.g.' and none of 'eg'.
        Paul King made changes -
        Field Original Value New Value
        Assignee Paul King [ paulk ]
        Fix Version/s 1.6 [ 13832 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Claus Ibsen added a comment -

        Paul the i.e. gives a problem in the javadoc summary where it would cut the sentence short.
        We should use encoded for dots (cant remember the syntax) when doing i.e. or e.g.

        People forget this often, and it would be easier to just use without the dots (eg or ie)

        Show
        Claus Ibsen added a comment - Paul the i.e. gives a problem in the javadoc summary where it would cut the sentence short. We should use encoded for dots (cant remember the syntax) when doing i.e. or e.g. People forget this often, and it would be easier to just use without the dots (eg or ie)
        Hide
        Paul King added a comment -

        OK, the lightbulb finally turns on. I'll reopen until we fix this.

        Show
        Paul King added a comment - OK, the lightbulb finally turns on. I'll reopen until we fix this.
        Paul King made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Paul King added a comment - - edited

        OK, now all instances of 'e.g.' and 'i.e.' are immediately followed by ' ' if used as the first sentence of javadoc. We can always do the 'eg/ie' swap later if this issue crops up again.

        Show
        Paul King added a comment - - edited OK, now all instances of 'e.g.' and 'i.e.' are immediately followed by ' ' if used as the first sentence of javadoc. We can always do the 'eg/ie' swap later if this issue crops up again.
        Paul King made changes -
        Resolution Fixed [ 1 ]
        Status Reopened [ 4 ] Resolved [ 5 ]
        Hide
        Paul King added a comment -

        close off release 1.5.4

        Show
        Paul King added a comment - close off release 1.5.4
        Paul King made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
        Mark Thomas made changes -
        Workflow jira [ 12731574 ] Default workflow, editable Closed status [ 12743482 ]
        Mark Thomas made changes -
        Flags Patch [ 10430 ]
        Patch Submitted Yes [ 10763 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
        Mark Thomas made changes -
        Workflow jira [ 12967510 ] Default workflow, editable Closed status [ 12975212 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        19h 9m 1 Paul King 21/Jan/08 01:26
        Resolved Resolved Reopened Reopened
        3h 13m 1 Paul King 21/Jan/08 04:39
        Reopened Reopened Resolved Resolved
        24m 16s 1 Paul King 21/Jan/08 05:03
        Resolved Resolved Closed Closed
        17d 20h 22m 1 Paul King 08/Feb/08 01:26

          People

          • Assignee:
            Paul King
            Reporter:
            Claus Ibsen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development