pspp-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[patch #5731] Reduce platform dependence


From: Ben Pfaff
Subject: [patch #5731] Reduce platform dependence
Date: Fri, 09 Feb 2007 04:44:39 +0000
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.1) Gecko/20061205 Iceweasel/2.0.0.1 (Debian-2.0.0.1+dfsg-1)

Follow-up Comment #3, patch #5731 (project pspp):

>This is an improvement, but obviously there is more to be done.

Portability is a process, not a product...

>> Remove `stat-macros' module. 
>Why did you do that? It no longer builds, since several files 
>include stat-macros.h

stat-macros.h is not useful.  Its functionality has been subsumed by the
sys_stat module.

I've now dropped #include "stat-macros.h" from everywhere it was included.

>While we're adjusting gl macros, can we get rid of strstr which no
>longer exists in gnulib ? 

OK.

> I think it's easier to read, if you put the function inside the
> #if / #endifs 

OK, done.

>I prefer this how it was before. 

OK, done.  I changed this to:


/* Returns true iff NAME specifies an absolute file name. */
bool
fn_is_absolute (const char *name)
{
  return name[0] == '/';
}


We could check for \ under windows, but what's the proper test macro for
that?  Should it be __WIN32__?

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?5731>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/





reply via email to

[Prev in Thread] Current Thread [Next in Thread]