adb: make local_build_list match remote_build_list.
local_build_list previously was returning an int, 0 on success and -1 on
failure, while remote_build_list was returning a bool, true on success
and false on failure.
Change-Id: Iced6c4142e2f843048d81c4e133d6b6dc75a35dd
diff --git a/adb/file_sync_client.cpp b/adb/file_sync_client.cpp
index 2262a93..44f3cfd 100644
--- a/adb/file_sync_client.cpp
+++ b/adb/file_sync_client.cpp
@@ -495,13 +495,14 @@
return name[0] == '.' && (name[1] == '\0' || (name[1] == '.' && name[2] == '\0'));
}
-static int local_build_list(SyncConnection& sc, std::vector<copyinfo>* filelist,
- const std::string& lpath, const std::string& rpath) {
+static bool local_build_list(SyncConnection& sc, std::vector<copyinfo>* filelist,
+ const std::string& lpath,
+ const std::string& rpath) {
std::vector<copyinfo> dirlist;
std::unique_ptr<DIR, int (*)(DIR*)> dir(opendir(lpath.c_str()), closedir);
if (!dir) {
sc.Error("cannot open '%s': %s", lpath.c_str(), strerror(errno));
- return -1;
+ return false;
}
dirent* de;
@@ -537,7 +538,7 @@
local_build_list(sc, filelist, ci.src.c_str(), ci.dst.c_str());
}
- return 0;
+ return true;
}
static bool copy_local_dir_remote(SyncConnection& sc, std::string lpath,
@@ -558,7 +559,7 @@
std::vector<copyinfo> filelist;
int pushed = 0;
int skipped = 0;
- if (local_build_list(sc, &filelist, lpath, rpath)) {
+ if (!local_build_list(sc, &filelist, lpath, rpath)) {
return false;
}