Replace '#warning' directive with '#pragma message'

If turn on '-Werror' option, '#warning' will be treated as an error,
which however cannot be suppressed even with "-Wno-error=cpp" option.

As a workaround, patch instead uses "#pragma message" which is information
only and is neither a warning nor an error.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612

Reviewed by: sns.park

Change-Id: Icfbe90bcd9d9817307704189755d413a808fcc22
Signed-off-by: SeungSeop Park <sns.park@samsung.com>

diff --git a/tizen_src/chromium_impl/content/browser/renderer_host/disambiguation_popup_efl.cc b/tizen_src/chromium_impl/content/browser/renderer_host/disambiguation_popup_efl.cc
index 3e8e3f60c2d8b4598dd20b1775928ad3939f098d..90241dc25634510d0faa48a2e7598396905a3d6c 100644 (file)
--- a/tizen_src/chromium_impl/content/browser/renderer_host/disambiguation_popup_efl.cc
+++ b/tizen_src/chromium_impl/content/browser/renderer_host/disambiguation_popup_efl.cc
@@ -97,7 +97,7 @@ void DisambiguationPopupEfl::Show(const gfx::Rect& target,
   content_image_ = evas_object_image_add(evas_object_evas_get(parent_view_));
   elm_object_content_set(popup_, content_image_);
 
-#warning "Add proper support for EFL Assist"
+#pragma message "Add proper support for EFL Assist"
 #if 0 //defined(OS_TIZEN_MOBILE)
   if (EflAssistHandle) {
     typedef void (*cb_type)(Evas_Object*, Ea_Callback_Type, Ea_Event_Cb func, void*);

diff --git a/tizen_src/chromium_impl/ui/base/clipboard/clipboard_helper_efl.cc b/tizen_src/chromium_impl/ui/base/clipboard/clipboard_helper_efl.cc
index 2b11eed915c099c11c9fb134a2672b05d237259b..8eded7f8e0c22f90cced70464da3a26c4954d60b 100644 (file)
--- a/tizen_src/chromium_impl/ui/base/clipboard/clipboard_helper_efl.cc
+++ b/tizen_src/chromium_impl/ui/base/clipboard/clipboard_helper_efl.cc
@@ -55,7 +55,7 @@ struct ClipData
   int bufferLength;
 
   ClipboardHelperEfl* clipboardHelper;
-#warning "TODO: Add proper delegate class for clipboard related functionality EWK needs."
+#pragma message "TODO: Add proper delegate class for clipboard related functionality EWK needs."
   EWebView* webView;
 };
 

diff --git a/tizen_src/ewk/efl_integration/browser/navigation_policy_handler_efl.cc b/tizen_src/ewk/efl_integration/browser/navigation_policy_handler_efl.cc
index 6914c97333903902a21e687ca07825505da6576d..7dde2b82b18d3ee35a9d191cdd64cdcbe5b6a5f6 100644 (file)
--- a/tizen_src/ewk/efl_integration/browser/navigation_policy_handler_efl.cc
+++ b/tizen_src/ewk/efl_integration/browser/navigation_policy_handler_efl.cc
@@ -24,7 +24,7 @@ NavigationPolicyHandlerEfl::~NavigationPolicyHandlerEfl() {
 void NavigationPolicyHandlerEfl::DownloadNavigation() {
   // indicate that we handle it
   if (SetDecision(Handled)) {
-#warning "[M37] RequestOpenURL was moved to WebContentsImpl, how to access it from here?"
+#pragma message "[M37] RequestOpenURL was moved to WebContentsImpl, how to access it from here?"
     //WindowOpenDisposition disposition = SAVE_TO_DISK;
     //rvh_->GetDelegate()->RequestOpenURL(rvh_, params_.url, params_.referrer,
     //    disposition, params_.frame_id, params_.is_redirect, false);

diff --git a/tizen_src/ewk/efl_integration/memory_purger.cc b/tizen_src/ewk/efl_integration/memory_purger.cc
index c6f063c28d75316a7edeb7a1e1dc7025770bdb92..1f79b4daff05421ea3924fb7d0deec1eb0540ef9 100644 (file)
--- a/tizen_src/ewk/efl_integration/memory_purger.cc
+++ b/tizen_src/ewk/efl_integration/memory_purger.cc
@@ -57,7 +57,7 @@ void PurgeMemoryIOHelper::AddRequestContextGetter(
 void PurgeMemoryIOHelper::PurgeMemoryOnIOThread() {
   // Ask ProxyServices to purge any memory they can (generally garbage in the
   // wrapped ProxyResolver's JS engine).
-#warning "[M37] Fix memory purge"
+#pragma message "[M37] Fix memory purge"
 #if 0
   for (size_t i = 0; i < request_context_getters_.size(); ++i) {
     request_context_getters_[i]->GetURLRequestContext()->proxy_service()->
@@ -80,13 +80,13 @@ void MemoryPurger::PurgeAll(BrowserContext* browser_context) {
 // static
 void MemoryPurger::PurgeBrowser(BrowserContext* browser_context) {
   // Dump the backing stores.
-#warning "[M37] backing stores were removed, so the code below is probably safe to remove, check this."
+#pragma message "[M37] backing stores were removed, so the code below is probably safe to remove, check this."
   //content::RenderWidgetHost::RemoveAllBackingStores();
   scoped_refptr<PurgeMemoryIOHelper> purge_memory_io_helper(
       new PurgeMemoryIOHelper());
   purge_memory_io_helper->AddRequestContextGetter(
         make_scoped_refptr(browser_context->GetRequestContext()));
-#warning "[M37] Another non existing function"
+#pragma message "[M37] Another non existing function"
   //BrowserContext::PurgeMemory(browser_context);
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,

diff --git a/tizen_src/ewk/efl_integration/public/ewk_settings.cc b/tizen_src/ewk/efl_integration/public/ewk_settings.cc
index d8dd663b832c651a073d951d8af79a1ad8d4a05c..98d20b057708a6877212d4b76d2638fe747ef228 100644 (file)
--- a/tizen_src/ewk/efl_integration/public/ewk_settings.cc
+++ b/tizen_src/ewk/efl_integration/public/ewk_settings.cc
@@ -39,7 +39,7 @@ void ewkUpdateWebkitPreferences(Evas_Object *ewkView)
 Eina_Bool ewk_settings_fullscreen_enabled_set(Ewk_Settings* settings, Eina_Bool enable)
 {
 #if !defined(EWK_BRINGUP)
-#warning "[M38] fullscreen_enabled flag has been deleted from webpreference.h"
+#pragma message "[M38] fullscreen_enabled flag has been deleted from webpreference.h"
   EINA_SAFETY_ON_NULL_RETURN_VAL(settings, false);
   settings->getPreferences().fullscreen_enabled = enable;
   ewkUpdateWebkitPreferences(settings->getEvasObject());
@@ -52,7 +52,7 @@ Eina_Bool ewk_settings_fullscreen_enabled_set(Ewk_Settings* settings, Eina_Bool
 Eina_Bool ewk_settings_fullscreen_enabled_get(const Ewk_Settings* settings)
 {
 #if !defined(EWK_BRINGUP)
-#warning "[M38] fullscreen_enabled flag has been deleted from webpreference.h"
+#pragma message "[M38] fullscreen_enabled flag has been deleted from webpreference.h"
   EINA_SAFETY_ON_NULL_RETURN_VAL(settings, false);
   return settings->getPreferences().fullscreen_enabled;
 #else