BIND 10 #749: Create C++ utility library
BIND 10 Development
do-not-reply at isc.org
Thu Apr 14 07:26:12 UTC 2011
#749: Create C++ utility library
-------------------------------------+-------------------------------------
Reporter: stephen | Owner: jelte
Type: | Status: reviewing
enhancement | Milestone:
Priority: minor | Sprint-20110419
Component: | Resolution:
Unclassified | Sensitive: 0
Keywords: | Add Hours to Ticket: 0
Estimated Number of Hours: 4.0 | Total Hours: 0
Billable?: 1 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Changes (by zzchen_pku):
* owner: zzchen_pku => jelte
Comment:
Replying to [comment:11 jelte]:
> The information in the README in util/ is now untrue...
Removed, may be we need a new readme file later.
> I'm not sure I consider buffer.h specifically io-related, and personally
I would have put it in util/ directly, one the same level as
strutil.[cc|h] (in fact, I also wonder if those readUint and writeUint
functions should be considered io)
Moved them to util/ directory.
> I also wouldn't consider sha1 and baseXXX to be 'compat', but even if we
do, it seems sha1 does not have namespace directives.
Renamed to 'coder', is it make sense to you, or you have a better
suggestion?
> Namespace of util/locks.h is also inconsistent with the rest (having
that extra namespace locks is probably fine, but it should probably fall
under isc/util as well)
>
> Same goes for strutil.h
Done. Thanks.
--
Ticket URL: <http://bind10.isc.org/ticket/749#comment:12>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list