From 758c2171204615614b7bb1b93b482ca56b205d48 Mon Sep 17 00:00:00 2001 From: Romain Goyet Date: Thu, 10 Oct 2019 16:49:46 +0200 Subject: [PATCH] [ion] ion_main takes const arguments --- apps/code/app.cpp | 2 +- apps/code/app.h | 2 +- apps/main.cpp | 4 ++-- escher/include/escher/app.h | 2 +- ion/include/ion.h | 2 +- ion/src/simulator/shared/main_headless.cpp | 1 - ion/src/simulator/shared/main_sdl.cpp | 2 +- quiz/src/runner.cpp | 2 +- 8 files changed, 8 insertions(+), 9 deletions(-) diff --git a/apps/code/app.cpp b/apps/code/app.cpp index a3ba01a1e4e..b446187e876 100644 --- a/apps/code/app.cpp +++ b/apps/code/app.cpp @@ -44,7 +44,7 @@ bool App::Snapshot::lockOnConsole() const { return m_lockOnConsole; } -void App::Snapshot::setOpt(const char * name, char * value) { +void App::Snapshot::setOpt(const char * name, const char * value) { if (strcmp(name, "script") == 0) { m_scriptStore.deleteAllScripts(); char * separator = const_cast(UTF8Helper::CodePointSearch(value, ':')); diff --git a/apps/code/app.h b/apps/code/app.h index 5b3e658b5a4..f683b5bbb42 100644 --- a/apps/code/app.h +++ b/apps/code/app.h @@ -28,7 +28,7 @@ class App : public Shared::InputEventHandlerDelegateApp { ScriptStore * scriptStore(); #if EPSILON_GETOPT bool lockOnConsole() const; - void setOpt(const char * name, char * value) override; + void setOpt(const char * name, const char * value) override; #endif private: #if EPSILON_GETOPT diff --git a/apps/main.cpp b/apps/main.cpp index ed4d56e8973..8b5a13e6169 100644 --- a/apps/main.cpp +++ b/apps/main.cpp @@ -5,7 +5,7 @@ #define DUMMY_MAIN 0 #if DUMMY_MAIN -void ion_main(int argc, char * argv[]) { +void ion_main(int argc, const char * const argv[]) { // Initialize the backlight Ion::Backlight::init(); while (1) { @@ -18,7 +18,7 @@ void ion_main(int argc, char * argv[]) { #else -void ion_main(int argc, char * argv[]) { +void ion_main(int argc, const char * const argv[]) { // Initialize Poincare::TreePool::sharedPool Poincare::Init(); diff --git a/escher/include/escher/app.h b/escher/include/escher/app.h index 18e7e76a965..dd437a5decf 100644 --- a/escher/include/escher/app.h +++ b/escher/include/escher/app.h @@ -38,7 +38,7 @@ class App : public Responder { virtual void storageDidChangeForRecord(Ion::Storage::Record) {} virtual Descriptor * descriptor() = 0; #if EPSILON_GETOPT - virtual void setOpt(const char * name, char * value) {} + virtual void setOpt(const char * name, const char * value) {} #endif /* tidy clean all dynamically-allocated data */ virtual void tidy(); diff --git a/ion/include/ion.h b/ion/include/ion.h index f9ae2f129a3..f4686a6923b 100644 --- a/ion/include/ion.h +++ b/ion/include/ion.h @@ -23,7 +23,7 @@ * Don't worry though, once all its initialization will be performed, ION will * jump to your code at ion_main, which you have to implement yourself. */ -void ion_main(int argc, char * argv[]); +void ion_main(int argc, const char * const argv[]); namespace Ion { diff --git a/ion/src/simulator/shared/main_headless.cpp b/ion/src/simulator/shared/main_headless.cpp index 0329d571878..8dd954729e6 100644 --- a/ion/src/simulator/shared/main_headless.cpp +++ b/ion/src/simulator/shared/main_headless.cpp @@ -20,7 +20,6 @@ char heap[kHeapSize]; extern "C" { char * _heap_start = (char *)heap; char * _heap_end = _heap_start+kHeapSize; - int main(int argc, char * argv[]); } void Ion::Timing::msleep(uint32_t ms) { diff --git a/ion/src/simulator/shared/main_sdl.cpp b/ion/src/simulator/shared/main_sdl.cpp index 0b8f00cc011..78c3fa00a3f 100644 --- a/ion/src/simulator/shared/main_sdl.cpp +++ b/ion/src/simulator/shared/main_sdl.cpp @@ -17,7 +17,7 @@ void Ion::Timing::msleep(uint32_t ms) { } int main(int argc, char * argv[]) { - std::vector arguments(argv, argv + argc); + std::vector arguments(argv, argv + argc); char * language = IonSimulatorGetLanguageCode(); if (language != nullptr) { diff --git a/quiz/src/runner.cpp b/quiz/src/runner.cpp index 892d7bfb765..bc61d67e31e 100644 --- a/quiz/src/runner.cpp +++ b/quiz/src/runner.cpp @@ -26,7 +26,7 @@ static inline void ion_main_inner() { quiz_print("ALL TESTS FINISHED"); } -void ion_main(int argc, char * argv[]) { +void ion_main(int argc, const char * const argv[]) { // Initialize the backlight Ion::Backlight::init(); // Initialize Poincare::TreePool::sharedPool