Order of conditions in Forever loop is wrong

[ScriptDevil]

back-link: ch02/index.html#console-application

forever {
        // test for empty string 'QString("")'
        if(line.isEmpty()) { continue;   }

        // test for null string 'String()'
        if(line.isNull()) { break;   }
}

should in reality be

forever {
        // test for null string 'QString()'
        if(line.isNull()) { break;   }

        // test for empty string 'QString("")'
        if(line.isEmpty()) { continue;   }
}

since isEmpty returns true even when the string is null. Your version would lead to an infinite loop. Further it should be QString() and not String().