refactor: convert atoi to strint_to_*

Author: portasynthinca3

applications/main/infrared/infrared_cli.c

@@ -178,8 +178,11 @@ static bool infrared_cli_parse_raw(const char* str, InfraredSignal* signal) {
     }
 
     uint32_t* timings = malloc(sizeof(uint32_t) * MAX_TIMINGS_AMOUNT);
-    uint32_t frequency = atoi(frequency_str);
-    float duty_cycle = (float)atoi(duty_cycle_str) / 100;
+    uint32_t frequency;
+    uint8_t duty_cycle_u8;
+    if(!strint_to_uint32(frequency_str, NULL, &frequency, 10)) return false;
+    if(!strint_to_uint8(duty_cycle_str, NULL, &duty_cycle_u8, 10)) return false;
+    float duty_cycle = duty_cycle_u8 / 100.0f;
 
     str += strlen(frequency_str) + strlen(duty_cycle_str) + INFRARED_CLI_BUF_SIZE;
 

lib/subghz/subghz_file_encoder_worker.c

@@ -4,6 +4,7 @@
 #include <flipper_format/flipper_format.h>
 #include <flipper_format/flipper_format_i.h>
 #include <lib/subghz/devices/devices.h>
+#include <lib/toolbox/strint.h>
 
 #define TAG "SubGhzFileEncoderWorker"
 
@@ -45,27 +46,26 @@ void subghz_file_encoder_worker_add_level_duration(
 }
 
 bool subghz_file_encoder_worker_data_parse(SubGhzFileEncoderWorker* instance, const char* strStart) {
-    char* str1;
-    bool res = false;
     // Line sample: "RAW_Data: -1, 2, -2..."
 
-    // Look for a key in the line
-    str1 = strstr(strStart, "RAW_Data: ");
+    // Look for the key in the line
+    char* str = strstr(strStart, "RAW_Data: ");
+    bool res = false;
 
-    if(str1 != NULL) {
+    if(str) {
         // Skip key
-        str1 = strchr(str1, ' ');
-
-        // Check that there is still an element in the line
-        while(strchr(str1, ' ') != NULL) {
-            str1 = strchr(str1, ' ');
+        str = strchr(str, ' ');
 
-            // Skip space
-            str1 += 1;
-            subghz_file_encoder_worker_add_level_duration(instance, atoi(str1));
+        // Parse next element
+        int32_t duration;
+        while(strint_to_int32(str, (char**)&str, &duration, 10) == StrintParseNoError) {
+            subghz_file_encoder_worker_add_level_duration(instance, duration);
+            if(*str == ',') str++; // could also be `\0`
         }
+
         res = true;
     }
+
     return res;
 }
 

lib/update_util/resources/manifest.c

@@ -1,6 +1,7 @@
 #include "manifest.h"
 
 #include <toolbox/stream/buffered_file_stream.h>
+#include <toolbox/strint.h>
 #include <toolbox/hex.h>
 
 struct ResourceManifestReader {
@@ -97,7 +98,12 @@ ResourceManifestEntry* resource_manifest_reader_next(ResourceManifestReader* res
             furi_string_right(
                 resource_manifest->linebuf, sizeof(resource_manifest->entry.hash) * 2 + 1);
 
-            resource_manifest->entry.size = atoi(furi_string_get_cstr(resource_manifest->linebuf));
+            if(strint_to_uint32(
+                   furi_string_get_cstr(resource_manifest->linebuf),
+                   NULL,
+                   &resource_manifest->entry.size,
+                   10) != StrintParseNoError)
+                break;
 
             /* Remove size */
             size_t offs = furi_string_search_char(resource_manifest->linebuf, ':');