QDox
  1. QDox
  2. QDOX-47

Remove DocletTag.setContext() interface

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.5
    • Component/s: Java API
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Tags are getting part of their context data via the factory (line-number), and part thru setContext(). This smells like a fish milkshake.

      Let's remove setContext() from the DocletTag interface, and provide all location info via the factory.

      public interface DocletTagFactory extends Serializable

      { DocletTag createDocletTag(String tag, String text); DocletTag createDocletTag(String tag, String text, AbstractJavaEntity context, int lineNumber); }

      See http://lists.codehaus.org/pipermail/qdox-user/2004-March/000549.html

        Activity

        Hide
        Mike Williams added a comment -

        Done pretty much as planned.

        Show
        Mike Williams added a comment - Done pretty much as planned.
        Mike Williams made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Mike Williams logged work - 17/May/04 9:15 AM
        • Time Spent:
          3 hours
           
          <No comment>
        Mike Williams made changes -
        Remaining Estimate 1 day [ 86400 ] 0 minutes [ 0 ]
        Time Spent 3 hours [ 10800 ]

          People

          • Assignee:
            Mike Williams
            Reporter:
            Mike Williams
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1 day
              1d
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 3 hours Time Not Required
              3h