2
0
mirror of https://gitlab.isc.org/isc-projects/bind9 synced 2025-09-01 06:55:30 +00:00

code review

This commit is contained in:
Brian Wellington
2000-03-16 00:54:24 +00:00
parent f72fbb5d0b
commit d0fe07af9c

View File

@@ -15,9 +15,11 @@
* SOFTWARE. * SOFTWARE.
*/ */
/* $Id: isdn_20.c,v 1.12 2000/02/03 23:42:59 halley Exp $ */ /* $Id: isdn_20.c,v 1.13 2000/03/16 00:54:24 bwelling Exp $ */
/* RFC 1183 */ /* Reviewed: Wed Mar 15 16:53:11 PST 2000 by bwelling */
/* RFC 1183 */
#ifndef RDATA_GENERIC_ISDN_20_C #ifndef RDATA_GENERIC_ISDN_20_C
#define RDATA_GENERIC_ISDN_20_C #define RDATA_GENERIC_ISDN_20_C
@@ -29,12 +31,12 @@ fromtext_isdn(dns_rdataclass_t rdclass, dns_rdatatype_t type,
{ {
isc_token_t token; isc_token_t token;
REQUIRE(type == 20); rdclass = rdclass; /*unused*/
rdclass = rdclass; /*unused*/
origin = origin; /*unused*/ origin = origin; /*unused*/
downcase = downcase; /*unused*/ downcase = downcase; /*unused*/
REQUIRE(type == 20);
/* ISDN-address */ /* ISDN-address */
RETERR(gettoken(lexer, &token, isc_tokentype_qstring, ISC_FALSE)); RETERR(gettoken(lexer, &token, isc_tokentype_qstring, ISC_FALSE));
RETERR(txt_fromtext(&token.value.as_textregion, target)); RETERR(txt_fromtext(&token.value.as_textregion, target));
@@ -55,10 +57,10 @@ totext_isdn(dns_rdata_t *rdata, dns_rdata_textctx_t *tctx,
{ {
isc_region_t region; isc_region_t region;
REQUIRE(rdata->type == 20);
tctx = tctx; /*unused*/ tctx = tctx; /*unused*/
REQUIRE(rdata->type == 20);
dns_rdata_toregion(rdata, &region); dns_rdata_toregion(rdata, &region);
RETERR(txt_totext(&region, target)); RETERR(txt_totext(&region, target));
if (region.length == 0) if (region.length == 0)
@@ -72,13 +74,12 @@ fromwire_isdn(dns_rdataclass_t rdclass, dns_rdatatype_t type,
isc_buffer_t *source, dns_decompress_t *dctx, isc_buffer_t *source, dns_decompress_t *dctx,
isc_boolean_t downcase, isc_buffer_t *target) isc_boolean_t downcase, isc_buffer_t *target)
{ {
dctx = dctx; /* unused */
rdclass = rdclass; /* unused */
downcase = downcase; /* unused */
REQUIRE(type == 20); REQUIRE(type == 20);
dctx = dctx; /* unused */
rdclass = rdclass; /* unused */
downcase = downcase; /* unused */
RETERR(txt_fromwire(source, target)); RETERR(txt_fromwire(source, target));
if (buffer_empty(source)) if (buffer_empty(source))
return (DNS_R_SUCCESS); return (DNS_R_SUCCESS);
@@ -87,11 +88,10 @@ fromwire_isdn(dns_rdataclass_t rdclass, dns_rdatatype_t type,
static inline isc_result_t static inline isc_result_t
towire_isdn(dns_rdata_t *rdata, dns_compress_t *cctx, isc_buffer_t *target) { towire_isdn(dns_rdata_t *rdata, dns_compress_t *cctx, isc_buffer_t *target) {
cctx = cctx; /*unused*/
REQUIRE(rdata->type == 20); REQUIRE(rdata->type == 20);
cctx = cctx; /*unused*/
return (mem_tobuffer(target, rdata->data, rdata->length)); return (mem_tobuffer(target, rdata->data, rdata->length));
} }
@@ -113,25 +113,22 @@ static inline isc_result_t
fromstruct_isdn(dns_rdataclass_t rdclass, dns_rdatatype_t type, void *source, fromstruct_isdn(dns_rdataclass_t rdclass, dns_rdatatype_t type, void *source,
isc_buffer_t *target) isc_buffer_t *target)
{ {
rdclass = rdclass; /*unused*/
source = source; /*unused*/
target = target; /*unused*/
REQUIRE(type == 20); REQUIRE(type == 20);
rdclass = rdclass; /*unused*/
source = source;
target = target;
return (DNS_R_NOTIMPLEMENTED); return (DNS_R_NOTIMPLEMENTED);
} }
static inline isc_result_t static inline isc_result_t
tostruct_isdn(dns_rdata_t *rdata, void *target, isc_mem_t *mctx) { tostruct_isdn(dns_rdata_t *rdata, void *target, isc_mem_t *mctx) {
target = target; /*unused*/
mctx = mctx; /*unused*/
REQUIRE(rdata->type == 20); REQUIRE(rdata->type == 20);
target = target;
mctx = mctx;
return (DNS_R_NOTIMPLEMENTED); return (DNS_R_NOTIMPLEMENTED);
} }
@@ -145,10 +142,10 @@ static inline isc_result_t
additionaldata_isdn(dns_rdata_t *rdata, dns_additionaldatafunc_t add, additionaldata_isdn(dns_rdata_t *rdata, dns_additionaldatafunc_t add,
void *arg) void *arg)
{ {
REQUIRE(rdata->type == 20); add = add; /*unused*/
arg = arg; /*unused*/
(void)add; REQUIRE(rdata->type == 20);
(void)arg;
return (DNS_R_SUCCESS); return (DNS_R_SUCCESS);
} }