MantisBT - Open CASCADE
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0029219Open CASCADE[OCCT] OCCT:Application Frameworkpublic2017-10-12 14:532019-09-04 17:49
Reporterkgv 
Assigned Tokgv 
PrioritynormalSeverityminor 
StatusfeedbackResolutionopen 
PlatformOSOS Version
Product Version[OCCT] 7.2.0 
Target Version[OCCT] 7.5.0*Fixed in Version 
Test case number
Summary0029219: Application Framework - XML document is written with CRLF on Windows
DescriptionIt seems that XML document has different EOL depending on the platform - CRLF on Windows and LF on other systems. Most likely, it is due to std::ofstream being opened in text mode instead of binary mode.

It would be useful saving XML files in consistent way across systems - e.g. using LF instead of CRLF on Windows (this would also reduce file size a little).
Steps To Reproduce
Additional information
and documentation updates
TagsNo tags attached.
Relationships
related to 0029218closed bugmaster Open CASCADE Application Framework - reading XML document calls LDOM parsing at least 3 times 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-10-12 14:53kgvNew Issue
2017-10-12 14:53kgvAssigned To => mpv
2017-10-12 14:53kgvRelationship addedrelated to 0029218
2017-10-12 14:54kgvRelationship addedrelated to 0029216
2017-10-16 10:04mpvNote Added: 0071511
2017-10-16 10:04mpvAssigned Tompv => kgv
2017-10-16 10:04mpvStatusnew => feedback
2017-10-16 10:28kgvNote Added: 0071516
2019-09-04 17:49kgvTarget Version7.4.0 => 7.5.0*

Notes
(0071511)
mpv   
2017-10-16 10:04   
Windows end-of-lines for text files is standard. XML is a text file. We cannot simply change it in OCCT.
(0071516)
kgv   
2017-10-16 10:28   
OK, the issue is not specific to XML export - BREP/STEP/IGES are also written with CRLF on Windows. However, behavior is still questionable:
- OCCT XML:    CRLF
- OCCT BREP:   CRLF
- OCCT STEP:   CRLF
- OCCT IGES:   CRLF
- OCCT VRML:   CRLF
- OCCT STL:    LF
- Blender X3D: CRLF
- Blender STL: CRLF
- Blender OBJ: LF
- Blender PLY: LF
- RPly:        LF