Re: [Flightgear-devel] Re: Fw: FlightGear/src/Scenery FGTileLoader.cxx, 1.7, 1.8

2004-06-07 Thread Erik Hofman 
Alex Romosan wrote:
Norman Vine [EMAIL PROTECTED] writes:

Doesn't SGPath.apapend just do the right thing here ? 

i.e. there shouldn't be a need to do the

! #ifdef _MSC_VER
! tmp.append( ;);
! #else
! tmp.append( :);
! #endif
kludge in the patch below
if append() doesn't do the right thing I suggest we would be
better off fixing append() rather then kludging every instance 
of it use.

we should use sgDirPathSep instead. it's defined in
simgear/misc/sg_path.cxx, but it should be moved to the header file
instead so it can be used elsewhere.
Ah, good catch!
I have been searching for that but couldn't find one. And I wanted to 
get something usable in there and look for a better solution later on.

I'll take a look and make the necessary changes.
Thanks.
Erik
___
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel

[Flightgear-devel] Re: Fw: FlightGear/src/Scenery FGTileLoader.cxx, 1.7, 1.8

2004-06-06 Thread Alex Romosan 
Norman Vine [EMAIL PROTECTED] writes:

 Doesn't SGPath.apapend just do the right thing here ? 

 i.e. there shouldn't be a need to do the

 ! #ifdef _MSC_VER
 ! tmp.append( ;);
 ! #else
 ! tmp.append( :);
 ! #endif

 kludge in the patch below

 if append() doesn't do the right thing I suggest we would be
 better off fixing append() rather then kludging every instance 
 of it use.

we should use sgDirPathSep instead. it's defined in
simgear/misc/sg_path.cxx, but it should be moved to the header file
instead so it can be used elsewhere.

--alex--

-- 
| I believe the moment is at hand when, by a paranoiac and active |
|  advance of the mind, it will be possible (simultaneously with  |
|  automatism and other passive states) to systematize confusion  |
|  and thus to help to discredit completely the world of reality. |

___
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel

RE: [Flightgear-devel] Re: Fw: FlightGear/src/Scenery FGTileLoader.cxx, 1.7, 1.8

2004-06-06 Thread Norman Vine 
Alex Romosan writes:
 
 Norman Vine [EMAIL PROTECTED] writes:
 
  Doesn't SGPath.apapend just do the right thing here ? 
 
  i.e. there shouldn't be a need to do the
 
  ! #ifdef _MSC_VER
  ! tmp.append( ;);
  ! #else
  ! tmp.append( :);
  ! #endif
 
  kludge in the patch below
 
  if append() doesn't do the right thing I suggest we would be
  better off fixing append() rather then kludging every instance 
  of it use.
 
 we should use sgDirPathSep instead. it's defined in
 simgear/misc/sg_path.cxx, but it should be moved to the header file
 instead so it can be used elsewhere.

Agreed but 
append() should/will just do this automagically  :-)

// append another piece to the existing path
void SGPath::append( const string p ) {
if ( path.size() == 0 ) {
path = p;
} else {
if ( p[0] != sgDirPathSep ) {
path += sgDirPathSep;
}
path += p;
}
fix();
}

Cheers

Norman

___
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel