[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Gap-dev-discuss] LuserNet: warning fix / code cleanup : char signedness
From: |
Riccardo Mottola |
Subject: |
[Gap-dev-discuss] LuserNet: warning fix / code cleanup : char signedness |
Date: |
Fri, 13 Jun 2014 15:31:41 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 SeaMonkey/2.26 |
Hi,
(Yavor, are you on gap-dev? perhaps you should so we can avoid sending a
lots of mails privately, I continued getting bounces from German for
example, I fear he did not get half of our conversation)
trying to squeeze out some warnings in LuserNET, before its release
together with Pantomime and GNUMail.
Alexander's code is for me very difficult to read and maintain, so I
proceed extremely carefully and only fix stuff when really
On NetBSD I get an error that atoi() expects a const char*, not a
unsigned char*. How is it on linux and other systems?
My first Idea was to apply the patch below instead of a cast, however
that triggers another maze of warnings and errors.
Any proposals? I'd leave it as it is now... and not silence the cast if
we are absolutely sure.
Index: NNTPServer.m
===================================================================
--- NNTPServer.m (revision 310)
+++ NNTPServer.m (working copy)
@@ -373,7 +373,7 @@
int resp;
int resp_code; /* numeric code in status */
char *resp_msg; /* status message (if any) */
- unsigned char *resp_buf; /* read buffer */
+ char *resp_buf; /* read buffer */
int resp_buf_len; /* length of buffer */
int resp_data_len; /* data has been processed up to this
point */
int resp_data_nline;
@@ -1767,7 +1767,7 @@
}
- [Gap-dev-discuss] LuserNet: warning fix / code cleanup : char signedness,
Riccardo Mottola <=