Bug 112141

SVN commit 478258 by alund:

Untranslated error message to inform users that we can't replace newlines.
Dont' attempt to, as it may lead to an infinite loop, again potentially leading to a crash at some point.
BUG:112141


 M  +8 -0      katecmds.cpp  


--- branches/KDE/3.5/kdelibs/kate/part/katecmds.cpp #478257:478258
@@ -373,6 +373,7 @@
   // * the existing insertion is fine, including the line calculation.
 
   QStringList patterns = QStringList::split( QRegExp("(^\\\\n|(?![^\\\\])\\\\n)"), find, true );
+
   if ( patterns.count() > 1 )
   {
     for ( uint i = 0; i < patterns.count(); i++ )
@@ -488,6 +489,13 @@
   exchangeAbbrevs(replace);
    kdDebug(13025)<< "SedReplace: replace=" << replace.latin1() <<endl;
 
+   if ( find.contains("\\n") )
+   {
+     // FIXME i18n this message (or make it work)
+     msg = "Sorry, but we can't replace newlines (yet)";
+     return false;
+   }
+
   KateDocument *doc = ((KateView*)view)->doc();
   if ( ! doc ) return false;
 

I'm VERY sorry, but i put the wrong number in my commit message. My deepest apologizes

No problem Anders :)

SVN commit 557315 by dhaumann:

forward port SVN commit 478258 by alund:

Error message to inform users that we can't replace newlines.
Dont' attempt to, as it may lead to an infinite loop, again potentially leading to a crash at some point.
CCBUG:112141


 M  +7 -0      katecmds.cpp  


--- trunk/KDE/kdelibs/kate/part/katecmds.cpp #557314:557315
@@ -497,6 +497,13 @@
   exchangeAbbrevs(replace);
    kDebug(13025)<< "SedReplace: replace=" << replace <<endl;
 
+  if ( find.contains("\\n") )
+  {
+    // FIXME: make replacing newlines work
+    msg = i18n("Sorry, but Kate is not able to replace newlines, yet");
+    return false;
+  }
+
   KateDocument *doc = ((KateView*)view)->doc();
   if ( ! doc ) return false;
 

Uh, I shouldn't have blindly used the commit log message, same as Anders:
I'm VERY sorry, but i put the wrong number in my commit message. My deepest apologizes ;)

*** This bug has been marked as a duplicate of 114237 ***