Opened 10 years ago

Closed 10 years ago

#45 closed task (fixed)

review: src/lib/exceptions

Reported by: jreed Owned by: jinmei
Priority: very high Milestone: 02. Running, functional authoritative-only server
Component: Unclassified Version:
Keywords: review Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: Add Hours to Ticket:
Total Hours: Internal?:

Description

Review what is in trunk as of revision 738 following code review procedures. If this is your code, please add notes here as appropriate. I initially assigned this to the code owner, but this ticket will be assigned to a reviewer. If there is already code review in process, please refer to the other ticket number.

Subtickets

Change History (1)

comment:1 Changed 10 years ago by jinmei

  • Resolution set to fixed
  • Status changed from new to closed

At Fri, 05 Feb 2010 18:22:07 -0000,
"BIND 10 Development" <bind10-staff@…> wrote:

#45: review: src/lib/exceptions

Review what is in trunk as of revision 738 following code review
procedures. If this is your code, please add notes here as appropriate. I
initially assigned this to the code owner, but this ticket will be
assigned to a reviewer. If there is already code review in process, please
refer to the other ticket number.

I believe we don't need additional review for the code itself (this
was simply moved from src/lib/dns, and the original code was already
reviewed), but I'm not sure if "lib/exceptions" is a good (sub-)module
name. I expect there are some other small pieces we may want to share
(e.g. base64 encoder and decoder), and wonder we may want to have the
next generation version of the "infamous" libisc. Maybe something
like lib/util/?

In any case, I'm going to close this particular ticket.

Note: See TracTickets for help on using tickets.