[Bug #3315] problems with ftp code in Borland C++ Builder

[nobody]

=================== BUG #3315: LATEST MODIFICATIONS ==================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3315&group_id=89

Changes by: Federico Montesino Pouzols <address@hidden>
Date: Sun 05/11/2003 at 15:02 (Europe/Madrid)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Later
         Assigned to | None                      | fedemp
                  CC |                           | address@hidden


------------------ Additional Follow-up Comments ----------------------------

 The second issue is fixed in CVS. As for the first one, I
suspect that your proposed change will break the library on
other platforms. Does anyone know if this happens because of
the istream implementation of Borland c++?.




=================== BUG #3315: FULL BUG SNAPSHOT ===================


Submitted by: awagger                 Project: commonc++                    
Submitted on: Thu 04/24/2003 at 00:36
Category:  None                       Severity:  5 - Major                  
Bug Group:  None                      Resolution:  Later                    
Assigned to:  fedemp                  Status:  Open                         

Summary:  problems with ftp code in Borland C++ Builder

Original Submission:  I found (and fixed) two problems with the ftp code in 
libcommonc++2-1.0.9 when using the Borland C++ Builder (version 5).

1) I had to comment out line ftp.cpp:647 in method 
FTPSocket::DirEntry::parse(string li):

    str.get();  // skip space

since the mentioned space seemed to be included in the variable "junk", filled 
in the line bevore (ftp.cpp:646):

    str >> access >> junk >> user >> group >> size >> junk >> junk >> junk;


I don't know if it is an istream operator implementation problem or a unix <-> 
windows problem.


2) I had to change line ftp.cpp:482:

    ofstream f( lfilename.c_str() );

by explicitly specifying binary output mode like this:

    ofstream f( lfilename.c_str(),
        std::ios::out|std::ios::trunc|std::ios::binary );

otherwise the char '\n' got written as two bytes (as usual in windows in 
text-mode). So this is clearly a unix <-> windows porting problem. But I think, 
it should be perfectly ok, to use the same code for unix, since the result 
should be the same.



Follow-up Comments
*******************

-------------------------------------------------------
Date: Sun 05/11/2003 at 15:02       By: fedemp

 The second issue is fixed in CVS. As for the first one, I
suspect that your proposed change will break the library on
other platforms. Does anyone know if this happens because of
the istream implementation of Borland c++?.



CC List
*******

CC Address                          | Comment
------------------------------------+-----------------------------
address@hidden               | 



No files currently attached


For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3315&group_id=89