# HG changeset patch
# User bcoudurier
# Date 1262645899 0
# Node ID 324c7c25e4adcaf019833a7e8e0eedb2477f39d4
# Parent  cce2ae0f48a0b3502fc2e12f1b961a5329b7d425
simplify error handling in mov_read_default

diff -r cce2ae0f48a0 -r 324c7c25e4ad mov.c
--- a/mov.c	Mon Jan 04 22:51:38 2010 +0000
+++ b/mov.c	Mon Jan 04 22:58:19 2010 +0000
@@ -181,11 +181,10 @@
     int64_t total_size = 0;
     MOVAtom a;
     int i;
-    int err = 0;
 
     if (atom.size < 0)
         atom.size = INT64_MAX;
-    while(((total_size + 8) < atom.size) && !url_feof(pb) && !err) {
+    while(((total_size + 8) < atom.size) && !url_feof(pb)) {
         int (*parse)(MOVContext*, ByteIOContext*, MOVAtom) = NULL;
         a.size = atom.size;
         a.type=0;
@@ -226,7 +225,9 @@
         } else {
             int64_t start_pos = url_ftell(pb);
             int64_t left;
-            err = parse(c, pb, a);
+            int err = parse(c, pb, a);
+            if (err < 0)
+                return err;
             if (url_is_streamed(pb) && c->found_moov && c->found_mdat)
                 break;
             left = a.size - url_ftell(pb) + start_pos;
@@ -237,10 +238,10 @@
         total_size += a.size;
     }
 
-    if (!err && total_size < atom.size && atom.size < 0x7ffff)
+    if (total_size < atom.size && atom.size < 0x7ffff)
         url_fskip(pb, atom.size - total_size);
 
-    return err;
+    return 0;
 }
 
 static int mov_read_dref(MOVContext *c, ByteIOContext *pb, MOVAtom atom)