Re: Patches for slrn
"John E. Davis" <[email protected]>
| Newsgroups | gmane.network.slrn.user |
|---|---|
| Message-ID | <[email protected]> |
Jörg Sommer <[email protected]> wrote: > I've added two new patches to the list: > Remove unused configure check for X libs and headers > > Slrn and slrnpull build fine without this check and I found no reference > to X file in the source code. However some systems require additional networking libraries that this check brings in. For example on solaris the macro will pull in the socket and nsl libraries. Without it, slrn will not compile. > and > > Correct format specifier for size_t > > On alpha, ia64 and s390 the type of size_t is not int but long unsinged > int. The correct format specifier for printing size_t is %zu. > This is not portable to all systems that slrn supports. A more portable method is to use "%lu" and cast the size_t argument to an unsigned long. Ten years ago I would have resisted using %u, because it was not supported by all systems that slrn ran on. I avoided the use of `const' for many years for a similar reason. At this point I believe that I can safely assume that the target system supports C90, but not C99. In fact, gcc does not yet provide full support for C99; see <http://gcc.gnu.org/c99status.html> for the current status. Thanks, --John ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/