Opened 10 years ago

Closed 8 years ago

#17 closed task (complete)

open issues of the name class

Reported by: jinmei Owned by: UnAssigned
Priority: low Milestone:
Component: libdns++ Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket:
Estimated Difficulty: 0.0 Add Hours to Ticket:
Total Hours: Internal?: no

Description

Imported from Ticket #16 (http://bind10.isc.org/ticket/16)

This seems to be a preference issue, but noted here so that we can
revisit it later.

  • Additional methods add operator+ for Name objects (concatenate)? Name::append(const Name& suffix)?

Will revisit these as we complete other part of the package. I'm
personally negative about adding more overloaded operators, but if
there's a good reason for having more I don't object to that.

  • Performance benchmark for example, to see if we want to minimize the initial offset vector capacity in the from-wire constructor

Subtickets

Change History (3)

comment:1 Changed 10 years ago by shane

  • Type changed from defect to task

comment:2 Changed 9 years ago by jinmei

  • billable set to 0
  • Internal? unset
  • Owner changed from jinmei to UnAssigned
  • Status changed from new to assigned

comment:3 Changed 8 years ago by shane

  • Defect Severity set to N/A
  • Resolution set to complete
  • Status changed from assigned to closed
  • Sub-Project set to DNS

I am going to say:

  • The current naming is just fine
  • Operator overloading is evil
  • Optimization is being handled in other contexts right now

And close the ticket.

If anyone wants to re-open, that is okay, or perhaps better to create a new ticket or two with only a single issue in each.

Note: See TracTickets for help on using tickets.