Skip to content

Address many “TODO” comments (and friends) in checklink #69

Description

@Zearin

Below is a list of the various comments with the TODO, FIXME, XXX, or (sometimes) @@@ markers in checklink.

Specifics of how I got this list

I generated the list with the following command:

grep --ignore-case --line-number --extended-regexp  '#\s*(@@@|todo|fixme|xxx|@@@)'  ./bin/checklink

There are a few results that may not be intended as “todo” items, but were nevertheless marked with a matching pattern. I left these results in, just in case people who know more than I do might interpret/scrutinize them differently.

I added links to the line numbers for convenience.

List of TODO (and friends) comments

  • 128: # @@@ Why not just $self->SUPER::simple_request?
  • 146: # @@@ TODO: when an LWP internal robots.txt request gets redirected,
  • 233: # @@@ Needs also W3C::UserAgent but can't use() it here.
  • 470: my $ua = W3C::UserAgent->new($AGENT); # @@@ TODO: admin address
  • 631: # @@@ Ignore invalid depth silently for now.
  • 1794: # @@@ subtract robot delay from the "fetched in" time?
  • 2074: # @@@@ In XHTML, <a name="foo" id="foo"> is mandatory
  • 2118: # @@@TODO: The reason for handling <base href> ourselves is that LWP's
  • 2148: # TODO: HTML 4 spec says applet/@codebase may only point to
  • 2267: # FIXME: are there CSS rules with URLs that aren't mixed content blockable?
  • 2326: # TODO: Check the rules defined in
  • 2402: # TODO: find content types for which fragment are defined
  • 2413: # @@@TODO: this isn't the best thing to do if a decode error occurred
  • 2431: # TODO #top is OK in HTML documents
  • 3343: { # XXX no easy way to check if cookie expired?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions