diff options
| author | Philip Langdale <philipl@overt.org> | 2013-01-03 20:31:27 -0800 |
|---|---|---|
| committer | Philip Langdale <philipl@overt.org> | 2013-01-11 20:31:49 -0800 |
| commit | fdc8f9cc6e2bcc41bb110f111d8c6705354fde67 (patch) | |
| tree | b026500ba09fa13c8f4918ac9b34e6c0843b7d8e /daemon | |
| parent | 6f66c0d3b12a9fddf5f845f809dde35427f3c238 (diff) | |
| download | gvfs-fdc8f9cc6e2bcc41bb110f111d8c6705354fde67.tar.gz | |
MTP: Review Feedback: Use g_strv_length()
Diffstat (limited to 'daemon')
| -rw-r--r-- | daemon/gvfsbackendmtp.c | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/daemon/gvfsbackendmtp.c b/daemon/gvfsbackendmtp.c index 99ffbbab..f23b18e1 100644 --- a/daemon/gvfsbackendmtp.c +++ b/daemon/gvfsbackendmtp.c @@ -808,8 +808,7 @@ do_enumerate (GVfsBackend *backend, GFileInfo *info; gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); DEBUG ("(I) do_enumerate (filename = %s, n_elements = %d) ", filename, ne); @@ -880,8 +879,7 @@ do_query_info (GVfsBackend *backend, g_mutex_lock (&G_VFS_BACKEND_MTP (backend)->mutex); gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); LIBMTP_mtpdevice_t *device; device = G_VFS_BACKEND_MTP (backend)->device; @@ -967,8 +965,7 @@ do_query_fs_info (GVfsBackend *backend, g_mutex_lock (&G_VFS_BACKEND_MTP (backend)->mutex); gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); LIBMTP_mtpdevice_t *device; device = G_VFS_BACKEND_MTP (backend)->device; @@ -1035,8 +1032,7 @@ do_make_directory (GVfsBackend *backend, g_mutex_lock (&G_VFS_BACKEND_MTP (backend)->mutex); gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); if (ne < 3) { g_vfs_job_failed_literal (G_VFS_JOB (job), @@ -1086,8 +1082,7 @@ do_pull (GVfsBackend *backend, GFileInfo *info = NULL; gchar **elements = g_strsplit_set (source, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); if (ne < 3) { g_vfs_job_failed_literal (G_VFS_JOB (job), @@ -1161,8 +1156,7 @@ do_push (GVfsBackend *backend, GFile *file = NULL; GFileInfo *info = NULL; gchar **elements = g_strsplit_set (destination, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); if (ne < 3) { g_vfs_job_failed_literal (G_VFS_JOB (job), @@ -1258,8 +1252,7 @@ do_delete (GVfsBackend *backend, g_mutex_lock (&G_VFS_BACKEND_MTP (backend)->mutex); gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); if (ne < 3) { g_vfs_job_failed_literal (G_VFS_JOB (job), @@ -1300,8 +1293,7 @@ do_set_display_name (GVfsBackend *backend, g_mutex_lock (&G_VFS_BACKEND_MTP (backend)->mutex); gchar **elements = g_strsplit_set (filename, "/", -1); - unsigned int ne = 0; - for (ne = 0; elements[ne] != NULL; ne++); + unsigned int ne = g_strv_length (elements); if (ne < 3) { g_vfs_job_failed_literal (G_VFS_JOB (job), |