QDox

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.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

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