Better check for QString::isEmpty() than for length() == 0

elpaso · elpaso · commit f6c7cdb4c8ba · 2019-01-06T17:54:03.000+01:00
Not that it really makes any difference ...
diff --git a/src/gui/qgsconfigureshortcutsdialog.cpp b/src/gui/qgsconfigureshortcutsdialog.cpp
@@ -83,7 +83,7 @@ void QgsConfigureShortcutsDialog::populateActions()
       continue;
     }
 
-    if ( actionText.length() == 0 )
+    if ( actionText.isEmpty() )
     {
       continue;
     }

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ void QgsConfigureShortcutsDialog::populateActions()`
`83`	`83`	`continue;`
`84`	`84`	`}`
`85`	`85`
`86`		`- if ( actionText.length() == 0 )`
	`86`	`+ if ( actionText.isEmpty() )`
`87`	`87`	`{`
`88`	`88`	`continue;`
`89`	`89`	`}`