Fix inifinite loop while parsing subdirs.

Improved clarity of the original while loop.

Signed-off-by: Slávek Banko <slavek.banko@axis.cz>

1 files changed, 5 insertions, 3 deletions

diff --git a/krename/batchrenamer.cpp b/krename/batchrenamer.cpp
index 371a0da..98b2ef4 100644
--- a/krename/batchrenamer.cpp
+++ b/krename/batchrenamer.cpp
@@ -839,8 +839,11 @@ void BatchRenamer::parseSubdirs( data* f )
 
         // create the missing subdir now
         int i = 0;
-        TQString d = "";
-        while (!(d = dirs.section("/", i, i, TQString::SectionSkipEmpty)).isEmpty()) { // asignment here!
+        TQString d = TQString::null;
+        for (d = dirs.section("/", i, i, TQString::SectionSkipEmpty);
+             !d.isEmpty();
+             i++, d = dirs.section("/", i, i, TQString::SectionSkipEmpty))
+        {
             KURL url = f->dst.url;
             // it is important to unescape here
             // to support dirnames containing "&" or 
@@ -852,7 +855,6 @@ void BatchRenamer::parseSubdirs( data* f )
 
             f->dst.url.addPath( d );
             f->dst.directory.append( d + "/" );
-            i++;
         }
     }
 }