diff options
author | YASUOKA Masahiko <yasuoka@cvs.openbsd.org> | 2010-07-01 03:38:18 +0000 |
---|---|---|
committer | YASUOKA Masahiko <yasuoka@cvs.openbsd.org> | 2010-07-01 03:38:18 +0000 |
commit | ef76f436407d7a730766f3dcbf3c8c25b0369a45 (patch) | |
tree | 4b2224173d86227e92e98cbcb374e5d9d3779c1c /usr.sbin/npppd/common | |
parent | 797ad9d0edde762ee142aa0ea7008cf766c8f357 (diff) |
Translate Japanese comments or labels into English. Translation was
done by IIJ people (MATSUI Yoshihiro, SAITOH Masanobu, Tomoyuki Sahara),
yuo@ and myself.
This diff also includes
- delete part of useless comments, correct spelling.
- add man page of npppdctl.
There is no functional change.
Diffstat (limited to 'usr.sbin/npppd/common')
-rw-r--r-- | usr.sbin/npppd/common/addr_range.c | 3 | ||||
-rw-r--r-- | usr.sbin/npppd/common/addr_range.h | 1 | ||||
-rw-r--r-- | usr.sbin/npppd/common/bytebuf.c | 7 | ||||
-rw-r--r-- | usr.sbin/npppd/common/bytebuf.h | 3 | ||||
-rw-r--r-- | usr.sbin/npppd/common/config_helper.c | 123 | ||||
-rw-r--r-- | usr.sbin/npppd/common/config_helper.h | 1 | ||||
-rw-r--r-- | usr.sbin/npppd/common/csvreader.c | 59 | ||||
-rw-r--r-- | usr.sbin/npppd/common/csvreader.h | 18 | ||||
-rw-r--r-- | usr.sbin/npppd/common/csvreader_test.c | 7 | ||||
-rw-r--r-- | usr.sbin/npppd/common/ipsec_util.c | 14 | ||||
-rw-r--r-- | usr.sbin/npppd/common/net_utils.c | 23 | ||||
-rw-r--r-- | usr.sbin/npppd/common/properties.c | 8 | ||||
-rw-r--r-- | usr.sbin/npppd/common/properties_test.c | 10 | ||||
-rw-r--r-- | usr.sbin/npppd/common/rt_zebra.c | 10 | ||||
-rw-r--r-- | usr.sbin/npppd/common/rtev_common.c | 6 | ||||
-rw-r--r-- | usr.sbin/npppd/common/rtev_libevent.c | 12 | ||||
-rw-r--r-- | usr.sbin/npppd/common/slist.c | 126 | ||||
-rw-r--r-- | usr.sbin/npppd/common/slist.h | 2 | ||||
-rw-r--r-- | usr.sbin/npppd/common/slist_test.c | 103 |
19 files changed, 270 insertions, 266 deletions
diff --git a/usr.sbin/npppd/common/addr_range.c b/usr.sbin/npppd/common/addr_range.c index f4df149fe45..fd1637a148b 100644 --- a/usr.sbin/npppd/common/addr_range.c +++ b/usr.sbin/npppd/common/addr_range.c @@ -1,3 +1,4 @@ +/* $OpenBSD: addr_range.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -55,7 +56,7 @@ * Author: * Yasuoka Masahiko <yasuoka@iij.ad.jp> * - * $Id: addr_range.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ + * $Id: addr_range.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #ifdef ADDR_RANGE_DEBUG #define IIJDEBUG diff --git a/usr.sbin/npppd/common/addr_range.h b/usr.sbin/npppd/common/addr_range.h index 7a6745dbd8d..9ebb6bd93ed 100644 --- a/usr.sbin/npppd/common/addr_range.h +++ b/usr.sbin/npppd/common/addr_range.h @@ -1,3 +1,4 @@ +/* $OpenBSD: addr_range.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. diff --git a/usr.sbin/npppd/common/bytebuf.c b/usr.sbin/npppd/common/bytebuf.c index 2f8c91ec96b..66359e49f05 100644 --- a/usr.sbin/npppd/common/bytebuf.c +++ b/usr.sbin/npppd/common/bytebuf.c @@ -1,3 +1,4 @@ +/* $OpenBSD: bytebuf.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -40,7 +41,7 @@ * }</pre> * * @author Yasuoka Masahiko - * $Id: bytebuf.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ + * $Id: bytebuf.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <stdlib.h> #include <string.h> @@ -102,7 +103,7 @@ bytebuffer_create(size_t capacity) if (capacity > 0) { if ((_this->data = malloc(capacity)) == NULL) - goto reigai; + goto fail; memset(_this->data, 0, capacity); _this->capacity = capacity; } else @@ -111,7 +112,7 @@ bytebuffer_create(size_t capacity) _this->limit = _this->capacity; _this->mark = -1; return _this; -reigai: +fail: if (_this != NULL) free(_this); return NULL; diff --git a/usr.sbin/npppd/common/bytebuf.h b/usr.sbin/npppd/common/bytebuf.h index c0ced991426..7127ac70c6a 100644 --- a/usr.sbin/npppd/common/bytebuf.h +++ b/usr.sbin/npppd/common/bytebuf.h @@ -1,3 +1,4 @@ +/* $OpenBSD: bytebuf.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -26,7 +27,7 @@ #ifndef BYTEBUF_H #define BYTEBUF_H 1 -/* $Id: bytebuf.h,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: bytebuf.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ typedef struct _bytebuffer bytebuffer; diff --git a/usr.sbin/npppd/common/config_helper.c b/usr.sbin/npppd/common/config_helper.c index 5fbf057ea9c..a6148e2885b 100644 --- a/usr.sbin/npppd/common/config_helper.c +++ b/usr.sbin/npppd/common/config_helper.c @@ -1,3 +1,4 @@ +/* $OpenBSD: config_helper.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -23,11 +24,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -/* $Id: config_helper.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ -/**@file コンフィグヘルパ。 - * <p> - * しています。</p> - */ +/* $Id: config_helper.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <sys/types.h> #include <stdio.h> #include <string.h> @@ -40,9 +37,9 @@ #define KEYBUFSZ 512 /** - * コンフィグキーを作成するための文字列連結 - * (("prefix", "suffix") => "prefix.suffix") を、行います。内部で固定のバッファ - * 領域を返します。 + * This function concatenates given prefix and the give suffix for making + * configuration key (("prefix", "suffix") => "prefix.suffix"). The string + * returned by this function resides in a static memory area. */ const char * config_key_prefix(const char *prefix, const char *suffix) @@ -57,11 +54,12 @@ config_key_prefix(const char *prefix, const char *suffix) } /** - * 設定を文字列で返します。 - * - * @param _this {@link ::properties}へのポインタ。 - * @param confKey 設定ファイルの設定項目名 - * @return 設定値。設定が存在しない場合には NULL が返ります。 + * Retrieve the configuration value as a 'string' that is specified by + * given configuration key. + * @param _this The pointer to {@link ::properties} + * @param confKey configuration key name. + * @return pointer to the configuration value. If no configuration value + * exists then it returns NULL. */ const char * config_str(struct properties *_this, const char *confKey) @@ -72,11 +70,12 @@ config_str(struct properties *_this, const char *confKey) } /** - * 設定を int で返します。 - * - * @param _this {@link ::properties}へのポインタ。 - * @param confKey 設定ファイルの設定項目名 - * @param defValue 設定が省略されている場合のデフォルトの値 + * Retrieve the configuration value as a 'int' that is specified by + * given configuration key. + * @param _this The pointer to {@link ::properties} + * @param confKey configuration key name. + * @param defValue The default value. This function will return this + * value in case no configuration exists */ int config_int(struct properties *_this, const char *confKey, int defValue) @@ -92,19 +91,19 @@ config_int(struct properties *_this, const char *confKey, int defValue) x = sscanf(val, "%d", &rval); if (x != 1) - /* 関数のインタフェースを変更して、エラーは区別すべきかも */ return defValue; return rval; } /** - * 設定があたえられた文字列と一致するかどうかを返します。 - * - * @param _this {@link ::properties}へのポインタ。 - * @param confKey 設定ファイルの設定項目名 - * @param defValue 設定が省略されている場合のデフォルトの値 - * @return 一致する場合には 1、一致しない場合には 0 が返ります。 + * Checks whether the configuration value equals given string. + * @param _this The pointer to {@link ::properties} + * @param confKey configuration key name. + * @param defValue The default value. This function will return this + * value in case no configuration exists + * @return return 1 if given string matches the configuration value, + * otherwise return 0. */ int config_str_equal(struct properties *_this, const char *confKey, @@ -121,13 +120,14 @@ config_str_equal(struct properties *_this, const char *confKey, } /** - * 設定があたえられた文字列と一致するかどうかを返します。ASCII 文字の - * 大文字小文字は無視します。 - * - * @param _this {@link ::properties}へのポインタ。 - * @param confKey 設定ファイルの設定項目名 - * @param defValue 設定が省略されている場合のデフォルトの値 - * @return 一致する場合には 1、一致しない場合には 0 が返ります。 + * Checks whether the configuration value equals given string ignoring + * case. + * @param _this The pointer to {@link ::properties} + * @param confKey configuration key name. + * @param defValue The default value. This function will return this + * value in case no configuration exists + * @return return 1 if given string equals the configuration value, + * otherwise return 0. */ int config_str_equali(struct properties *_this, const char *confKey, @@ -144,25 +144,29 @@ config_str_equali(struct properties *_this, const char *confKey, } /*********************************************************************** - * 設定項目名に指定したプレフィックスをつけて設定を取得し、設定がなければ - * プレフィックスなしの設定項目で設定を取得するための関数です。 + * Following functions are to get configuration value by given + * configuration key. At first the function will try to get the value + * by the key with the prefix, if it fails, then it will try to get the + * value by the key without the prefix. * - * たとえば - * - * pppoe.service_name: default_service - * PPPoE0.pppoe.service_name: my_service + * For example, we have following configuration * - * という設定があった場合、 - * config_prefixed_str(prop, "PPPoE0", "service_name") - * を呼び出すと "my_service" が取得できます。設定に、 + * pppoe.service_name: default_service + * PPPoE0.pppoe.service_name: my_service * - * PPPoE0.pppoe.service_name: my_service + * calling * - * がない場合には、"default_service" が取得できます。 + * config_prefixed_str(prop, "PPPoE0", "service_name") * - * config_helper.h に定義されている PREFIXED_CONFIG_FUNCTIONS マクロを - * 使って、プレフィックス部分の指定方法を固定して使うこともできます。 - ***********************************************************************/ + * returns "my_service". If + * + * PPPoE0.pppoe.service_name: my_service + * + * does not exist, then it returns "default_service". + * + * Functions that have fixed prefix can be generated by + * PREFIXED_CONFIG_FUNCTIONS macro that is defined in config_helper.h. + */ const char * config_prefixed_str(struct properties *_this, const char *prefix, const char *confKey) { @@ -234,24 +238,29 @@ config_prefixed_str_equali(struct properties *_this, const char *prefix, } /*********************************************************************** - * 設定項目名に指定したプレフィックスと指定した名前をつけて設定を取得し、 - * 設定がなければプレフィックスに設定項目で設定を取得するための関数です。 + * Following functions are to get configuration value by given + * configuration key. At first the function will try to get the value + * by the key with the prefix and given label, if it fail, then it will + * try to get the value by the key without the label. * - * たとえば + * For example, we have following configuration * - * ipcp.dns_primary: 192.168.0.1 - * ipcp.ipcp0.dns_primary: 192.168.0.2 + * ipcp.dns_primary: 192.168.0.1 + * ipcp.ipcp0.dns_primary: 192.168.0.2 + * + * calling * - * という設定があった場合、 * config_named_prefix_str(prop, "ipcp", "ipcp0", "dns_primary"); - * を呼び出すと "192.168.0.2" が取得できます。設定に、 * - * ipcp.ipcp0.dns_primary: 192.168.0.2 + * will returns "192.168.0.2". If + * + * ipcp.ipcp0.dns_primary: 192.168.0.2 * - * がない場合には、"192.168.0.1" が取得できます。 + * was not exists, then it returns "default_service". * - * config_helper.h に定義されている NAMED_PREFIX_CONFIG_FUNCTIONS マクロ - * を使って、プレフィックス部分の指定方法を固定して使うこともできます。 + * Functions that has fixed prefix can be generated by + * NAMED_PREFIXED_CONFIG_FUNCTIONS macro that is defined in + * config_helper.h. ***********************************************************************/ const char * config_named_prefix_str(struct properties *_this, const char *prefix, diff --git a/usr.sbin/npppd/common/config_helper.h b/usr.sbin/npppd/common/config_helper.h index 8f618727a9b..812e9763b60 100644 --- a/usr.sbin/npppd/common/config_helper.h +++ b/usr.sbin/npppd/common/config_helper.h @@ -1,3 +1,4 @@ +/* $OpenBSD: config_helper.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. diff --git a/usr.sbin/npppd/common/csvreader.c b/usr.sbin/npppd/common/csvreader.c index ce8a852018f..00023e51560 100644 --- a/usr.sbin/npppd/common/csvreader.c +++ b/usr.sbin/npppd/common/csvreader.c @@ -1,3 +1,4 @@ +/* $OpenBSD: csvreader.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -23,9 +24,9 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -/* original version は CSVReader.java */ +/* The original version is CSVReader.java */ /* @file - * CSV(RFC4180) を読み込むための補助的な関数 + * Subroutines to read CSV(RFC4180) * <pre> * csvreader *csv; * const char **cols; @@ -53,7 +54,7 @@ * csvreader_destroy(csv); *</pre> */ -/* $Id: csvreader.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: csvreader.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <unistd.h> #include <stdlib.h> #include <string.h> @@ -82,7 +83,7 @@ struct _csvreader { #define CSV_FLUSH_WAIT 4 #define DQUOTE '"' -#define COLON ',' +#define COMMA ',' #define CR '\r' #define LF '\n' @@ -91,8 +92,7 @@ static int csvreader_flush_column (csvreader *); static CSVREADER_STATUS csvreader_parse_flush0 (csvreader *, int); /** - * cvsreader コンテキストを作成して返却します。メモリ割り当てに失敗すると、 - * NULL が返ります。 + * Make a cvsreader context and returns it. Return null if malloc() failed. */ csvreader * csvreader_create(void) @@ -109,7 +109,7 @@ csvreader_create(void) /** - * cvsreader コンテキストを解放します。 + * Free a cvsreader context. */ void csvreader_destroy(csvreader *_this) @@ -122,7 +122,7 @@ csvreader_destroy(csvreader *_this) } /** - * パース済みのカラム配列の数を取得します。 + * get the number of parsed columns. */ int csvreader_get_number_of_column(csvreader *_this) @@ -133,7 +133,7 @@ csvreader_get_number_of_column(csvreader *_this) } /** - * パース済みのカラム配列(=行) を取得します。 + * get a parsed column. */ const char ** csvreader_get_column(csvreader *_this) @@ -150,7 +150,7 @@ csvreader_get_column(csvreader *_this) } /** - * cvsreader コンテキストをリセットします。 + * Reset a cvsreader context. */ void csvreader_reset(csvreader *_this) @@ -165,11 +165,11 @@ csvreader_reset(csvreader *_this) } /** - * パース途中のカラムのパースを完了します。 + * Finish parsing of a column on the way. * <p> - * CSVファイルの末尾など、続く行 - * が存在しない場合や、CSV としての行の区切りであることが確実な場合に呼び - * だします。フィールドが終了していない場合などにはエラーが返ります。</p> + * Call this function when it's sure that there is no next line or the end + * of the CSV. + * It will return error when the parsing of the field isn't finished. </p> */ CSVREADER_STATUS csvreader_parse_flush(csvreader *_this) @@ -178,10 +178,10 @@ csvreader_parse_flush(csvreader *_this) } /** - * 行をパースします。 + * parse a line. * - * @param csvreader コンテキスト - * @param パースする行。 + * @param a csvreader context + * @param a line to parse */ CSVREADER_STATUS csvreader_parse(csvreader *_this, const char *line) @@ -205,14 +205,14 @@ csvreader_parse(csvreader *_this, const char *line) _this->column_start_with_quote = 1; break; } - /* FALL THROUGH */ + /* FALLTHROUGH */ case CSV_IN_DATA: if (_this->column_start_with_quote != 0) { if (line[off] == DQUOTE) _this->state = CSV_HAS_DQUOTE; break; } - if (line[off] == COLON) { + if (line[off] == COMMA) { append = 0; csvreader_flush_column(_this); } @@ -227,12 +227,12 @@ csvreader_parse(csvreader *_this, const char *line) break; } _this->state = CSV_WAIT_DELIM; - /* FALL THROUGH */ + /* FALLTHROUGH */ case CSV_WAIT_DELIM: if (line[off] == CR || line[off] == LF) goto eol; append = 0; - if (line[off] != COLON) + if (line[off] != COMMA) return CSVREADER_PARSE_ERROR; csvreader_flush_column(_this); break; @@ -258,11 +258,11 @@ csvreader_parse_flush0(csvreader *_this, int is_final) /* wait next line */ return CSVREADER_NO_ERROR; } - /* FALL THROUGH */ + /* FALLTHROUGH */ case CSV_INIT: if (is_final && _this->col_pos == 0) return CSVREADER_NO_ERROR; - /* FALL THROUGH */ + /* FALLTHROUGH */ case CSV_HAS_DQUOTE: case CSV_WAIT_DELIM: csvreader_flush_column(_this); @@ -273,12 +273,13 @@ csvreader_parse_flush0(csvreader *_this, int is_final) } /** - * char ポインタの配列に格納された列を CSV の表現の行文字列に変換します。 - - * @param cols 変換する列。NULL 要素は、列の終わりとみなします。 - * @param ncols 列の数。 - * @param buffer 変換後の行文字列を書き込むスペース - * @param lbuffer 変換後の行文字列を書き込むスペースの大きさ。 + * Convert columns stored in char *[] to a CVS line string. + * + * @param cols columns to be converted. NULL means the end of the + * column. + * @param ncols number of columns + * @param buffer the output buffer to write a converted line. + * @param lbuffer the size of the output buffer. */ int csvreader_toline(const char **cols, int ncols, char *buffer, int lbuffer) diff --git a/usr.sbin/npppd/common/csvreader.h b/usr.sbin/npppd/common/csvreader.h index 8dd46f8c481..0d53c6aab90 100644 --- a/usr.sbin/npppd/common/csvreader.h +++ b/usr.sbin/npppd/common/csvreader.h @@ -1,3 +1,4 @@ +/* $OpenBSD: csvreader.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -25,22 +26,23 @@ */ #ifndef CSVREADER_H #define CSVREADER_H 1 -/* $Id: csvreader.h,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: csvreader.h,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ -/** cvsreader のステータスを表す */ +/** status of cvsreader */ typedef enum _CSVREADER_STATUS { - /** 正常に処理が完了した */ + /** complete without any error */ CSVREADER_NO_ERROR = 0, /** - * 処理中の列が存在する + * A processing column exists. * <p> - * csvreader_get_columns で取り出す前に、次の行をパースすると - * このエラーが発生します。</p> + * This error occurs when parsing next line before getting a + * new coloumn by csvreader_get_columns(). + * </p> */ CSVREADER_HAS_PENDING_COLUMN = 10001, - /** メモリー割り当てに失敗した */ + /** Failed in memory allocation */ CSVREADER_OUT_OF_MEMORY = 10002, - /** パースエラー */ + /** Parse Error */ CSVREADER_PARSE_ERROR = 10003 } CSVREADER_STATUS; diff --git a/usr.sbin/npppd/common/csvreader_test.c b/usr.sbin/npppd/common/csvreader_test.c index 865fcd73018..b5b329f3d00 100644 --- a/usr.sbin/npppd/common/csvreader_test.c +++ b/usr.sbin/npppd/common/csvreader_test.c @@ -1,3 +1,4 @@ +/* $OpenBSD: csvreader_test.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /*- * Copyright (c) 2009 Internet Initiative Japan Inc. * All rights reserved. @@ -26,7 +27,7 @@ /* * cc -o csvreader_test csvreader.c csvreader_test.c */ -/* $Id: csvreader_test.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: csvreader_test.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <stdio.h> #include <stdlib.h> #include <string.h> @@ -149,7 +150,7 @@ test02(void) column = csvreader_get_column(csv); ASSERT(column != NULL); ASSERT(strcmp(column[0], "hogehoge") == 0); - //printf("**%s**\n", column[1]); + /* printf("**%s**\n", column[1]); */ ASSERT(strcmp(column[1], "fuga\nfuga") == 0); ASSERT(column[2] == NULL); @@ -158,7 +159,7 @@ test02(void) column = csvreader_get_column(csv); ASSERT(column != NULL); ASSERT(strcmp(column[0], "hogehoge") == 0); - //printf("**%s**\n", column[1]); + /* printf("**%s**\n", column[1]); */ ASSERT(strcmp(column[1], "fuga\rfuga") == 0); ASSERT(column[2] == NULL); diff --git a/usr.sbin/npppd/common/ipsec_util.c b/usr.sbin/npppd/common/ipsec_util.c index 10c634f0250..fb3965b2545 100644 --- a/usr.sbin/npppd/common/ipsec_util.c +++ b/usr.sbin/npppd/common/ipsec_util.c @@ -2,8 +2,8 @@ * Copyright 2007, 2009 * Internet Initiative Japan Inc. All rights reserved. */ -/* $Id: ipsec_util.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ -/*@file IPsec 関連ユーティリティ */ +/* $Id: ipsec_util.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ +/*@file IPsec related utility functions */ /* * RFC 2367 PF_KEY Key Management API, Version 2 */ @@ -59,25 +59,25 @@ ipsec_util_purge_transport_sa(struct sockaddr *sock, struct sockaddr *peer, tv = KEYSOCK_RCVTIMEO; if (setsockopt(key_sock, SOL_SOCKET, SO_RCVTIMEO, &tv, sizeof(tv)) != 0) - goto reigai; + goto fail; del_in.is_valid = del_out.is_valid = 0; if (delete_prepare(key_sock, sock, peer, proto, &del_in, &del_out) != 0) - goto reigai; + goto fail; if (del_in.is_valid && (dir & IPSEC_UTIL_DIRECTION_IN) != 0) { if (send_sadb_delete(key_sock, &del_in)) - goto reigai; + goto fail; } if (del_out.is_valid && (dir & IPSEC_UTIL_DIRECTION_OUT) != 0) { if (send_sadb_delete(key_sock, &del_out)) - goto reigai; + goto fail; } close(key_sock); return 0; -reigai: +fail: close(key_sock); return -1; diff --git a/usr.sbin/npppd/common/net_utils.c b/usr.sbin/npppd/common/net_utils.c index 14e4cd22fee..76519143e13 100644 --- a/usr.sbin/npppd/common/net_utils.c +++ b/usr.sbin/npppd/common/net_utils.c @@ -23,7 +23,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -/* $Id: net_utils.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: net_utils.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <sys/types.h> #include <sys/socket.h> #include <netinet/in.h> @@ -39,7 +39,7 @@ #include "rtev.h" #endif -/** struct sockaddr から、インタフェース名を取得します */ +/** Get an interface name from sockaddr */ const char * get_ifname_by_sockaddr(struct sockaddr *sa, char *ifname) { @@ -51,7 +51,7 @@ get_ifname_by_sockaddr(struct sockaddr *sa, char *ifname) #endif ifname0 = NULL; - /* リニアサーチしかないなんて... */ + /* I want other way than linear search */ getifaddrs(&addr0); for (addr = addr0; ifname0 == NULL&& addr != NULL; addr = addr->ifa_next) { @@ -88,8 +88,8 @@ get_ifname_by_sockaddr(struct sockaddr *sa, char *ifname) return ifname0; } /** - * "192.168.160.1:1723/tcp" や "[::1]:1723/tcp" という文字列を、getaddrinfo(3) - * の引数の仕様に併せて実行する。現在は、"/tcp" の部分は無視します。 + * Cconvert argument like "192.168.160.1:1723/tcp" or "[::1]:1723/tcp" to + * match getaddrinfo(3)'s specification and pass them to getaddrinfo(3). */ int addrport_parse(const char *addrport, int proto, struct addrinfo **p_ai) @@ -114,7 +114,7 @@ addrport_parse(const char *addrport, int proto, struct addrinfo **p_ai) slash = strrchr(servp, '/'); if (slash != NULL) { /* - * "/tcp" などは無視する。 + * Ignore like "/tcp" */ *slash = '\0'; slash++; @@ -138,10 +138,11 @@ addrport_parse(const char *addrport, int proto, struct addrinfo **p_ai) } /** - * struct sockaddr から、 "192.168.160.1:1723" や "[::1]:1723" という文字列 - * を作成します。 - * @param buf 文字列を格納するバッファ - * @param lbuf 文字列を格納するバッファの長さ + * Make a string like "192.168.160.1:1723" or "[::1]:1723" from a struct + * sockaddr + * + * @param buf the buffer to be stored a string + * @param lbuf the length of the buf */ const char * addrport_tostring(struct sockaddr *sa, socklen_t salen, char *buf, int lbuf) @@ -171,7 +172,7 @@ addrport_tostring(struct sockaddr *sa, socklen_t salen, char *buf, int lbuf) return buf; } -/** IPv4 ネットマスクをプレフィックス長に変換します。ホストバイトオーダーで */ +/** Convert 32bit IPv4 netmask to the prefix length in host byte order */ int netmask2prefixlen(uint32_t mask) { diff --git a/usr.sbin/npppd/common/properties.c b/usr.sbin/npppd/common/properties.c index 56cdcd474f9..4f60a93b636 100644 --- a/usr.sbin/npppd/common/properties.c +++ b/usr.sbin/npppd/common/properties.c @@ -24,7 +24,7 @@ * SUCH DAMAGE. */ /* - * $Id: properties.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ + * $Id: properties.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /* LINTLIBRARY */ #include <sys/types.h> @@ -389,7 +389,7 @@ properties_load(struct properties *_this, FILE *fp) char *line = NULL, *line0, *line1; if ((line = (char *)malloc(linesz)) == NULL) - goto reigai; + goto fail; linecont = 0; while (fgets(buf0, sizeof(buf0), fp) != NULL) { @@ -407,7 +407,7 @@ properties_load(struct properties *_this, FILE *fp) while (lineoff + linelen + 128 > linesz) { if ((line1 = realloc(line, linesz * 2)) == NULL) - goto reigai; + goto fail; line = line1; linesz *= 2; } @@ -449,7 +449,7 @@ properties_load(struct properties *_this, FILE *fp) if (line != NULL) free(line); return 0; -reigai: +fail: if (line != NULL) free(line); diff --git a/usr.sbin/npppd/common/properties_test.c b/usr.sbin/npppd/common/properties_test.c index 4c31df6f91b..861394f243b 100644 --- a/usr.sbin/npppd/common/properties_test.c +++ b/usr.sbin/npppd/common/properties_test.c @@ -86,9 +86,8 @@ test0(void) } static void test1() { - // [IDGW-DEV 5246] 1024 bytes + /* 1024 bytes */ set_and_get("pptpd.listener_in", "PPTP 10.10.10.50 PPTP 100.100.100.100 PPTP 100.100.100.101 PPTP 100.100.100.102 PPTP 100.100.100.103 PPTP 100.100.100.104 PPTP 100.100.100.105 PPTP 100.100.100.106 PPTP 100.100.100.107 PPTP 100.100.100.108 PPTP 100.100.100.109 PPTP 100.100.100.110 PPTP 100.100.100.111 PPTP 100.100.100.112 PPTP 100.100.100.113 PPTP 100.100.100.114 PPTP 100.100.100.115 PPTP 100.100.100.116 PPTP 100.100.100.117 PPTP 100.100.100.118 PPTP 100.100.100.119 PPTP 100.100.100.120 PPTP 100.100.100.121 PPTP 100.100.100.122 PPTP 100.100.100.123 PPTP 100.100.100.124 PPTP 100.100.100.125 PPTP 100.100.100.126 PPTP 100.100.100.127 PPTP 100.100.100.128 PPTP 100.100.100.129 PPTP 100.100.100.130 PPTP 100.100.100.131 PPTP 100.100.100.132 PPTP 100.100.100.133 PPTP 100.100.100.134 PPTP 100.100.100.135 PPTP 100.100.100.136 PPTP 100.100.100.137 PPTP 100.100.100.138 PPTP 100.100.100.139 PPTP 100.100.100.140 PPTP 100.100.100.141 PPTP 100.100.100.142 PPTP 100.100.100.143 PPTP 100.100.100.144 PPTP 100.100.100.145 PPTP 100.100.100.146 PPTP 100.100.100.147 PPTP 100.100.100.148 PPTP 100.100.100.149 PPTP 100.100.100.150 PPTP 100.100.100.151 PPTP 100.100.100.152 PPTP 100.100.100.153 PPTP 100.100.100.154 PPTP 100.100.100.155 PPTP 100.100.100.156 PPTP 100.100.100.157 PPTP 100.100.100.158 PPTP 100.100.100.159 PPTP 100.100.100.160 PPTP 100.100.100.161 PPTP 100.100.100.162 PPTP 100.100.100.163 PPTP 100.100.100.164 PPTP 100.100.100.165 PPTP 100.100.100.166 PPTP 100.100.100.167 PPTP 100.100.100.168 PPTP 100.100.100.169 PPTP 100.100.100.170 PPTP 100.100.100.171 PPTP 100.100.100.172 PPTP 100.100.100.173 PPTP 100.100.100.174 PPTP 100.100.100.175 PPTP 100.100.100.176 PPTP 100.100.100.177 PPTP 100.100.100.178 PPTP 100.100.100.179 PPTP 100.100.100.180 PPTP 100.100.100.181 PPTP 100.100.100.182 PPTP 100.100.100.183 PPTP 100.100.100.184 PPTP 100.100.100.185 PPTP 100.100.100.186 PPTP 100.100.100.187 PPTP 100.100.100.188 PPTP 100.100.100.189 PPTP 100.100.100.190 PPTP 100.100.100.191 PPTP 100.100.100.192 PPTP 100.100.100.193 PPTP 100.100.100.194 PPTP 100.100.100.195 PPTP 100.100.100.196 PPTP 100.100.100.197 PPTP 100.100.100.198 PPTP 100.100.100.200"); - // [IDGW-CVS 9044] set_and_get("pptpd.ip4_allow", "pptpd.ip4_allow: 192.168.10.1/32 192.168.10.2/32 192.168.10.3/32 192.168.10.4/32 192.168.10.5/32 192.168.10.6/32 192.168.10.7/32 192.168.10.8/32 192.168.10.9/32 192.168.10.10/32 192.168.10.11/32 192.168.10.12/32 192.168.10.13/32 192.168.10.14/32 192.168.10.15/32 192.168.10.16/32 192.168.10.17/32 192.168.10.18/32 192.168.10.19/32 192.168.10.20/32 192.168.10.21/32 192.168.10.22/32 192.168.10.23/32 192.168.10.24/32 192.168.10.25/32 192.168.10.26/32 192.168.10.27/32 192.168.10.28/32 192.168.10.29/32 192.168.10.30/32 192.168.10.31/32 192.168.10.32/32 192.168.10.33/32 192.168.10.34/32 192.168.10.35/32 192.168.10.36/32 192.168.10.37/32 192.168.10.38/32 192.168.10.39/32 192.168.10.40/32 192.168.10.41/32 192.168.10.42/32 192.168.10.43/32 192.168.10.44/32 192.168.10.45/32 192.168.10.46/32 192.168.10.47/32 192.168.10.48/32 192.168.10.49/32 192.168.10.50/32 192.168.10.51/32 192.168.10.52/32 192.168.10.53/32 192.168.10.54/32 192.168.10.55/32 192.168.10.56/32 192.168.1.57/32 192.168.1.58/32 192.168.1.59/32 192.168.10.60/32 192.168.10.61/32 192.168.10.62/32 192.168.10.63/32 192.168.10.64/32 192.168.10.65/32 192.168.10.66/32 192.168.10.67/32 192.168.10.68/32 192.168.10.69/32 192.168.10.70/32 192.168.10.71/32 192.168.10.72/32 192.168.10.73/32 192.168.10.74/32 192.168.10.75/32 192.168.10.76/32 192.168.10.77/32 192.168.10.78/32 192.168.10.79/32 192.168.10.80/32 192.168.10.81/32 192.168.10.82/32 192.168.10.83/32 192.168.10.84/32 192.168.10.85/32 192.168.10.86/32 192.168.10.87/32 192.168.10.88/32 192.168.10.89/32 192.168.10.90/32 192.168.10.91/32 192.168.10.92/32 192.168.10.93/32 192.168.10.94/32 192.168.10.95/32 192.168.10.96/32 192.168.10.97/32 192.168.10.98/32 192.168.10.99/32 192.168.10.100/32 192.168.10.101/32 192.168.10.102/32 192.168.10.103/32 192.168.10.104/32 192.168.10.105/32 192.168.10.106/32 192.168.10.107/32 192.168.10.108/32 192.168.10.109/32 192.168.10.110/32 192.168.10.111/32 192.168.10.112/32 192.168.10.113/32 192.168.10.114/32 192.168.10.115/32 192.168.10.116/32 192.168.10.117/32 192.168.10.118/32 192.168.1.119/32 192.168.1.120/32 192.168.1.121/32"); } @@ -96,15 +95,10 @@ static void test1() static void test2() { - // [IDGW-DEV 5246] 1024 bytes set_and_get("hogehoge", "hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge"); - // [IDGW-DEV 5246] 1024 bytes - 1024 bytes set_and_get("hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge", "hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge"); - // [IDGW-DEV 5246] 1024 bytes - 1024 bytes set_and_get("hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge", "hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge"); - // [IDGW-DEV 5246] 2048 bytes set_and_get("hogehoge", "hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge"); - // [IDGW-DEV 5246] 2048 bytes set_and_get("hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge", "hogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehogehoge"); } void @@ -140,7 +134,7 @@ test4_read_fn(void *cookie, char *buf, int len) rval = MIN(len, sizeof(test4_buf) - test4_off); if (rval == 0) - return 0; // EOF + return 0; /* EOF */ memcpy(buf, test4_buf + test4_off, rval); test4_off += rval; diff --git a/usr.sbin/npppd/common/rt_zebra.c b/usr.sbin/npppd/common/rt_zebra.c index 726262b6741..7dcdbb36d0f 100644 --- a/usr.sbin/npppd/common/rt_zebra.c +++ b/usr.sbin/npppd/common/rt_zebra.c @@ -2,7 +2,7 @@ * Copyright (c) 2007 * Internet Initiative Japan Inc. All rights reserved. */ -/* $Id: rt_zebra.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: rt_zebra.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /* * @file This file provides utility functions to help add/delete routing * information with GNU Zebra. This utility uses event(3) and uses a UNIX @@ -154,10 +154,10 @@ rt_zebra_start(rt_zebra *_this) } if ((ival = fcntl(sock, F_GETFL, 0)) < 0) { log_printf(LOG_ERR, "fcntl(,F_GETFL) failed: %m"); - goto reigai; + goto fail; } else if (fcntl(sock, F_SETFL, ival | O_NONBLOCK) < 0) { log_printf(LOG_ERR, "fcntl(,F_SETFL, +O_NONBLOCK) failed: %m"); - goto reigai; + goto fail; } _this->state = ZEBRA_STATUS_CONNECTING; @@ -175,7 +175,7 @@ rt_zebra_start(rt_zebra *_this) default: log_printf(LOG_ERR, "Connection to the zserv failed: %m"); - goto reigai; + goto fail; } } event_set(&_this->ev_sock, _this->sock, EV_READ|EV_WRITE, @@ -183,7 +183,7 @@ rt_zebra_start(rt_zebra *_this) event_add(&_this->ev_sock, NULL); return 0; -reigai: +fail: if (sock >= 0) close(sock); rt_zebra_stop(_this); diff --git a/usr.sbin/npppd/common/rtev_common.c b/usr.sbin/npppd/common/rtev_common.c index ea9403067eb..bcb52ca44dd 100644 --- a/usr.sbin/npppd/common/rtev_common.c +++ b/usr.sbin/npppd/common/rtev_common.c @@ -23,7 +23,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -/* $Id: rtev_common.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: rtev_common.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ /* * PF_ROUTE related utility functions. * <p> @@ -283,7 +283,7 @@ rtev_base_init(rtev_impl *impl, int flags) if (rtev_sndbuf == NULL) { if ((rtev_sndbuf = bytebuffer_wrap(rtev_buffer_space, sizeof(rtev_buffer_space))) == NULL) - goto reigai; + goto fail; bytebuffer_clear(rtev_sndbuf); } impl->base_on_rtevent = rtev_base_on_rtevent; @@ -292,7 +292,7 @@ rtev_base_init(rtev_impl *impl, int flags) singleton_impl = impl; return 0; -reigai: +fail: rtev_fini(); return 1; diff --git a/usr.sbin/npppd/common/rtev_libevent.c b/usr.sbin/npppd/common/rtev_libevent.c index c22fa80970f..bd062300ba6 100644 --- a/usr.sbin/npppd/common/rtev_libevent.c +++ b/usr.sbin/npppd/common/rtev_libevent.c @@ -23,7 +23,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -/* $Id: rtev_libevent.c,v 1.1 2010/01/11 04:20:57 yasuoka Exp $ */ +/* $Id: rtev_libevent.c,v 1.2 2010/07/01 03:38:17 yasuoka Exp $ */ #include <sys/types.h> #include <sys/socket.h> #include <net/route.h> @@ -252,17 +252,17 @@ rtev_libevent_init0(rtev_libevent *_this, int rt_delay, int send_delay, _this->impl.impl_on_write = rtev_libevent_on_write; if (rtev_base_init(&_this->impl, flags) != 0) - goto reigai; + goto fail; if ((sock = socket(PF_ROUTE, SOCK_RAW, AF_UNSPEC)) < 0) - goto reigai; + goto fail; dummy = 0; if ((fflags = fcntl(sock, F_GETFL, dummy)) < 0) - goto reigai; + goto fail; if (fcntl(sock, F_SETFL, fflags | O_NONBLOCK) < 0) - goto reigai; + goto fail; _this->sock = sock; @@ -274,7 +274,7 @@ rtev_libevent_init0(rtev_libevent *_this, int rt_delay, int send_delay, rtev_libevent_reset_event(_this); return 0; -reigai: +fail: if (sock >= 0) close(sock); diff --git a/usr.sbin/npppd/common/slist.c b/usr.sbin/npppd/common/slist.c index 7f6c67b05ce..5fedb7d141d 100644 --- a/usr.sbin/npppd/common/slist.c +++ b/usr.sbin/npppd/common/slist.c @@ -24,21 +24,20 @@ * SUCH DAMAGE. */ /**@file - * 任意のポインタに関するリスト操作を提供します。 + * provide list accesses against any pointer */ /* * void **list; - * list_size; // list に割り当てたサイズ。 - * last_idx; // 最初のインデックス - * first_idx; // 最後のインデックス + * list_size; // allocated size for the list + * last_idx; // The last index + * first_idx; // The first index * - * ・first_idx == last_idx は空を示します。 - * ・fist_idx と last_idx は 0 以上 list_size - 1 以下です。 - * ・使っているサイズは、(last_idx - first_idx) % list_size です。 - * list_size まで使ってしまうと、空と区別ができず区別しようとすると複雑 - * になるので、そういう状況を作りません。このため、list に割り当てたサイズ - * のうち 1個分は使いません。 - * ・XXX itr_curr が削除されると、 + * - first_idx == last_idx means empty. + * - 0 <= (fist_idx and last_idx) <= (list_size - 1) + * - Allocated size is (last_idx - first_idx) % list_size. + * To make the code for checking empty and full simple, we use only + * list_size-1 items instead of using the full size. + * - XXX Wnen itr_curr is removed... */ #include <sys/types.h> @@ -64,28 +63,27 @@ #endif /** - * 内部のインデックスが範囲内ならば、1 を範囲を越えている場合には 0 を - * を返すマクロです。 + * Returns 1 if a index is in the valid range, otherwise returns 0. */ #define VALID_IDX(_list, _idx) \ (((_list)->first_idx <= (_list)->last_idx) \ ? (((_list)->first_idx <= (_idx) && (_idx) < (_list)->last_idx)? 1 : 0)\ : (((_list)->first_idx <= (_idx) || (_idx) < (_list)->last_idx)? 1 : 0)) -/** インデックスを内部のインデックスに変換します。 */ +/** Convert an index into the internal index */ #define REAL_IDX(_list, _idx) \ (((_list)->first_idx + (_idx)) % (_list)->list_size) -/** 内部のインデックスをインデックスに変換します。 */ +/** Convert a virtual index into the index */ #define VIRT_IDX(_list, _idx) (((_list)->first_idx <= (_idx)) \ ? (_idx) - (_list)->first_idx \ : (_list)->list_size - (_list)->first_idx + (_idx)) -/** インデックスを示すメンバー変数を decrement します */ +/** Decrement an index */ #define DECR_IDX(_list, _memb) \ (_list)->_memb = ((_list)->list_size + --((_list)->_memb)) \ % (_list)->list_size -/** インデックスを示すメンバー変数を increment します */ +/** Increment an index */ #define INCR_IDX(_list, _memb) \ (_list)->_memb = (++((_list)->_memb)) % (_list)->list_size @@ -96,7 +94,7 @@ static __inline void slist_swap0 (slist *, int, int); #define itr_is_valid(list) ((list)->itr_next >= 0) #define itr_invalidate(list) ((list)->itr_next = -1) -/** 初期化処理を行います */ +/** Initialize a slist */ void slist_init(slist *list) { @@ -105,8 +103,8 @@ slist_init(slist *list) } /** - * リストのサイズを指定します。1つの要素は内部で利用されるので、必要なサイズ - * + 1 を指定します。サイズは小さくなりません。 + * Specify the size of a list. The size must be specified with the size you + * want to use +1. Extra 1 entry is for internal use. The size doesn't shrink. */ int slist_set_size(slist *list, int size) @@ -117,7 +115,7 @@ slist_set_size(slist *list, int size) return 0; } -/** 終了化処理を行います */ +/** Finish using. Free the buffers and reinit. */ void slist_fini(slist *list) { @@ -126,7 +124,7 @@ slist_fini(slist *list) slist_init(list); } -/** このリストの長さ */ +/** The length of the list */ int slist_length(slist *list) { @@ -136,16 +134,16 @@ slist_length(slist *list) : (list->list_size - list->first_idx + list->last_idx); } -/** リストがいっぱいの場合に割り当てサイズを成長させます。 */ +/** Extend the size. Used if the list is full. */ static int slist_grow0(slist *list, int grow_sz) { int size_new; void **list_new = NULL; - /* ひとつ追加できるか。できるならそのまま抜ける。*/ + /* just return if it is possible to add one item */ if (slist_length(list) + 1 < list->list_size) - /* list_size == slist_length() という状況は作らない */ + /* "+ 1" to avoid the situation list_size == slist_length() */ return 0; size_new = list->list_size + grow_sz; @@ -158,23 +156,27 @@ slist_grow0(slist *list, int grow_sz) list->list = list_new; if (list->last_idx < list->first_idx && list->last_idx >= 0) { + /* - * 空きが真ん中にある状況で右側に空きを作ったので、左側 - * を右にもっていく。 + * space is created at the right side when center has space, + * so move left side to right side */ if (list->last_idx <= grow_sz) { /* - * 左側を右側にもっていく場合に十分なスペースがある - * のですべて移動 + * The right side has enough space, so move the left + * side to right side. */ memmove(&list->list[list->list_size], &list->list[0], PTR_SIZE * list->last_idx); list->last_idx = list->list_size + list->last_idx; } else { - /* 左側をできるかぎり右端に copy */ + /* + * Copy the left side to right side as long as we + * can + */ memmove(&list->list[list->list_size], &list->list[0], PTR_SIZE * grow_sz); - /* 左側、copy した分を左にずらす */ + /* Shift the remain to left */ memmove(&list->list[0], &list->list[grow_sz], PTR_SIZE *(list->last_idx - grow_sz)); @@ -192,7 +194,7 @@ slist_grow(slist *list) return slist_grow0(list, GROW_SIZE); } -/** リストの末尾に要素を追加します。*/ +/** Add an item to a list */ void * slist_add(slist *list, void *item) { @@ -213,7 +215,7 @@ slist_add(slist *list, void *item) #define slist_get0(list_, idx) ((list_)->list[REAL_IDX((list_), (idx))]) -/** リストの末尾に指定したリストの要素全てを追加します */ +/** Add all items in add_items to a list. */ int slist_add_all(slist *list, slist *add_items) { @@ -228,7 +230,7 @@ slist_add_all(slist *list, slist *add_items) return 0; } -/** idx番目の要素を返します。*/ +/** Return "idx"th item. */ void * slist_get(slist *list, int idx) { @@ -241,7 +243,7 @@ slist_get(slist *list, int idx) return slist_get0(list, idx); } -/** idx番目の要素をセットします。*/ +/** Store a value in "idx"th item */ int slist_set(slist *list, int idx, void *item) { @@ -256,7 +258,7 @@ slist_set(slist *list, int idx, void *item) return 0; } -/** 1番目の要素を削除して取り出します。*/ +/** Remove the 1st entry and return it. */ void * slist_remove_first(slist *list) { @@ -278,10 +280,7 @@ slist_remove_first(slist *list) return oldVal; } - - - -/** 最後の要素を削除して取り出します。*/ +/** Remove the last entry and return it */ void * slist_remove_last(slist *list) { @@ -295,7 +294,7 @@ slist_remove_last(slist *list) return list->list[list->last_idx]; } -/** 全て要素を削除します */ +/** Remove all entries */ void slist_remove_all(slist *list) { @@ -306,20 +305,20 @@ slist_remove_all(slist *list) list->list = list0; } -/* this doesn't check boudary. */ +/* Swap items. This doesn't check boudary. */ static __inline void slist_swap0(slist *list, int m, int n) { void *m0; - itr_invalidate(list); /* イテレータ無効 */ + itr_invalidate(list); /* Invalidate iterator */ m0 = list->list[REAL_IDX(list, m)]; list->list[REAL_IDX(list, m)] = list->list[REAL_IDX(list, n)]; list->list[REAL_IDX(list, n)] = m0; } -/** リストの m 番目の要素と n 番目の要素を入れ換えます。 */ +/** Swap between mth and nth */ void slist_swap(slist *list, int m, int n) { @@ -339,7 +338,7 @@ slist_swap(slist *list, int m, int n) slist_swap0(list, m, n); } -/** idx 番目の要素を削除します */ +/** Remove "idx"th item */ void * slist_remove(slist *list, int idx) { @@ -365,9 +364,9 @@ slist_remove(slist *list, int idx) list->itr_next = -2; /* on the last item */ } - /* last 側を縮めるか、first 側を縮めるか。*/ + /* should we reduce the last side or the first side? */ if (list->first_idx < list->last_idx) { - /* いちおう短い方を選択 */ + /* take the smaller side */ if (idx0 - list->first_idx < list->last_idx - idx0) { first = list->first_idx; INCR_IDX(list, first_idx); @@ -377,7 +376,8 @@ slist_remove(slist *list, int idx) } } else { /* - * 0 < last (未使用) first < idx < size なので first 側を縮める + * 0 < last (unused) first < idx < size, so let's reduce the + * first. */ if (list->first_idx <= idx0) { first = list->first_idx; @@ -388,10 +388,10 @@ slist_remove(slist *list, int idx) } } - /* last側 */ + /* the last side */ if (last != -1 && last != 0 && last != idx0) { - /* idx0 〜 last を左にひとつずらす */ + /* move left the items that is from idx0 to the last */ if (itr_is_valid(list) && idx0 <= list->itr_next && list->itr_next <= last) { DECR_IDX(list, itr_next); @@ -402,10 +402,10 @@ slist_remove(slist *list, int idx) memmove(&list->list[idx0], &list->list[idx0 + 1], (PTR_SIZE) * (last - idx0)); } - /* first側 */ + /* the first side */ if (first != -1 && first != idx0) { - /* first 〜 idx0 を右にひとつずらす */ + /* move right the items that is from first to the idx0 */ if (itr_is_valid(list) && first <= list->itr_next && list->itr_next <= idx0) { INCR_IDX(list, itr_next); @@ -425,10 +425,8 @@ slist_remove(slist *list, int idx) } /** - * シャッフルします。 - * <p> - * <b>slist_shuffle は random(3) を使ってます。使用前に srandom(3) してく - * ださい。</b></p> + * Shuffle items. + * slist_shuffle() uses random(3). Call srandom(3) before use it. */ void slist_shuffle(slist *list) @@ -440,9 +438,7 @@ slist_shuffle(slist *list) slist_swap0(list, i - 1, (int)(random() % i)); } -/** - * イテレータを初期化します。ひとつの slist インスタンスでひとつしか使えません。 - */ +/** Init an iterator. Only one iterator exists. */ void slist_itr_first(slist *list) { @@ -452,10 +448,10 @@ slist_itr_first(slist *list) } /** - * イテレータが次の要素に進めるかどうかを返します。 - * @return イテレータが次の要素を返すことができる場合に 1 を返します。 - * 終端に達したか、イテレータが最後まで達したか、リスト構造の変更があって、 - * 続行不能な場合には 0 が返ります。 + * Return whether a iterator can go to the next item. + * @return Return 1 if the iterator can return the next item. + * Return 0 it reaches the end of the list or the list is modified + * destructively. */ int slist_itr_has_next(slist *list) @@ -465,7 +461,7 @@ slist_itr_has_next(slist *list) return VALID_IDX(list, list->itr_next); } -/** イテレータの次の要素を取り出しつつ、次の要素に進めます。 */ +/** Return the next item and iterate to the next */ void * slist_itr_next(slist *list) { @@ -488,7 +484,7 @@ slist_itr_next(slist *list) return rval; } -/** イテレータの現在の要素を削除します */ +/** Delete the current iterated item */ void * slist_itr_remove(slist *list) { diff --git a/usr.sbin/npppd/common/slist.h b/usr.sbin/npppd/common/slist.h index 10fea5c8929..020fd081e88 100644 --- a/usr.sbin/npppd/common/slist.h +++ b/usr.sbin/npppd/common/slist.h @@ -53,7 +53,7 @@ void *slist_remove_first (slist *); void *slist_remove_last (slist *); void slist_swap (slist *, int, int); void *slist_remove (slist *, int); -/* slist_shuffle は random(3) を使ってます。使用前に srandom(3) してください。*/ +/* slist_shuffle() uses random(3), so call srandom(3) before use. */ void slist_shuffle (slist *); void slist_itr_first (slist *); int slist_itr_has_next (slist *); diff --git a/usr.sbin/npppd/common/slist_test.c b/usr.sbin/npppd/common/slist_test.c index f7ac334be69..0457f08ec0e 100644 --- a/usr.sbin/npppd/common/slist_test.c +++ b/usr.sbin/npppd/common/slist_test.c @@ -77,7 +77,7 @@ dump(slist *l) fprintf(stderr, "\n"); } -// まんなかに空きの場合に削除系のテスト +/* Test code for removing of the first, last and middle item. */ static void test_01a() { @@ -99,7 +99,7 @@ test_01a() } \ } - // 先頭要素削除 + /* Remove the first item. */ SETUP(); f = 0; while (slist_length(l) > 0) { @@ -110,7 +110,7 @@ test_01a() } } - // 最終要素削除 + /* Remove the last item. */ SETUP(); while (slist_length(l) > 0) { slist_remove(l, slist_length(l) - 1); @@ -118,7 +118,7 @@ test_01a() ASSERT((int)slist_get(l, i) == i); } } - // 最終要素-1削除 + /* Remove the second item from the end. */ SETUP(); while (slist_length(l) > 1) { slist_remove(l, slist_length(l) - 2); @@ -156,13 +156,13 @@ test_01() ASSERT((int)slist_get(&sl, 254) == 129 + 253); ASSERT((int)slist_length(&sl) == 255); - //dump(&sl); - //printf("==\n"); + /* dump(&sl); */ + /* printf("==\n"); */ slist_add(&sl, (void *)(128 + 255)); ASSERT((int)slist_get(&sl, 127) == 255); - //ASSERT((int)slist_get(&sl, 255) == 128 + 255); + /* ASSERT((int)slist_get(&sl, 255) == 128 + 255); */ ASSERT((int)slist_length(&sl) == 256); - //dump(&sl); + /* dump(&sl); */ } static void @@ -175,7 +175,7 @@ test_02() slist_init(&sl); - // 内部配置が、左側に 300 個、右側に 211 個になるように配置 + /* Place 300 items for left side and 211 items for right side. */ for (i = 0; i < 511; i++) slist_add(&sl, (void *)i); for (i = 0; i <= 300; i++) @@ -184,27 +184,27 @@ test_02() slist_add(&sl, (void *)i); - // index 番号になるように再度割り当て + /* Set values to make index number and value the same. */ for (i = 0; i < slist_length(&sl); i++) slist_set(&sl, i, (void *)(i + 1)); - ASSERT(slist_length(&sl) == 511); //論理サイズは511 - ASSERT((int)sl.list[511] == 211); //右端が 211番目 - ASSERT((int)sl.list[0] == 212); //左端が 212番目 - ASSERT(sl.list_size == 512); //物理サイズは 512 + ASSERT(slist_length(&sl) == 511); /* The logical length is 511. */ + ASSERT((int)sl.list[511] == 211); /* The most right is 211th. */ + ASSERT((int)sl.list[0] == 212); /* The most left is 212th. */ + ASSERT(sl.list_size == 512); /* The physical size is 512. */ - slist_add(&sl, (void *)512); // 512番めを追加 + slist_add(&sl, (void *)512); /* Add 512th item. */ - ASSERT(sl.list_size == 768); //物理サイズが拡大 - ASSERT(slist_length(&sl) == 512); //論理サイズは512 - ASSERT((int)sl.list[511] == 211); //繋め - ASSERT((int)sl.list[512] == 212); //繋め - ASSERT((int)sl.list[767] == 467); //右端が 467番目 - ASSERT((int)sl.list[0] == 468); //左端が 468番目 + ASSERT(sl.list_size == 768); /* The physical size is extended. */ + ASSERT(slist_length(&sl) == 512); /* The logical length is 512. */ + ASSERT((int)sl.list[511] == 211); /* boundary */ + ASSERT((int)sl.list[512] == 212); /* boundary */ + ASSERT((int)sl.list[767] == 467); /* The most right is 467th. */ + ASSERT((int)sl.list[0] == 468); /* The most left is 468th. */ - //全部チェック + /* Check all items */ for (i = 0; i < slist_length(&sl); i++) - ASSERT((int)slist_get(&sl, i) == i + 1); // チェック + ASSERT((int)slist_get(&sl, i) == i + 1); /* check */ } static void @@ -225,8 +225,8 @@ test_03() } slist_remove(&sl, 0); ASSERT(slist_length(&sl) == 0); - //dump(&sl); - //TEST(test_02); + /* dump(&sl); */ + /* TEST(test_02); */ } static void @@ -238,30 +238,31 @@ test_itr_subr_01(slist *l) slist_set(l, i, (void *)(i + 1)); slist_itr_first(l); - ASSERT((int)slist_itr_next(l) == 1); // 普通にイテレート - ASSERT((int)slist_itr_next(l) == 2); // 普通にイテレート - slist_remove(l, 2); // next を削除 - // "3" が削除 - ASSERT((int)slist_itr_next(l) == 4); // 削除したものはスキップ - slist_remove(l, 1); // 通りすぎたところを削除 - // "2" を削除 - ASSERT((int)slist_itr_next(l) == 5); // 影響なし - ASSERT((int)slist_get(l, 0) == 1); // 削除確認 - ASSERT((int)slist_get(l, 1) == 4); // 削除確認 - ASSERT((int)slist_get(l, 2) == 5); // 削除確認 - - // 255 アイテム中 2 個削除し、4回イテレートし、1回の削除は通りすぎ - // たあとなので、残り 250回 + ASSERT((int)slist_itr_next(l) == 1); /* normal iterate */ + ASSERT((int)slist_itr_next(l) == 2); /* normal iterate */ + slist_remove(l, 2); /* remove next. "3" is removed */ + ASSERT((int)slist_itr_next(l) == 4); /* removed item is skipped */ + slist_remove(l, 1); /* remove past item. "2" is removed */ + ASSERT((int)slist_itr_next(l) == 5); /* no influence */ + ASSERT((int)slist_get(l, 0) == 1); /* checking for removing */ + ASSERT((int)slist_get(l, 1) == 4); /* checking for removing */ + ASSERT((int)slist_get(l, 2) == 5); /* checking for removing */ + /* + * Total number was 255. We removed 2 items and iterated 4 times. + * 1 removing was past item, so the remaining is 250. + */ for (i = 0; i < 249; i++) ASSERT(slist_itr_next(l) != NULL); ASSERT(slist_itr_next(l) != NULL); ASSERT(slist_itr_next(l) == NULL); - // 上記と同じだが、最後を取り出す前に削除 + /* + * Same as above except removing before getting the last item. + */ - // リセット (253アイテム) + /* Reset (253 items) */ for (i = 0; i < slist_length(l); i++) slist_set(l, i, (void *)(i + 1)); slist_itr_first(l); @@ -272,7 +273,7 @@ test_itr_subr_01(slist *l) ASSERT(slist_itr_next(l) != NULL); slist_remove(l, 252); - ASSERT(slist_itr_next(l) == NULL); // 最後を指してたけど、NULL + ASSERT(slist_itr_next(l) == NULL); /* The last item is NULL */ slist_itr_first(l); while (slist_length(l) > 0) @@ -295,7 +296,7 @@ test_04() test_itr_subr_01(&sl); for (i = 0; i < 256; i++) { - // ローテーションして、どんな物理配置でも成功すること確認 + /* Verify any physical placements are OK by rotating. */ sl.first_idx = i; sl.last_idx = sl.first_idx + 255; sl.last_idx %= sl.list_size; @@ -304,7 +305,7 @@ test_04() } } -// 物理配置の一番最後の要素を削除しても、大丈夫か。 +/* Verify removing the last item on the physical location */ static void test_05() { @@ -313,16 +314,16 @@ test_05() slist *l = &sl; slist_init(&sl); - // ぎりぎりまで追加 + /* Fill */ for (i = 0; i < 255; i++) { slist_add(&sl, (void *)i); } - // 254 個削除 + /* Remove 254 items */ for (i = 0; i < 254; i++) { slist_remove_first(&sl); } slist_set(l, 0, (void *)0); - // 7個追加 + /* Add 7 items */ for (i = 0; i < 8; i++) { slist_add(&sl, (void *)i + 1); } @@ -401,7 +402,6 @@ test_07() static void test_08() { - //int i, x; slist sl; slist *l = &sl; @@ -431,8 +431,6 @@ test_08() ASSERT((int)slist_get(l, 1) == 3); ASSERT((int)slist_get(l, 2) == 4); ASSERT((int)slist_get(l, 3) == 5); - - //dump(l); } static void @@ -456,7 +454,7 @@ test_09() ASSERT((int)slist_itr_next(l) == 3); /* 3 */ /* reaches the last */ slist_add(l, (void *)4); /* add a new item */ - ASSERT(slist_itr_has_next(l)); /* iterates the new*/ + ASSERT(slist_itr_has_next(l)); /* iterates the new */ ASSERT((int)slist_itr_next(l) == 4); slist_fini(l); @@ -475,11 +473,8 @@ test_09() ASSERT((int)slist_itr_next(l) == 2); /* 2 */ ASSERT((int)slist_itr_next(l) == 3); /* 3 */ /* reaches the last */ - //dump(l); slist_itr_remove(l); /* and remove the last*/ - //dump(l); slist_add(l, (void *)4); /* add 4 (new last)*/ - //dump(l); ASSERT(slist_itr_has_next(l)); /* */ ASSERT((int)slist_itr_next(l) == 4); /* 4 */ slist_fini(l); |