Skip to content

Commit f986a25

Browse files
authored
[multi-packages] Fix issues from static analysis (#264)
* Missing break * Uninitialized field * Incorrect comparison Signed-off-by: MuHong Byun <[email protected]>
1 parent 4a99a6c commit f986a25

File tree

5 files changed

+7
-6
lines changed

5 files changed

+7
-6
lines changed

packages/battery/tizen/src/battery_tizen_plugin.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ class BatteryTizenPlugin : public flutter::Plugin {
194194
std::unique_ptr<flutter::EventChannel<flutter::EncodableValue>>
195195
event_channel_;
196196
std::unique_ptr<flutter::EventSink<flutter::EncodableValue>> events_;
197-
bool is_full_;
197+
bool is_full_ = false;
198198
};
199199

200200
void BatteryTizenPluginRegisterWithRegistrar(

packages/battery_plus/tizen/src/battery_plus_tizen_plugin.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ class BatteryPlusTizenPlugin : public flutter::Plugin {
194194
std::unique_ptr<flutter::EventChannel<flutter::EncodableValue>>
195195
event_channel_;
196196
std::unique_ptr<flutter::EventSink<flutter::EncodableValue>> events_;
197-
bool is_full_;
197+
bool is_full_ = false;
198198
};
199199

200200
void BatteryPlusTizenPluginRegisterWithRegistrar(

packages/camera/tizen/src/camera_device.cc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@ static RecorderOrientationTag ChooseRecorderOrientationTag(
9494
break;
9595
case OrientationType::kLandscapeRight:
9696
tag = RecorderOrientationTag::kNone;
97+
break;
9798
default:
9899
LOG_ERROR("Unknown RecorderOrientationTag!");
99100
break;

packages/image_picker/tizen/src/image_resize.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ class ImageResize {
1919
image_util_image_h& dst_image);
2020
bool EncodeImage(image_util_encode_h encode_h, image_util_image_h dst_image,
2121
image_util_type_e encoder_type, const std::string& dst_file);
22-
unsigned int max_width_;
23-
unsigned int max_height_;
24-
int quality_;
22+
unsigned int max_width_ = 0;
23+
unsigned int max_height_ = 0;
24+
int quality_ = 0;
2525
};
2626

2727
#endif

packages/webview_flutter/tizen/src/webview.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ void WebView::ApplySettings(flutter::EncodableMap settings) {
323323
// NOTE: Not supported by LWE on Tizen.
324324
} else if ("gestureNavigationEnabled" == k) {
325325
// NOTE: Not supported by LWE on Tizen.
326-
} else if ("allowsInlineMediaPlayback") {
326+
} else if ("allowsInlineMediaPlayback" == k) {
327327
// no-op inline media playback is always allowed on Tizen.
328328
} else if ("userAgent" == k) {
329329
if (std::holds_alternative<std::string>(val)) {

0 commit comments

Comments
 (0)