tree-wide: use FREE_AND_NULLIFY macro and reformat code

Jeez idk why i forgot the pass-by-value feature, so the original version
never gets the pointer nullified.

Fix it by using our favourite C-style macro.

4 files changed, 19 insertions, 19 deletions

diff --git a/src/extractors/bilibili.c b/src/extractors/bilibili.c
index 6112bef..6bddee5 100644
--- a/src/extractors/bilibili.c
+++ b/src/extractors/bilibili.c
@@ -345,7 +345,7 @@ static const char *id2quality_desc(int id) {
 
 static void multipage_cleanup(Multipage *multipage_struct) {
   for (unsigned short i = 0; i < multipage_struct->sections.n; i++) {
-    // free_and_nullify(multipage_struct->sections[i].episodes);
+    // FREE_AND_NULLIFY(multipage_struct->sections[i].episodes);
     Multi_episode_data *section = get_element(&multipage_struct->sections, i);
     free_array(&section->episodes);
   }
@@ -381,7 +381,7 @@ static int download(Bilibili_options *bilibili_options) {
   get(bilibili_options->api, &resp);
   if (get_dash(resp, &dash)) {
     LOG("Bilibili", "Get dash failed.");
-    free_and_nullify(resp);
+    FREE_AND_NULLIFY(resp);
     dash_cleanup(&dash);
     return 1;
   };
@@ -429,7 +429,7 @@ static int download(Bilibili_options *bilibili_options) {
     set_str_element(&callback_struct, 2, audiofn);
     add_url(audio->baseUrl, NULL, audiofn, &bilibili_merge, &callback_struct);
   }
-  free_and_nullify(resp);
+  FREE_AND_NULLIFY(resp);
   dash_cleanup(&dash);
   return 0;
 }
@@ -478,18 +478,18 @@ void bilibili_extract(struct options *options) {
   DEBUG_PRINT("title: %s\n", bilibili_options.title);
 
   if (generate_api(&bilibili_options, 127)) {
-    free_and_nullify(bilibili_options.api);
+    FREE_AND_NULLIFY(bilibili_options.api);
     multipage_cleanup(&multipage_struct);
     return;
   }
   DEBUG_PRINT("Generated API: %s\n", bilibili_options.api);
 
   if (download(&bilibili_options)) {
-    free_and_nullify(bilibili_options.api);
+    FREE_AND_NULLIFY(bilibili_options.api);
     multipage_cleanup(&multipage_struct);
     return;
   }
 
-  free_and_nullify(bilibili_options.api);
+  FREE_AND_NULLIFY(bilibili_options.api);
   multipage_cleanup(&multipage_struct);
 }
diff --git a/src/extractors/extractor.c b/src/extractors/extractor.c
index c38c1f2..06d52a5 100644
--- a/src/extractors/extractor.c
+++ b/src/extractors/extractor.c
@@ -12,10 +12,10 @@ Site_map site_map = {{{"www.bilibili.com", SITE_BILIBILI},
                      3};
 
 void options_cleanup(Options *options) {
-  free_and_nullify(options->URL);
-  free_and_nullify(options->path);
-  free_and_nullify(options->query);
-  free_and_nullify(options->pagedata);
+  FREE_AND_NULLIFY(options->URL);
+  FREE_AND_NULLIFY(options->path);
+  FREE_AND_NULLIFY(options->query);
+  FREE_AND_NULLIFY(options->pagedata);
 }
 
 int extract(void *v) {
diff --git a/src/extractors/haokan.c b/src/extractors/haokan.c
index f289c3e..b3e943d 100644
--- a/src/extractors/haokan.c
+++ b/src/extractors/haokan.c
@@ -38,8 +38,8 @@ void haokan_extract(Options *options) {
 
     end:
       free_str_array(&results);
-      free_and_nullify(filename);
-      free_and_nullify(videoURL);
+      FREE_AND_NULLIFY(filename);
+      FREE_AND_NULLIFY(videoURL);
       return;
     }
     free_str_array(&results);
diff --git a/src/extractors/youku.c b/src/extractors/youku.c
index e8fa0a5..091141d 100644
--- a/src/extractors/youku.c
+++ b/src/extractors/youku.c
@@ -67,9 +67,9 @@ static int parse_jsondata(cJSON *json, Youku_data *youku_data) {
 }
 
 static void youku_options_cleanup(Youku_options *youku_options) {
-  free_and_nullify(youku_options->vid);
-  free_and_nullify(youku_options->utid);
-  free_and_nullify(youku_options->api);
+  FREE_AND_NULLIFY(youku_options->vid);
+  FREE_AND_NULLIFY(youku_options->utid);
+  FREE_AND_NULLIFY(youku_options->api);
   cJSON_Delete(youku_options->json);
 }
 
@@ -147,13 +147,13 @@ void youku_extract(Options *options) {
   char *resp = NULL;
   if (get(youku_options.api, &resp)) {
     LOG("Youku", "Get API data failed.\n");
-    free_and_nullify(resp);
+    FREE_AND_NULLIFY(resp);
     goto end;
   }
   // DEBUG_PRINT("resp: %s\n", resp);
 
   youku_options.json = cJSON_Parse(resp);
-  free_and_nullify(resp);
+  FREE_AND_NULLIFY(resp);
   Youku_data youku_data = {0};
   if (parse_jsondata(youku_options.json, &youku_data)) {
     LOG("Youku", "Parse jsondata failed.\n");
@@ -170,8 +170,8 @@ void youku_extract(Options *options) {
           youku_data.height);
   set_str_element(&callback_struct, 1, filename);
   add_url(youku_data.m3u8_url, NULL, m3u8fn, youku_convert, &callback_struct);
-  free_and_nullify(m3u8fn);
-  free_and_nullify(filename);
+  FREE_AND_NULLIFY(m3u8fn);
+  FREE_AND_NULLIFY(filename);
 
 end:
   free_str_array(&results);