Changeset 1314 for libcacaTweet

Timestamp:

Nov 9, 2006, 6:20:47 PM (16 years ago)

Author:

Sam Hocevar

Message:

• If not enough data is available for import, return 0 instead of an error.

File:

• libcaca/trunk/cucul/import.c (modified) (9 diffs)

libcaca/trunk/cucul/import.c

@@ -68 +68 @@
  *  Valid values for \c format are:
  *  - \c "": attempt to autodetect the file format.
+ *  - \c "caca": import native libcaca files.
  *  - \c "text": import ASCII text files.
  *  - \c "ansi": import ANSI files.
  *  - \c "utf8": import UTF-8 files with ANSI colour codes.
- *  - \c "caca": import native libcaca files.
+ *
+ *  The number of bytes read is returned. If the file format is valid, but
+ *  not enough data was available, 0 is returned.
  *
  *  If an error occurs, -1 is returned and \b errno is set accordingly:
@@ -81 +84 @@
  *  \param len The size in bytes of the memory area.
  *  \param format A string describing the input format.
- *  \return The number of bytes read, or -1 if an error occurred.
+ *  \return The number of bytes read, or 0 if there was not enough data,
+ *  or -1 if an error occurred.
  */
 long int cucul_import_memory(cucul_canvas_t *cv, void const *data,
@@ -129 +133 @@
  *  Valid values for \c format are:
  *  - \c "": attempt to autodetect the file format.
+ *  - \c "caca": import native libcaca files.
  *  - \c "text": import ASCII text files.
  *  - \c "ansi": import ANSI files.
  *  - \c "utf8": import UTF-8 files with ANSI colour codes.
- *  - \c "caca": import native libcaca files.
+ *
+ *  The number of bytes read is returned. If the file format is valid, but
+ *  not enough data was available, 0 is returned.
  *
  *  If an error occurs, -1 is returned and \b errno is set accordingly:
@@ -144 +151 @@
  *  \param filename The name of the file to load.
  *  \param format A string describing the input format.
- *  \return The number of bytes read, or -1 if an error occurred.
+ *  \return The number of bytes read, or 0 if there was not enough data,
+ *  or -1 if an error occurred.
  */
 long int cucul_import_file(cucul_canvas_t *cv, char const *filename,
@@ -202 +210 @@
 {
     static char const * const list[] =
-        {
-            "", "autodetect",
-            "text", "plain text",
-            "caca", "native libcaca format",
-            "ansi", "ANSI coloured text",
-            NULL, NULL
-        };
+    {
+        "", "autodetect",
+        "caca", "native libcaca format",
+        "text", "plain text",
+        "ansi", "ANSI coloured text",
+        "utf8", "UTF-8 files with ANSI colour codes",
+        NULL, NULL
+    };
 
     return list;
@@ -221 +230 @@
 {
     uint8_t const *buf = (uint8_t const *)data;
-    unsigned int control_size, data_size, full_size, frames, f, n;
+    unsigned int control_size, data_size, expected_size, frames, f, n;
     uint16_t version, flags;
 
-    cucul_set_canvas_size(cv, 0, 0);
-
     if(size < 20)
-        goto invalid_caca;
+        return 0;
 
     if(buf[0] != 0xca || buf[1] != 0xca || buf[2] != 'C' || buf[3] != 'V')
@@ -238 +245 @@
     flags = sscanu16(buf + 18);
 
-    if(size != 4 + control_size + data_size)
-        goto invalid_caca;
+    if(size < 4 + control_size + data_size)
+        return 0;
 
     if(control_size < 16 + frames * 24)
         goto invalid_caca;
 
-    for(full_size = 0, f = 0; f < frames; f++)
+    for(expected_size = 0, f = 0; f < frames; f++)
     {
         unsigned int width, height, duration;
@@ -257 +264 @@
         y = (int32_t)sscanu32(buf + 4 + 16 + f * 24 + 20);
 
-        full_size += width * height * 8;
-    }
-
-    if(full_size != data_size)
+        expected_size += width * height * 8;
+    }
+
+    if(expected_size != data_size)
         goto invalid_caca;
 
     /* FIXME: read all frames, not only the first one */
+    cucul_set_canvas_size(cv, 0, 0);
     cucul_set_canvas_size(cv, sscanu32(buf + 4 + 16),
                               sscanu32(buf + 4 + 16 + 4));
@@ -277 +285 @@
     cv->curattr = sscanu32(buf + 4 + 16 + 12);
 
-    return size;
+    return 4 + control_size + data_size;
 
 invalid_caca: