summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2019-03-30 16:33:15 +0100
committerandroid-build-merger <android-build-merger@google.com>2019-03-30 16:33:15 +0100
commiteffd4ca88305181028f13dffe3c936493c5beccb (patch)
treebca9e55c10ab5d09e5fb76cf9c71bc5ff8138cdd
parentMerge "Move install to separate module" (diff)
parentMerge changes Ibbe7084e,If584fc8a (diff)
downloadandroid_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar.gz
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar.bz2
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar.lz
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar.xz
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.tar.zst
android_bootable_recovery-effd4ca88305181028f13dffe3c936493c5beccb.zip
-rw-r--r--common.h11
-rw-r--r--recovery.cpp21
-rw-r--r--recovery_main.cpp6
-rw-r--r--recovery_ui/include/recovery_ui/device.h4
4 files changed, 9 insertions, 33 deletions
diff --git a/common.h b/common.h
index 22b2f0a0d..a524a4184 100644
--- a/common.h
+++ b/common.h
@@ -14,11 +14,7 @@
* limitations under the License.
*/
-#ifndef RECOVERY_COMMON_H
-#define RECOVERY_COMMON_H
-
-#include <stdarg.h>
-#include <stdio.h>
+#pragma once
#include <string>
@@ -39,9 +35,4 @@ extern std::string stage;
// The reason argument provided in "--reason=".
extern const char* reason;
-void ui_print(const char* format, ...) __printflike(1, 2);
-
bool is_ro_debuggable();
-
-bool SetUsbConfig(const std::string& state);
-#endif // RECOVERY_COMMON_H
diff --git a/recovery.cpp b/recovery.cpp
index 421bc12f2..034918498 100644
--- a/recovery.cpp
+++ b/recovery.cpp
@@ -24,7 +24,6 @@
#include <limits.h>
#include <linux/fs.h>
#include <linux/input.h>
-#include <stdarg.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
@@ -272,12 +271,6 @@ static bool erase_volume(const char* volume) {
return (result == 0);
}
-// Sets the usb config to 'state'
-bool SetUsbConfig(const std::string& state) {
- android::base::SetProperty("sys.usb.config", state);
- return android::base::WaitForProperty("sys.usb.state", state);
-}
-
static bool yes_no(Device* device, const char* question1, const char* question2) {
std::vector<std::string> headers{ question1, question2 };
std::vector<std::string> items{ " No", " Yes" };
@@ -746,20 +739,6 @@ static void print_property(const char* key, const char* name, void* /* cookie */
printf("%s=%s\n", key, name);
}
-void ui_print(const char* format, ...) {
- std::string buffer;
- va_list ap;
- va_start(ap, format);
- android::base::StringAppendV(&buffer, format, ap);
- va_end(ap);
-
- if (ui != nullptr) {
- ui->Print("%s", buffer.c_str());
- } else {
- fputs(buffer.c_str(), stdout);
- }
-}
-
static bool is_battery_ok(int* required_battery_level) {
using android::hardware::health::V1_0::BatteryStatus;
using android::hardware::health::V2_0::get_health_service;
diff --git a/recovery_main.cpp b/recovery_main.cpp
index 56ebe16fa..b41368d7b 100644
--- a/recovery_main.cpp
+++ b/recovery_main.cpp
@@ -178,6 +178,12 @@ static std::string load_locale_from_cache() {
return android::base::Trim(content);
}
+// Sets the usb config to 'state'.
+static bool SetUsbConfig(const std::string& state) {
+ android::base::SetProperty("sys.usb.config", state);
+ return android::base::WaitForProperty("sys.usb.state", state);
+}
+
static void ListenRecoverySocket(RecoveryUI* ui, std::atomic<Device::BuiltinAction>& action) {
android::base::unique_fd sock_fd(android_get_control_socket("recovery"));
if (sock_fd < 0) {
diff --git a/recovery_ui/include/recovery_ui/device.h b/recovery_ui/include/recovery_ui/device.h
index cfa914e77..3c44510c8 100644
--- a/recovery_ui/include/recovery_ui/device.h
+++ b/recovery_ui/include/recovery_ui/device.h
@@ -93,8 +93,8 @@ class Device {
// Performs a recovery action selected from the menu. 'menu_position' will be the index of the
// selected menu item, or a non-negative value returned from HandleMenuKey(). The menu will be
- // hidden when this is called; implementations can call ui_print() to print information to the
- // screen. If the menu position is one of the builtin actions, you can just return the
+ // hidden when this is called; implementations can call GetUI()->Print() to print information to
+ // the screen. If the menu position is one of the builtin actions, you can just return the
// corresponding enum value. If it is an action specific to your device, you actually perform it
// here and return NO_ACTION.
virtual BuiltinAction InvokeMenuItem(size_t menu_position);