Summary: | Windows Mac linefeeds exchanged in codegeneration | ||
---|---|---|---|
Product: | [Applications] umbrello | Reporter: | Ferenc Veres <lion> |
Component: | general | Assignee: | Umbrello Development Group <umbrello-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ralf.habacker |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/umbrello/eb4a0018810fc8c96fac93e39cd6a71115bb3036 | Version Fixed In: | 2.21.80 (KDE Applications 17.03.80) |
Attachments: | Patch to replace dos and mac linefeed to make it correctly generated. |
Description
Ferenc Veres
2008-11-03 16:14:29 UTC
Created attachment 28299 [details]
Patch to replace dos and mac linefeed to make it correctly generated.
@Lays: Please add test cases for this issue to this bug and if you can confirm this bug. Then apply this patch to a local build and check if it solves the mentioned issues. Hi guys, The current code on quickgit.kde.org reads as: case MAC: m_lineEndingChars = QString(QLatin1String("\r\n")); break; case DOS: m_lineEndingChars = QString(QLatin1String("\r")); break; It's no rocket science to see this problem still exists in the code, case MAC should be "\r" and case DOS should be "\r\n". However I didn't check if this code is still in use, as I remember a new code generating subsystem was being implemented back then. https://quickgit.kde.org/?p=umbrello.git&a=blob&h=6033f2c2642dbdab7d351b622d0d631581cb5ea1&hb=6051fd05ef97566ea8122858c8c6a69a3b8c10ff&f=umbrello%2Fcodegenerators%2Fcodegenerationpolicy.cpp (In reply to Ferenc Veres from comment #3) > Hi guys, > The current code on quickgit.kde.org reads as: > > case MAC: > m_lineEndingChars = QString(QLatin1String("\r\n")); > break; > case DOS: > m_lineEndingChars = QString(QLatin1String("\r")); > break; > > It's no rocket science to see this problem still exists in the code, case > MAC should be "\r" and case DOS should be "\r\n". > > However I didn't check if this code is still in use, as I remember a new > code generating subsystem was being implemented back then. Thanks for the info. There is a GSOC student currently studying umbrello code and bug fix workflow. This bug has been selected for this purpose. Git commit eb4a0018810fc8c96fac93e39cd6a71115bb3036 by Ralf Habacker. Committed on 13/03/2017 at 13:42. Pushed by habacker into branch 'master'. Fix 'Windows Mac linefeeds exchanged in codegeneration'. FIXED-IN:2.21.80 (KDE Applications 17.03.80) M +2 -2 umbrello/codegenerators/codegenerationpolicy.cpp https://commits.kde.org/umbrello/eb4a0018810fc8c96fac93e39cd6a71115bb3036 |