]> sigrok.org Git - libsigrok.git/blobdiff - session_file.c
Revert "Add sr_session_append(): add captured data to an existing session file"
[libsigrok.git] / session_file.c
index 6cd9b72025ad54a3146d40c8d1e856963d4fa23e..4838a6be00566b2f0fc7fb140697bc7f4b653b2c 100644 (file)
 extern struct sr_session *session;
 extern SR_PRIV struct sr_dev_driver session_driver;
 
-/** @private */
-SR_PRIV int sr_sessionfile_check(const char *filename)
+/**
+ * Load the session from the specified filename.
+ *
+ * @param filename The name of the session file to load. Must not be NULL.
+ *
+ * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments,
+ *         SR_ERR_MALLOC upon memory allocation errors, or SR_ERR upon
+ *         other errors.
+ */
+SR_API int sr_session_load(const char *filename)
 {
+       GKeyFile *kf;
+       GPtrArray *capturefiles;
        struct zip *archive;
        struct zip_file *zf;
        struct zip_stat zs;
-       int version, ret;
-       char s[11];
+       struct sr_dev_inst *sdi;
+       struct sr_probe *probe;
+       int ret, probenum, devcnt, version, i, j;
+       uint64_t tmp_u64, total_probes, enabled_probes, p;
+       char **sections, **keys, *metafile, *val, s[11];
+       char probename[SR_MAX_PROBENAME_LEN + 1];
 
-       if (!filename)
+       if (!filename) {
+               sr_err("%s: filename was NULL", __func__);
                return SR_ERR_ARG;
+       }
 
-       if (!(archive = zip_open(filename, 0, &ret)))
-               /* No logging: this can be used just to check if it's
-                * a sigrok session file or not. */
+       if (!(archive = zip_open(filename, 0, &ret))) {
+               sr_dbg("Failed to open session file: zip error %d", ret);
                return SR_ERR;
+       }
 
        /* check "version" */
        version = 0;
        if (!(zf = zip_fopen(archive, "version", 0))) {
-               sr_dbg("Not a sigrok session file: no version found.");
+               sr_dbg("Not a sigrok session file.");
                return SR_ERR;
        }
-       if ((ret = zip_fread(zf, s, 10)) == -1)
+       if ((ret = zip_fread(zf, s, 10)) == -1) {
+               sr_dbg("Not a valid sigrok session file.");
                return SR_ERR;
+       }
        zip_fclose(zf);
        s[ret] = 0;
        version = strtoull(s, NULL, 10);
        if (version != 1) {
-               sr_dbg("Cannot handle sigrok session file version %d.", version);
+               sr_dbg("Not a valid sigrok session file version.");
                return SR_ERR;
        }
 
@@ -90,41 +108,6 @@ SR_PRIV int sr_sessionfile_check(const char *filename)
                return SR_ERR;
        }
 
-       return SR_OK;
-}
-
-/**
- * Load the session from the specified filename.
- *
- * @param filename The name of the session file to load. Must not be NULL.
- *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments,
- *         SR_ERR_MALLOC upon memory allocation errors, or SR_ERR upon
- *         other errors.
- */
-SR_API int sr_session_load(const char *filename)
-{
-       GKeyFile *kf;
-       GPtrArray *capturefiles;
-       struct zip *archive;
-       struct zip_file *zf;
-       struct zip_stat zs;
-       struct sr_dev_inst *sdi;
-       struct sr_probe *probe;
-       int ret, probenum, devcnt, i, j;
-       uint64_t tmp_u64, total_probes, enabled_probes, p;
-       char **sections, **keys, *metafile, *val;
-       char probename[SR_MAX_PROBENAME_LEN + 1];
-
-       if ((ret = sr_sessionfile_check(filename)) != SR_OK)
-               return ret;
-
-       if (!(archive = zip_open(filename, 0, &ret)))
-               return SR_ERR;
-
-       if (zip_stat(archive, "metadata", 0, &zs) == -1)
-               return SR_ERR;
-
        if (!(metafile = g_try_malloc(zs.size))) {
                sr_err("%s: metafile malloc failed", __func__);
                return SR_ERR_MALLOC;
@@ -323,65 +306,4 @@ SR_API int sr_session_save(const char *filename, const struct sr_dev_inst *sdi,
        return SR_OK;
 }
 
-/**
- * Append data to an existing session file.
- *
- * @param filename The name of the filename to append to. Must not be NULL.
- * @param buf The data to be appended.
- * @param unitsize The number of bytes per sample.
- * @param units The number of samples.
- *
- * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments, or SR_ERR
- *         upon other errors.
- */
-SR_API int sr_session_append(const char *filename, unsigned char *buf,
-               int unitsize, int units)
-{
-       struct zip *archive;
-       struct zip_source *logicsrc;
-       zip_int64_t num_files;
-       int chunk_num, next_chunk_num, ret, i;
-       const char *entry_name;
-       char chunkname[16];
-
-       if ((ret = sr_sessionfile_check(filename)) != SR_OK)
-               return ret;
-
-       if (!(archive = zip_open(filename, 0, &ret)))
-               return SR_ERR;
-
-       next_chunk_num = 1;
-       num_files = zip_get_num_entries(archive, 0);
-       for (i = 0; i < num_files; i++) {
-               entry_name = zip_get_name(archive, i, 0);
-               if (strncmp(entry_name, "logic-1", 7))
-                       continue;
-               if (strlen(entry_name) == 7) {
-                       /* This file has no extra chunks, just a single "logic-1".
-                        * Rename it to "logic-1-1" * and continue with chunk 2. */
-                       if (zip_rename(archive, i, "logic-1-1") == -1) {
-                               sr_err("Failed to rename 'logic-1' to 'logic-1-1'.");
-                               return SR_ERR;
-                       }
-                       next_chunk_num = 2;
-                       break;
-               } else if (strlen(entry_name) > 8 && entry_name[7] == '-') {
-                       chunk_num = strtoull(entry_name + 8, NULL, 10);
-                       if (chunk_num >= next_chunk_num)
-                               next_chunk_num = chunk_num + 1;
-               }
-       }
-       snprintf(chunkname, 15, "logic-1-%d", next_chunk_num);
-       if (!(logicsrc = zip_source_buffer(archive, buf, units * unitsize, FALSE)))
-               return SR_ERR;
-       if (zip_add(archive, chunkname, logicsrc) == -1)
-               return SR_ERR;
-       if ((ret = zip_close(archive)) == -1) {
-               sr_info("error saving session file: %s", zip_strerror(archive));
-               return SR_ERR;
-       }
-
-       return SR_OK;
-}
-
 /** @} */