feat: add some error handling wrt exit codes

Author: techno-disaster

lib/bloc/process_bloc/process_bloc.dart

@@ -56,14 +56,18 @@ class ProcessBloc extends Bloc<ProcessEvent, ProcessState> {
     );
     unawaited(_extractor
         .extractFile(
-          file,
-          listenProgress: (progress) =>
-              add(ProcessFileExtractorProgress(progress)),
-          listenOutput: (line) => add(ProcessFileExtractorOutput(line)),
-          listenVideoDetails: (videoDetails) =>
-              add(ProcessFileVideoDetails(videoDetails)),
-        )
-        .then((value) => add(ProcessFileComplete(file))));
+      file,
+      listenProgress: (progress) => add(ProcessFileExtractorProgress(progress)),
+      listenOutput: (line) => add(ProcessFileExtractorOutput(line)),
+      listenVideoDetails: (videoDetails) =>
+          add(ProcessFileVideoDetails(videoDetails)),
+    )
+        .then((value) {
+      if (value != 0) {
+        add(ProcessError(value));
+      }
+      add(ProcessFileComplete(file));
+    }));
   }
 
   @override
@@ -164,6 +168,8 @@ class ProcessBloc extends Bloc<ProcessEvent, ProcessState> {
         current: state.current,
         version: ccxVersion,
       );
+    } else if (event is ProcessError) {
+      yield state.copyWith(current: state.current, exitCode: event.exitCode);
     }
   }
 }

lib/bloc/process_bloc/process_event.dart

@@ -55,4 +55,9 @@ class ProcessFileComplete extends ProcessEvent {
   const ProcessFileComplete(this.file);
 }
 
+class ProcessError extends ProcessEvent {
+  final int exitCode;
+  const ProcessError(this.exitCode);
+}
+
 class GetCCExtractorVersion extends ProcessEvent {}

lib/bloc/process_bloc/process_state.dart

@@ -10,6 +10,7 @@ class ProcessState extends Equatable {
   final XFile? current;
   final bool started;
   final String? version;
+  final int? exitCode;
   const ProcessState({
     required this.orignalList,
     required this.queue,
@@ -20,6 +21,7 @@ class ProcessState extends Equatable {
     required this.progress,
     required this.current,
     required this.version,
+    this.exitCode,
   });
 
   ProcessState copyWith({
@@ -32,18 +34,19 @@ class ProcessState extends Equatable {
     String? progress,
     String? version,
     required XFile? current,
+    int? exitCode,
   }) =>
       ProcessState(
-        orignalList: orignalList ?? this.orignalList,
-        queue: queue ?? this.queue,
-        processed: processed ?? this.processed,
-        log: log ?? this.log,
-        started: started ?? this.started,
-        progress: progress ?? this.progress,
-        videoDetails: videoDetails ?? this.videoDetails,
-        version: version ?? this.version,
-        current: current,
-      );
+          orignalList: orignalList ?? this.orignalList,
+          queue: queue ?? this.queue,
+          processed: processed ?? this.processed,
+          log: log ?? this.log,
+          started: started ?? this.started,
+          progress: progress ?? this.progress,
+          videoDetails: videoDetails ?? this.videoDetails,
+          version: version ?? this.version,
+          current: current,
+          exitCode: exitCode ?? this.exitCode);
 
   @override
   List<Object?> get props => [
@@ -56,5 +59,6 @@ class ProcessState extends Equatable {
         videoDetails,
         orignalList,
         version,
+        exitCode,
       ];
 }

lib/repositories/ccextractor.dart

@@ -39,11 +39,11 @@ class CCExtractor {
     );
 
     process.stdout.transform(latin1.decoder).listen((update) {
-      print(update);
+      //print(update);
     });
 
     process.stderr.transform(latin1.decoder).listen((update) {
-      print(update);
+      // print(update);
       if (progressRegx.hasMatch(update)) {
         for (RegExpMatch i in progressRegx.allMatches(update)) {
           listenProgress(i[1]!);
@@ -83,4 +83,24 @@ class CCExtractor {
     });
     return ccxStdOut;
   }
+
+  static Map<int, String> exitCodes = {
+    0: 'EXIT_OK',
+    2: 'EXIT_NO_INPUT_FILES',
+    3: 'EXIT_TOO_MANY_INPUT_FILES',
+    4: 'EXIT_INCOMPATIBLE_PARAMETERS',
+    6: 'EXIT_UNABLE_TO_DETERMINE_FILE_SIZE',
+    7: 'EXIT_MALFORMED_PARAMETER',
+    8: 'EXIT_READ_ERROR',
+    10: 'EXIT_NO_CAPTIONS',
+    300: 'EXIT_NOT_CLASSIFIED',
+    501: 'EXIT_ERROR_IN_CAPITALIZATION_FILE',
+    502: 'EXIT_BUFFER_FULL',
+    1001: 'EXIT_MISSING_ASF_HEADER',
+    1002: 'EXIT_MISSING_RCWT_HEADER',
+    5: 'CCX_COMMON_EXIT_FILE_CREATION_FAILED',
+    9: 'CCX_COMMON_EXIT_UNSUPPORTED',
+    500: 'EXIT_NOT_ENOUGH_MEMORY',
+    1000: 'CCX_COMMON_EXIT_BUG_BUG',
+  };
 }

lib/screens/dashboard/dashboard.dart

@@ -1,3 +1,4 @@
+import 'package:ccxgui/repositories/ccextractor.dart';
 import 'package:flutter/material.dart';
 
 import 'package:flutter_bloc/flutter_bloc.dart';
@@ -50,7 +51,14 @@ class Dashboard extends StatelessWidget {
 class ClearFilesButton extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
-    return BlocBuilder<ProcessBloc, ProcessState>(
+    return BlocConsumer<ProcessBloc, ProcessState>(
+      listenWhen: (previous, current) {
+        return previous.exitCode != current.exitCode;
+      },
+      listener: (context, processState) {
+        CustomSnackBarMessage.show(
+            context, CCExtractor.exitCodes[processState.exitCode]!);
+      },
       builder: (context, processState) {
         return BlocBuilder<DashboardBloc, DashboardState>(
           builder: (context, dashboardState) {