From 18 Jan, 2021 0:00 UTC: All services will be temporary unavailable for maintenance
  • R/O
  • SSH
  • HTTPS

ttssh2: Commit


Commit MetaInfo

Revision9116 (tree)
Time2020-12-29 00:37:47
Authorzmatsuo

Log Message

コンパイラ警告対策

- size_t は unsigned なので、常に0または+の値となる

- >= 0 や < 0 などの常に成り立つ条件式を削除
- warning: comparison of unsigned expression in '< 0' is always false [-Wtype-limits]

- switch() 内の break なし部分に FALLTHROUGH 追加

Change Summary

Incremental Difference

--- trunk/TTProxy/ProxyWSockHook.h (revision 9115)
+++ trunk/TTProxy/ProxyWSockHook.h (revision 9116)
@@ -1065,7 +1065,7 @@
10651065 ProxyInfo proxy;
10661066 SettingDialog():lock(false) {
10671067 }
1068-
1068+
10691069 int open(HWND owner) {
10701070 return Dialog::open(instance().resource_module, IDD_SETTING, owner);
10711071 }
@@ -1243,7 +1243,7 @@
12431243 if (WSAGetLastError() != WSAEWOULDBLOCK) {
12441244 return SOCKET_ERROR;
12451245 }
1246- // no break
1246+ /* FALLTHROUGH */ // no break
12471247 case 0:
12481248 end = dst; /* end of stream */
12491249 break;
--- trunk/TTProxy/YCL/include/YCL/String.h (revision 9115)
+++ trunk/TTProxy/YCL/include/YCL/String.h (revision 9116)
@@ -242,9 +242,7 @@
242242 // 返値:
243243 // 文字の見つかったインデックス。見つからなければ-1。
244244 int indexOf(char chr, size_t from)const {
245- if (from < 0)
246- from = 0;
247- else if (from >= length())
245+ if (from >= length())
248246 return -1;
249247 const char* found = strchr(string + from, chr);
250248 if (found == NULL)
@@ -267,9 +265,7 @@
267265 // 文字列の見つかったインデックス。見つからなければ-1。
268266 //
269267 int indexOf(const char* str, size_t from)const {
270- if (from < 0)
271- from = 0;
272- else if (from >= length())
268+ if (from >= length())
273269 return -1;
274270 const char* found = strstr(string + from, str);
275271 if (found == NULL)
@@ -360,7 +356,7 @@
360356 // 返値:
361357 // 指定の位置にある文字。
362358 char charAt(size_t index)const {
363- return index >= 0 && index < length() ? string[index] : '\0';
359+ return index < length() ? string[index] : '\0';
364360 }
365361 // 指定の文字を指定の文字に置き換えます。
366362 // 引数:
--- trunk/TTProxy/YCL/include/YCL/StringBuffer.h (revision 9115)
+++ trunk/TTProxy/YCL/include/YCL/StringBuffer.h (revision 9116)
@@ -43,7 +43,9 @@
4343 }else{
4444 buffer = new char[bufferSize];
4545 }
46- memcpy(buffer, source, validLength);
46+ if (source != NULL) {
47+ memcpy(buffer, source, validLength);
48+ }
4749 memset(buffer + validLength, '\0', bufferSize - validLength);
4850 }
4951 public:
@@ -110,7 +112,7 @@
110112 // 返値:
111113 // 指定の位置の文字。
112114 char charAt(size_t index)const {
113- return index >= 0 && index < validLength ? buffer[index] : '\0';
115+ return index < validLength ? buffer[index] : '\0';
114116 }
115117 // 指定の位置の文字を取得する。
116118 // 引数:
@@ -118,9 +120,7 @@
118120 // 返値:
119121 // 指定の位置の文字の参照。
120122 char& charAt(size_t index) {
121- if (index < 0) {
122- index = 0;
123- }else if (index >= validLength) {
123+ if (index >= validLength) {
124124 ensureCapacity(validLength + 1);
125125 index = validLength++;
126126 }
@@ -178,8 +178,6 @@
178178 // 返値:
179179 // 削除結果。
180180 StringBuffer& remove(size_t start, size_t end) {
181- if (start <= 0)
182- start = 0;
183181 if (start < end) {
184182 if (end < validLength){
185183 memcpy(buffer + start, buffer + end, validLength - end);
@@ -198,8 +196,6 @@
198196 // 返値:
199197 // 置換結果。
200198 StringBuffer& replace(size_t start, size_t end, const char* source) {
201- if (start < 0)
202- start = 0;
203199 if (end > validLength)
204200 end = validLength;
205201 if (start < end) {
@@ -217,8 +213,6 @@
217213 // 返値:
218214 // 指定の位置の文字列。
219215 String substring(size_t index)const {
220- if (index < 0)
221- index = 0;
222216 return String(buffer + index, validLength - index);
223217 }
224218 // 指定の位置の文字列を取得する。
@@ -228,8 +222,6 @@
228222 // 返値:
229223 // 指定の位置の文字列。
230224 String substring(size_t start, size_t end)const {
231- if (start < 0)
232- start = 0;
233225 if (end > validLength)
234226 end = validLength;
235227 return String(buffer + start, end - start);
@@ -260,9 +252,7 @@
260252 // 返値:
261253 // 挿入結果。
262254 StringBuffer& insert(size_t index, const char* source, size_t length) {
263- if (index < 0)
264- index = 0;
265- else if (index >= validLength)
255+ if (index >= validLength)
266256 index = validLength;
267257 size_t oldLength = validLength;
268258 ensureCapacity(validLength + length);
Show on old repository browser