X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=aft.c;h=6b74b0ae499b1c2fc60e0757d5876b7d90c6f693;hp=e5409244c5a626204cb1e645ae52b903f2a49448;hb=c00d7e25de7006d6f88aafb2b485057377a70324;hpb=732bf143e456df7fc2e845884fbbdfdaf3fafebc

diff --git a/aft.c b/aft.c
index e5409244..6b74b0ae 100644
--- a/aft.c
+++ b/aft.c
@@ -9,6 +9,7 @@
 #include <dirent.h> /* readdir() */
 #include <openssl/rc4.h>
 
+#include <osl.h>
 #include "para.h"
 #include "error.h"
 #include "crypt.h"
@@ -26,6 +27,8 @@
 #include "portable_io.h"
 
 static struct osl_table *audio_file_table;
+static char *status_items;
+static char *parser_friendly_status_items;
 
 /** The different sorting methods of the ls command. */
 enum ls_sorting_method {
@@ -66,7 +69,9 @@ enum ls_listing_mode {
 	/** -lm */
 	LS_MODE_MBOX,
 	/** -lc */
-	LS_MODE_CHUNKS
+	LS_MODE_CHUNKS,
+	/** -lp */
+	LS_MODE_PARSER,
 };
 
 /** The flags accepted by the ls command. */
@@ -223,12 +228,27 @@ enum audio_file_table_columns {
 	NUM_AFT_COLUMNS
 };
 
+/**
+ * Compare two osl objects pointing to hash values.
+ *
+ * \param obj1 Pointer to the first hash object.
+ * \param obj2 Pointer to the second hash object.
+ *
+ * \return The values required for an osl compare function.
+ *
+ * \sa osl_compare_func, uint32_compare().
+ */
+int aft_hash_compare(const struct osl_object *obj1, const struct osl_object *obj2)
+{
+	return hash_compare((HASH_TYPE *)obj1->data, (HASH_TYPE *)obj2->data);
+}
+
 static struct osl_column_description aft_cols[] = {
 	[AFTCOL_HASH] = {
 		.storage_type = OSL_MAPPED_STORAGE,
 		.storage_flags = OSL_RBTREE | OSL_FIXED_SIZE | OSL_UNIQUE,
 		.name = "hash",
-		.compare_function = osl_hash_compare,
+		.compare_function = aft_hash_compare,
 		.data_size = HASH_SIZE
 	},
 	[AFTCOL_PATH] = {
@@ -341,23 +361,29 @@ enum afhi_offsets {
 	CHUNK_TV_TV_USEC_OFFSET = 36,
 	/** Number of channels is stored here. (1 byte) */
 	AFHI_CHANNELS_OFFSET = 40,
-	/** EOF timeout in ms. (2 byte) */
-	AFHI_EOF_OFFSET = 41,
 	/** The tag info position. */
-	AFHI_INFO_STRING_OFFSET = 43,
+	AFHI_INFO_STRING_OFFSET = 41,
 	/** Minimal on-disk size of a valid afhi struct. */
-	MIN_AFHI_SIZE = 44
+	MIN_AFHI_SIZE = 47, /* at least 6 null bytes for techinfo/tags */
 };
 
 static unsigned sizeof_afhi_buf(const struct afh_info *afhi)
 {
 	if (!afhi)
 		return 0;
-	return strlen(afhi->info_string) + MIN_AFHI_SIZE;
+	return MIN_AFHI_SIZE
+		+ strlen(afhi->techinfo)
+		+ strlen(afhi->tags.artist)
+		+ strlen(afhi->tags.title)
+		+ strlen(afhi->tags.year)
+		+ strlen(afhi->tags.album)
+		+ strlen(afhi->tags.comment);
 }
 
 static void save_afhi(struct afh_info *afhi, char *buf)
 {
+	char *p;
+
 	if (!afhi)
 		return;
 	write_u32(buf + AFHI_SECONDS_TOTAL_OFFSET, afhi->seconds_total);
@@ -371,8 +397,14 @@ static void save_afhi(struct afh_info *afhi, char *buf)
 	write_u32(buf + HEADER_OFFSET_OFFSET, afhi->header_offset);
 	write_u32(buf + CHUNK_TV_TV_SEC_OFFSET, afhi->chunk_tv.tv_sec);
 	write_u32(buf + CHUNK_TV_TV_USEC_OFFSET, afhi->chunk_tv.tv_usec);
-	write_u16(buf + AFHI_EOF_OFFSET, tv2ms(&afhi->eof_tv));
-	strcpy(buf + AFHI_INFO_STRING_OFFSET, afhi->info_string); /* OK */
+	p = buf + AFHI_INFO_STRING_OFFSET;
+	/* The sprintf's below are OK as our caller made sure that buf is large enough */
+	p += sprintf(p, "%s", afhi->techinfo) + 1;
+	p += sprintf(p, "%s", afhi->tags.artist) + 1;
+	p += sprintf(p, "%s", afhi->tags.title) + 1;
+	p += sprintf(p, "%s", afhi->tags.year) + 1;
+	p += sprintf(p, "%s", afhi->tags.album) + 1;
+	sprintf(p, "%s", afhi->tags.comment);
 }
 
 static void load_afhi(const char *buf, struct afh_info *afhi)
@@ -388,8 +420,12 @@ static void load_afhi(const char *buf, struct afh_info *afhi)
 	afhi->header_offset = read_u32(buf + HEADER_OFFSET_OFFSET);
 	afhi->chunk_tv.tv_sec = read_u32(buf + CHUNK_TV_TV_SEC_OFFSET);
 	afhi->chunk_tv.tv_usec = read_u32(buf + CHUNK_TV_TV_USEC_OFFSET);
-	ms2tv(read_u16(buf + AFHI_EOF_OFFSET), &afhi->eof_tv);
-	afhi->info_string = para_strdup(buf + AFHI_INFO_STRING_OFFSET);
+	afhi->techinfo = (char *)buf + AFHI_INFO_STRING_OFFSET;
+	afhi->tags.artist = afhi->techinfo + strlen(afhi->techinfo) + 1;
+	afhi->tags.title = afhi->tags.artist + strlen(afhi->tags.artist) + 1;
+	afhi->tags.year = afhi->tags.title + strlen(afhi->tags.title) + 1;
+	afhi->tags.album = afhi->tags.year + strlen(afhi->tags.year) + 1;
+	afhi->tags.comment = afhi->tags.album + strlen(afhi->tags.album) + 1;
 }
 
 static unsigned sizeof_chunk_table(struct afh_info *afhi)
@@ -422,27 +458,27 @@ static void load_chunk_table(struct afh_info *afhi, char *buf)
  * \param path The full path of the audio file.
  * \param row Result pointer.
  *
- * \return The return value of the underlying call to osl_get_row().
+ * \return Standard.
  */
 int aft_get_row_of_path(const char *path, struct osl_row **row)
 {
 	struct osl_object obj = {.data = (char *)path, .size = strlen(path) + 1};
 
-	return osl_get_row(audio_file_table, AFTCOL_PATH, &obj, row);
+	return osl(osl_get_row(audio_file_table, AFTCOL_PATH, &obj, row));
 }
 
 /**
  * Get the row of the audio file table corresponding to the given hash value.
  *
  * \param hash The hash value of the desired audio file.
- * \param row resul pointer.
+ * \param row Result pointer.
  *
- * \return The return value of the underlying call to osl_get_row().
+ * \return Standard.
  */
 int aft_get_row_of_hash(HASH_TYPE *hash, struct osl_row **row)
 {
 	const struct osl_object obj = {.data = hash, .size = HASH_SIZE};
-	return osl_get_row(audio_file_table, AFTCOL_HASH, &obj, row);
+	return osl(osl_get_row(audio_file_table, AFTCOL_HASH, &obj, row));
 }
 
 /**
@@ -451,11 +487,11 @@ int aft_get_row_of_hash(HASH_TYPE *hash, struct osl_row **row)
  * \param row Pointer to a row in the audio file table.
  * \param obj Result pointer.
  *
- * \return The return value of the underlying call to osl_get_object().
+ * \return Standard.
  */
 int get_afsi_object_of_row(const struct osl_row *row, struct osl_object *obj)
 {
-	return osl_get_object(audio_file_table, row, AFTCOL_AFSI, obj);
+	return osl(osl_get_object(audio_file_table, row, AFTCOL_AFSI, obj));
 }
 
 /**
@@ -524,8 +560,8 @@ int get_afsi_of_path(const char *path, struct afs_info *afsi)
 int get_audio_file_path_of_row(const struct osl_row *row, char **path)
 {
 	struct osl_object path_obj;
-	int ret = osl_get_object(audio_file_table, row, AFTCOL_PATH,
-		&path_obj);
+	int ret = osl(osl_get_object(audio_file_table, row, AFTCOL_PATH,
+		&path_obj));
 	if (ret < 0)
 		return ret;
 	*path = path_obj.data;
@@ -542,9 +578,10 @@ int get_audio_file_path_of_row(const struct osl_row *row, char **path)
  *
  * \sa get_hash_of_row().
  */
-static int get_hash_object_of_aft_row(const struct osl_row *row, struct osl_object *obj)
+static int get_hash_object_of_aft_row(const struct osl_row *row,
+		struct osl_object *obj)
 {
-	return osl_get_object(audio_file_table, row, AFTCOL_HASH, obj);
+	return osl(osl_get_object(audio_file_table, row, AFTCOL_HASH, obj));
 }
 
 /**
@@ -582,8 +619,8 @@ static int get_hash_of_row(const struct osl_row *row, HASH_TYPE **hash)
 int get_afhi_of_row(const struct osl_row *row, struct afh_info *afhi)
 {
 	struct osl_object obj;
-	int ret = osl_get_object(audio_file_table, row, AFTCOL_AFHI,
-		&obj);
+	int ret = osl(osl_get_object(audio_file_table, row, AFTCOL_AFHI,
+		&obj));
 	if (ret < 0)
 		return ret;
 	load_afhi(obj.data, afhi);
@@ -694,56 +731,68 @@ static void get_duration_buf(int seconds, char *buf, struct ls_options *opts)
 	}
 }
 
-static char *make_attribute_lines(const char *att_bitmap, struct afs_info *afsi)
+
+static int write_attribute_items(struct para_buffer *b,
+		const char *att_bitmap, struct afs_info *afsi)
 {
-	char *att_text, *att_lines;
+	char *att_text;
+	int ret;
 
-	get_attribute_text(&afsi->attributes, " ", &att_text);
-	if (!att_text)
-		return para_strdup(att_bitmap);
-	att_lines = make_message("%s: %s\n%s: %s",
-		status_item_list[SI_ATTRIBUTES_BITMAP], att_bitmap,
-		status_item_list[SI_ATTRIBUTES_TXT], att_text);
+	ret = WRITE_STATUS_ITEM(b, SI_ATTRIBUTES_BITMAP, "%s\n", att_bitmap);
+	if (ret < 0)
+		return ret;
+	ret = get_attribute_text(&afsi->attributes, " ", &att_text);
+	if (ret < 0)
+		return ret;
+	ret = WRITE_STATUS_ITEM(b, SI_ATTRIBUTES_TXT, "%s\n", att_text);
 	free(att_text);
-	return att_lines;
+	return ret;
 }
 
-static char *make_lyrics_lines(struct afs_info *afsi)
+static int write_lyrics_items(struct para_buffer *b, struct afs_info *afsi)
 {
 	char *lyrics_name;
+	int ret;
 
+	ret = WRITE_STATUS_ITEM(b, SI_LYRICS_ID, "%u\n", afsi->lyrics_id);
+	if (ret < 0)
+		return ret;
 	lyr_get_name_by_id(afsi->lyrics_id, &lyrics_name);
-	return make_message("%s: %u\n%s: %s\n",
-		status_item_list[SI_LYRICS_ID], afsi->lyrics_id,
-		status_item_list[SI_LYRICS_NAME], lyrics_name?
-			lyrics_name : "(none)");
+	return WRITE_STATUS_ITEM(b, SI_LYRICS_NAME, "%s\n", lyrics_name?
+		lyrics_name : "(none)");
 }
 
-static char *make_image_lines(struct afs_info *afsi)
+static int write_image_items(struct para_buffer *b, struct afs_info *afsi)
 {
 	char *image_name;
+	int ret;
+
+	ret = WRITE_STATUS_ITEM(b, SI_IMAGE_ID, "%u\n", afsi->image_id);
+	if (ret < 0)
+		return ret;
 	img_get_name_by_id(afsi->image_id, &image_name);
-	return make_message("%s: %u\n%s: %s\n",
-		status_item_list[SI_IMAGE_ID], afsi->image_id,
-		status_item_list[SI_IMAGE_NAME], image_name?
-			image_name : "(none)");
+	return WRITE_STATUS_ITEM(b, SI_IMAGE_NAME, "%s\n", image_name?
+		image_name : "(none)");
 }
 
-static char *make_filename_lines(const char *path, unsigned flags)
+static int write_filename_items(struct para_buffer *b, const char *path,
+		unsigned flags)
 {
-	char *dirname, *ret;
-	const char *basename;
+	char *val;
+	int ret;
 
 	if (!(flags & LS_FLAG_FULL_PATH))
-		return make_message("%s: %s\n",
-			status_item_list[SI_BASENAME], path);
-	basename = para_basename(path),
-	dirname = para_dirname(path);
-	ret = make_message("%s: %s\n%s: %s\n%s: %s\n",
-		status_item_list[SI_PATH], path,
-		status_item_list[SI_DIRECTORY], dirname? dirname : "?",
-		status_item_list[SI_BASENAME], basename? basename : "?");
-	free(dirname);
+		return WRITE_STATUS_ITEM(b, SI_BASENAME, "%s\n", path);
+	ret = WRITE_STATUS_ITEM(b, SI_PATH, "%s\n", path);
+	if (ret < 0)
+		return ret;
+	val = para_basename(path);
+	ret = WRITE_STATUS_ITEM(b, SI_BASENAME, "%s\n", val? val : "");
+	if (ret < 0)
+		return ret;
+	val = para_dirname(path);
+	ret = WRITE_STATUS_ITEM(b, SI_DIRECTORY, "%s\n", val? val : "");
+	free(val);
 	return ret;
 }
 
@@ -781,6 +830,14 @@ out:
 	return ret;
 }
 
+static int write_score(struct para_buffer *b, struct ls_data *d,
+		struct ls_options *opts)
+{
+	if (!(opts->flags & LS_FLAG_ADMISSIBLE_ONLY)) /* no score*/
+		return 0;
+	return WRITE_STATUS_ITEM(b, SI_SCORE, "%li\n", d->score);
+}
+
 static int print_list_item(struct ls_data *d, struct ls_options *opts,
 	struct para_buffer *b, time_t current_time)
 {
@@ -788,13 +845,9 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 	char att_buf[65];
 	char last_played_time[30];
 	char duration_buf[30]; /* nobody has an audio file long enough to overflow this */
-	char score_buf[30] = "";
 	struct afs_info *afsi = &d->afsi;
 	struct afh_info *afhi = &d->afhi;
-	struct ls_widths *w = &opts->widths;
-	int have_score = opts->flags & LS_FLAG_ADMISSIBLE_ONLY;
 	char asc_hash[2 * HASH_SIZE + 1];
-	char *att_lines, *lyrics_lines, *image_lines, *filename_lines;
 
 	if (opts->mode == LS_MODE_SHORT) {
 		ret = para_printf(b, "%s\n", d->path);
@@ -815,16 +868,15 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 			goto out;
 	}
 	get_duration_buf(afhi->seconds_total, duration_buf, opts);
-	if (have_score) {
-		if (opts->mode == LS_MODE_LONG)
-			sprintf(score_buf, "%*li ", w->score_width, d->score);
-		else
-			sprintf(score_buf, "%li ", d->score);
-	}
-
 	if (opts->mode == LS_MODE_LONG) {
+		struct ls_widths *w = &opts->widths;
+		if (opts->flags & LS_FLAG_ADMISSIBLE_ONLY) {
+			ret = para_printf(b, "%*li ",
+				opts->widths.score_width, d->score);
+			if (ret < 0)
+				goto out;
+		}
 		ret = para_printf(b,
-			"%s"    /* score */
 			"%s "   /* attributes */
 			"%*u "  /* amp */
 			"%*d "  /* image_id  */
@@ -837,7 +889,6 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 			"%*d "  /* num_played */
 			"%s "   /* last_played */
 			"%s\n", /* path */
-			score_buf,
 			att_buf,
 			w->amp_width, afsi->amp,
 			w->image_id_width, afsi->image_id,
@@ -853,11 +904,6 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 		);
 		goto out;
 	}
-	hash_to_asc(d->hash, asc_hash);
-	att_lines = make_attribute_lines(att_buf, afsi);
-	lyrics_lines = make_lyrics_lines(afsi);
-	image_lines = make_image_lines(afsi);
-	filename_lines = make_filename_lines(d->path, opts->flags);
 	if (opts->mode == LS_MODE_MBOX) {
 		const char *bn = para_basename(d->path);
 		ret = para_printf(b,
@@ -869,47 +915,78 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 		if (ret < 0)
 			goto out;
 	}
-	ret = para_printf(b,
-		"%s" /* filename stuff */
-		"%s%s%s%s" /* score */
-		"%s\n" /* attributes */
-		"%s: %s\n" /* hash */
-		"%s" /* image id, image name */
-		"%s" /* lyrics */
-		"%s: %dkbit/s\n" /* bitrate */
-		"%s: %s\n" /* format */
-		"%s: %dHz\n" /* frequency */
-		"%s: %d\n" /* channels */
-		"%s: %s\n" /* duration */
-		"%s: %lu\n" /* seconds total */
-		"%s: %s\n" /* last played time */
-		"%s: %d\n" /* num_played */
-		"%s: %u\n" /* ampplification */
-		"%s" /* tag info */
-		"%s: %lu\n" /* chunk time */
-		"%s: %lu\n", /* num chunks */
-		filename_lines,
-		have_score? status_item_list[SI_SCORE] : "",
-			have_score? ": " : "",
-			score_buf,
-			have_score? "\n" : "",
-		att_lines,
-		status_item_list[SI_HASH], asc_hash,
-		image_lines,
-		lyrics_lines,
-		status_item_list[SI_BITRATE], afhi->bitrate,
-		status_item_list[SI_FORMAT], audio_format_name(afsi->audio_format_id),
-		status_item_list[SI_FREQUENCY], afhi->frequency,
-		status_item_list[SI_CHANNELS], afhi->channels,
-		status_item_list[SI_DURATION], duration_buf,
-		status_item_list[SI_SECONDS_TOTAL], afhi->seconds_total,
-		status_item_list[SI_LAST_PLAYED], last_played_time,
-		status_item_list[SI_NUM_PLAYED], afsi->num_played,
-		status_item_list[SI_AMPLIFICATION], afsi->amp,
-		afhi->info_string,
-		status_item_list[SI_CHUNK_TIME], tv2ms(&afhi->chunk_tv),
-		status_item_list[SI_NUM_CHUNKS], afhi->chunks_total
-	);
+	ret = write_filename_items(b, d->path, opts->flags);
+	if (ret < 0)
+		goto out;
+	ret = write_score(b, d, opts);
+	if (ret < 0)
+		goto out;
+	ret = write_attribute_items(b, att_buf, afsi);
+	if (ret < 0)
+		goto out;
+	ret = write_image_items(b, afsi);
+	if (ret < 0)
+		goto out;
+	ret = write_lyrics_items(b, afsi);
+	if (ret < 0)
+		goto out;
+	hash_to_asc(d->hash, asc_hash);
+	ret = WRITE_STATUS_ITEM(b, SI_HASH, "%s\n", asc_hash);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_BITRATE, "%dkbit/s\n", afhi->bitrate);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_FORMAT, "%s\n",
+		audio_format_name(afsi->audio_format_id));
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_FREQUENCY, "%dHz\n", afhi->frequency);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_CHANNELS, "%d\n", afhi->channels);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_DURATION, "%s\n", duration_buf);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_SECONDS_TOTAL, "%lu\n",
+		afhi->seconds_total);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_LAST_PLAYED, "%s\n", last_played_time);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_NUM_PLAYED, "%d\n", afsi->num_played);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_AMPLIFICATION, "%u\n", afsi->amp);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_CHUNK_TIME, "%lu\n",
+		tv2ms(&afhi->chunk_tv));
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_NUM_CHUNKS, "%lu\n",
+		afhi->chunks_total);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_TECHINFO, "%s\n", afhi->techinfo);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_ARTIST, "%s\n", afhi->tags.artist);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_TITLE, "%s\n", afhi->tags.title);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_YEAR, "%s\n", afhi->tags.year);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_ALBUM, "%s\n", afhi->tags.album);
+	if (ret < 0)
+		goto out;
+	ret = WRITE_STATUS_ITEM(b, SI_COMMENT, "%s\n", afhi->tags.comment);
 	if (ret < 0)
 		goto out;
 	if (opts->mode == LS_MODE_MBOX) {
@@ -921,129 +998,10 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 			osl_close_disk_object(&lyrics_def);
 		}
 	}
-	free(att_lines);
-	free(lyrics_lines);
-	free(image_lines);
-	free(filename_lines);
 out:
-	free(afhi->info_string);
 	return ret;
 }
 
-/**
- * Write a list of audio-file related status items with empty values.
- *
- * \param buf Result pointer.
- *
- * This is used by vss when currently no audio file is open.
- */
-void make_empty_status_items(char *buf)
-{
-	sprintf(buf,
-		"%s: \n" /* path */
-		"%s: \n" /* dirname */
-		"%s: \n" /* basename */
-		"%s: \n" /* score */
-		"%s: \n" /* attributes bitmap */
-		"%s: \n" /* attributes txt */
-		"%s: \n" /* hash */
-		"%s: \n" /* image id */
-		"%s: \n" /* image name */
-		"%s: \n" /* lyrics id */
-		"%s: \n" /* lyrics name */
-		"%s: \n" /* bitrate */
-		"%s: \n" /* format */
-		"%s: \n" /* frequency */
-		"%s: \n" /* channels */
-		"%s: \n" /* duration */
-		"%s: \n" /* seconds total */
-		"%s: \n" /* num played */
-		"%s: \n" /* last played */
-		"%s: \n" /* audio file info */
-		"%s: \n" /* taginfo1 */
-		"%s: \n" /* taginfo2 */
-		"%s: \n" /* amplification */
-		,
-		status_item_list[SI_PATH],
-		status_item_list[SI_DIRECTORY],
-		status_item_list[SI_BASENAME],
-		status_item_list[SI_SCORE],
-		status_item_list[SI_ATTRIBUTES_BITMAP],
-		status_item_list[SI_ATTRIBUTES_TXT],
-		status_item_list[SI_HASH],
-		status_item_list[SI_IMAGE_ID],
-		status_item_list[SI_IMAGE_NAME],
-		status_item_list[SI_LYRICS_ID],
-		status_item_list[SI_LYRICS_NAME],
-		status_item_list[SI_BITRATE],
-		status_item_list[SI_FORMAT],
-		status_item_list[SI_FREQUENCY],
-		status_item_list[SI_CHANNELS],
-		status_item_list[SI_DURATION],
-		status_item_list[SI_SECONDS_TOTAL],
-		status_item_list[SI_NUM_PLAYED],
-		status_item_list[SI_LAST_PLAYED],
-		status_item_list[SI_AUDIO_FILE_INFO],
-		status_item_list[SI_TAGINFO1],
-		status_item_list[SI_TAGINFO2],
-		status_item_list[SI_AMPLIFICATION]
-	);
-}
-
-static void fixup_taginfo(char *begin, char *end)
-{
-	char *p = begin;
-
-	for (;;) {
-		p = strchr(p, '\n');
-		if (!p)
-			break;
-		if (p >= end - 1)
-			break;
-		*p = ' ';
-		p++;
-	}
-}
-
-/* crap, remove this ASAP. */
-static int fixup_info_string(char *info_string)
-{
-	char *t1, *t2, *end;
-
-	if (strncmp(info_string, "audio_file_info:", 16))
-		return -ERRNO_TO_PARA_ERROR(EINVAL);
-	t1 = strstr(info_string, "\ntaginfo1:");
-	if (!t1)
-		return -ERRNO_TO_PARA_ERROR(EINVAL);
-	t2 = strstr(info_string, "\ntaginfo2: ");
-	if (!t2)
-		return -ERRNO_TO_PARA_ERROR(EINVAL);
-
-	end = t2 + strlen(t2) + 1;
-	fixup_taginfo(info_string + 16, t1);
-	fixup_taginfo(t1 + 10, t2);
-	fixup_taginfo(t2 + 10, end);
-
-	if (t1 - info_string < 80 && t2 - t1 < 80 && end - t2 < 80)
-		return 0;
-	if (t1 - info_string >= 80) {
-		memmove(info_string + 80, t1, end - t1);
-		t1 = info_string + 80;
-		t2 -= t1 - info_string - 80;
-		end -= t1 - info_string - 80;
-	}
-	if (t2 - t1 >= 80) {
-		memmove(t1 + 80, t2, end - t2);
-		end -= t2 - t1 - 80;
-		t2 = t1 + 80;
-	}
-	if (end - t2 >= 80) {
-		t2[78] = '\n';
-		t2[79] = '\0';
-	}
-	return 1;
-}
-
 static int make_status_items(struct audio_file_data *afd,
 		struct afs_info *afsi, char *path, long score,
 		HASH_TYPE *hash)
@@ -1059,26 +1017,28 @@ static int make_status_items(struct audio_file_data *afd,
 		.flags = LS_FLAG_FULL_PATH | LS_FLAG_ADMISSIBLE_ONLY,
 		.mode = LS_MODE_VERBOSE,
 	};
-	struct para_buffer pb = {.max_size = VERBOSE_LS_OUTPUT_SIZE - 1};
+	struct para_buffer pb = {.max_size = SHMMAX - 1};
 	time_t current_time;
 	int ret;
 
-	ret = fixup_info_string(afd->afhi.info_string);
-	if (ret < 0) {
-		PARA_WARNING_LOG("ignoring invalid tag info\n");
-		afd->afhi.info_string[0] = '\0';
-	} else if (ret)
-		PARA_NOTICE_LOG("truncated overlong tag info\n");
 	time(&current_time);
-	ret = print_list_item(&d, &opts, &pb, current_time); /* frees info string */
-	afd->afhi.info_string = NULL;
+	ret = print_list_item(&d, &opts, &pb, current_time);
 	if (ret < 0)
-		goto out;
-	strncpy(afd->verbose_ls_output, pb.buf, VERBOSE_LS_OUTPUT_SIZE);
-	afd->verbose_ls_output[VERBOSE_LS_OUTPUT_SIZE - 1] = '\0';
-out:
-	free(pb.buf);
-	return ret;
+		return ret;
+	free(status_items);
+	status_items = pb.buf;
+	memset(&pb, 0, sizeof(pb));
+	pb.max_size = SHMMAX - 1;
+	pb.flags = PBF_SIZE_PREFIX;
+	ret = print_list_item(&d, &opts, &pb, current_time);
+	if (ret < 0) {
+		free(status_items);
+		status_items = NULL;
+		return ret;
+	}
+	free(parser_friendly_status_items);
+	parser_friendly_status_items = pb.buf;
+	return 1;
 }
 
 /**
@@ -1150,7 +1110,6 @@ int open_and_update_audio_file(struct osl_row *aft_row, long score,
 	ret = save_afd(afd);
 err:
 	free(afd->afhi.chunk_table);
-	free(afd->afhi.info_string);
 	osl_close_disk_object(&chunk_table_obj);
 	return ret;
 }
@@ -1353,7 +1312,6 @@ static int prepare_ls_row(struct osl_row *row, void *ls_opts)
 	}
 	return 1;
 err:
-	free(d->afhi.info_string);
 	return ret;
 }
 
@@ -1362,11 +1320,11 @@ static void com_ls_callback(int fd, const struct osl_object *query)
 	struct ls_options *opts = query->data;
 	char *p, *pattern_start = (char *)query->data + sizeof(*opts);
 	struct para_buffer b = {.max_size = SHMMAX,
+		.flags = (opts->mode == LS_MODE_PARSER)? PBF_SIZE_PREFIX : 0,
 		.max_size_handler = pass_buffer_as_shm, .private_data = &fd};
 	int i = 0, ret;
 	time_t current_time;
 
-
 	if (opts->num_patterns) {
 		opts->patterns = para_malloc(opts->num_patterns * sizeof(char *));
 		for (i = 0, p = pattern_start; i < opts->num_patterns; i++) {
@@ -1378,8 +1336,8 @@ static void com_ls_callback(int fd, const struct osl_object *query)
 	if (opts->flags & LS_FLAG_ADMISSIBLE_ONLY)
 		ret = admissible_file_loop(opts, prepare_ls_row);
 	else
-		ret = osl_rbtree_loop(audio_file_table, AFTCOL_PATH, opts,
-			prepare_ls_row);
+		ret = osl(osl_rbtree_loop(audio_file_table, AFTCOL_PATH, opts,
+			prepare_ls_row));
 	if (ret < 0)
 		goto out;
 	if (!opts->num_matching_paths)
@@ -1452,6 +1410,9 @@ int com_ls(struct rc4_context *rc4c, int argc, char * const * const argv)
 			case 'c':
 				mode = LS_MODE_CHUNKS;
 				continue;
+			case 'p':
+				mode = LS_MODE_PARSER;
+				continue;
 			default:
 				return -E_AFT_SYNTAX;
 			}
@@ -1531,12 +1492,12 @@ int com_ls(struct rc4_context *rc4c, int argc, char * const * const argv)
  * \param private_data An arbitrary data pointer, passed to \a func.
  * \param func The custom function to be called.
  *
- * \return The return value of the underlying call to osl_rbtree_loop().
+ * \return Standard.
  */
 int audio_file_loop(void *private_data, osl_rbtree_loop_func *func)
 {
-	return osl_rbtree_loop(audio_file_table, AFTCOL_HASH, private_data,
-		func);
+	return osl(osl_rbtree_loop(audio_file_table, AFTCOL_HASH, private_data,
+		func));
 }
 
 static struct osl_row *find_hash_sister(HASH_TYPE *hash)
@@ -1696,7 +1657,7 @@ static void com_add_callback(int fd, const struct osl_object *query)
 	PARA_INFO_LOG("request to add %s\n", path);
 	hs = find_hash_sister(hash);
 	ret = aft_get_row_of_path(path, &pb);
-	if (ret < 0 && ret != -E_RB_KEY_NOT_FOUND)
+	if (ret < 0 && ret != -OSL_ERRNO_TO_PARA_ERROR(E_OSL_RB_KEY_NOT_FOUND))
 		goto out;
 	if (hs && pb && hs == pb && !(flags & ADD_FLAG_FORCE)) {
 		if (flags & ADD_FLAG_VERBOSE)
@@ -1713,23 +1674,23 @@ static void com_add_callback(int fd, const struct osl_object *query)
 				if (ret < 0)
 					goto out;
 			}
-			ret = osl_del_row(audio_file_table, pb);
+			ret = osl(osl_del_row(audio_file_table, pb));
 			if (ret < 0)
 				goto out;
 			pb = NULL;
 		}
 		/* file rename, update hs' path */
 		if (flags & ADD_FLAG_VERBOSE) {
-			ret = osl_get_object(audio_file_table, hs,
-				AFTCOL_PATH, &obj);
+			ret = osl(osl_get_object(audio_file_table, hs,
+				AFTCOL_PATH, &obj));
 			if (ret < 0)
 				goto out;
 			ret = para_printf(&msg, "renamed from %s\n", (char *)obj.data);
 			if (ret < 0)
 				goto out;
 		}
-		ret = osl_update_object(audio_file_table, hs, AFTCOL_PATH,
-			&objs[AFTCOL_PATH]);
+		ret = osl(osl_update_object(audio_file_table, hs, AFTCOL_PATH,
+			&objs[AFTCOL_PATH]));
 		if (ret < 0)
 			goto out;
 		afs_event(AUDIO_FILE_RENAME, &msg, hs);
@@ -1773,12 +1734,12 @@ static void com_add_callback(int fd, const struct osl_object *query)
 			if (ret < 0)
 				goto out;
 		}
-		ret = osl_update_object(audio_file_table, row, AFTCOL_AFHI,
-			&objs[AFTCOL_AFHI]);
+		ret = osl(osl_update_object(audio_file_table, row, AFTCOL_AFHI,
+			&objs[AFTCOL_AFHI]));
 		if (ret < 0)
 			goto out;
-		ret = osl_update_object(audio_file_table, row, AFTCOL_CHUNKS,
-			&objs[AFTCOL_CHUNKS]);
+		ret = osl(osl_update_object(audio_file_table, row, AFTCOL_CHUNKS,
+			&objs[AFTCOL_CHUNKS]));
 		if (ret < 0)
 			goto out;
 		afs_event(AFHI_CHANGE, &msg, row);
@@ -1796,7 +1757,7 @@ static void com_add_callback(int fd, const struct osl_object *query)
 	objs[AFTCOL_AFSI].data = &afsi_buf;
 	objs[AFTCOL_AFSI].size = AFSI_SIZE;
 	save_afsi(&default_afsi, &objs[AFTCOL_AFSI]);
-	ret = osl_add_and_get_row(audio_file_table, objs, &aft_row);
+	ret = osl(osl_add_and_get_row(audio_file_table, objs, &aft_row));
 	afs_event(AUDIO_FILE_ADD, &msg, aft_row);
 out:
 	if (ret < 0)
@@ -1859,7 +1820,7 @@ static int add_one_audio_file(const char *path, void *private_data)
 	query.size = strlen(path) + 1;
 	ret = send_callback_request(path_brother_callback, &query,
 		get_row_pointer_from_result, &pb);
-	if (ret < 0 && ret != -E_RB_KEY_NOT_FOUND)
+	if (ret < 0 && ret != -OSL_ERRNO_TO_PARA_ERROR(E_OSL_RB_KEY_NOT_FOUND))
 		goto out_free;
 	ret = 1;
 	if (pb && (pad->flags & ADD_FLAG_LAZY)) { /* lazy is really cheap */
@@ -1922,7 +1883,12 @@ out_free:
 	free(obj.data);
 	if (afhi_ptr) {
 		free(afhi_ptr->chunk_table);
-		free(afhi_ptr->info_string);
+		free(afhi_ptr->techinfo);
+		free(afhi_ptr->tags.artist);
+		free(afhi_ptr->tags.title);
+		free(afhi_ptr->tags.year);
+		free(afhi_ptr->tags.album);
+		free(afhi_ptr->tags.comment);
 	}
 	/* Stop adding files only on send errors. */
 	return send_ret;
@@ -2139,33 +2105,33 @@ int com_touch(struct rc4_context *rc4c, int argc, char * const * const argv)
 			i++;
 			break;
 		}
-		if (!strncmp(arg, "-n", 2)) {
-			ret = para_atoi32(arg + 2, &cto.num_played);
+		if (!strncmp(arg, "-n=", 3)) {
+			ret = para_atoi32(arg + 3, &cto.num_played);
 			if (ret < 0)
 				return ret;
 			continue;
 		}
-		if (!strncmp(arg, "-l", 2)) {
-			ret = para_atoi64(arg + 2, &cto.last_played);
+		if (!strncmp(arg, "-l=", 3)) {
+			ret = para_atoi64(arg + 3, &cto.last_played);
 			if (ret < 0)
 				return ret;
 			continue;
 		}
-		if (!strncmp(arg, "-y", 2)) {
-			ret = para_atoi32(arg + 2, &cto.lyrics_id);
+		if (!strncmp(arg, "-y=", 3)) {
+			ret = para_atoi32(arg + 3, &cto.lyrics_id);
 			if (ret < 0)
 				return ret;
 			continue;
 		}
-		if (!strncmp(arg, "-i", 2)) {
-			ret = para_atoi32(arg + 2, &cto.image_id);
+		if (!strncmp(arg, "-i=", 3)) {
+			ret = para_atoi32(arg + 3, &cto.image_id);
 			if (ret < 0)
 				return ret;
 			continue;
 		}
-		if (!strncmp(arg, "-a", 2)) {
+		if (!strncmp(arg, "-a=", 3)) {
 			int32_t val;
-			ret = para_atoi32(arg + 2, &val);
+			ret = para_atoi32(arg + 3, &val);
 			if (ret < 0)
 				return ret;
 			if (val < 0 || val > 255)
@@ -2224,7 +2190,7 @@ static int remove_audio_file(__a_unused struct osl_table *table,
 			return ret;
 	}
 	afs_event(AUDIO_FILE_REMOVE, &crd->pb, row);
-	ret = osl_del_row(audio_file_table, row);
+	ret = osl(osl_del_row(audio_file_table, row));
 	if (ret < 0)
 		para_printf(&crd->pb, "%s: %s\n", name, para_strerror(-ret));
 	else
@@ -2458,7 +2424,7 @@ int com_cpsi(struct rc4_context *rc4c, int argc,  char * const * const argv)
 		}
 		break;
 	}
-	if (i + 1 >= argc) /* need at least souce file and pattern */
+	if (i + 1 >= argc) /* need at least source file and pattern */
 		return -E_AFT_SYNTAX;
 	if (!(flags & ~CPSI_FLAG_VERBOSE)) /* no copy flags given */
 		flags = ~(unsigned)CPSI_FLAG_VERBOSE | flags;
@@ -2469,6 +2435,25 @@ int com_cpsi(struct rc4_context *rc4c, int argc,  char * const * const argv)
 	return ret;
 }
 
+void afs_stat_callback(int fd, const struct osl_object *query)
+{
+	int *parser_friendly = query->data;
+	char *buf = *parser_friendly?
+		parser_friendly_status_items : status_items;
+
+	if (!buf)
+		return;
+	pass_buffer_as_shm(buf, strlen(buf), &fd);
+}
+
+int send_afs_status(struct rc4_context *rc4c, int parser_friendly)
+{
+	struct osl_object query = {.data = &parser_friendly,
+		.size = sizeof(parser_friendly)};
+
+	return send_callback_request(afs_stat_callback, &query, rc4_send_result, rc4c);
+}
+
 /* TODO: optionally fix problems by removing offending rows */
 static int check_audio_file(struct osl_row *row, void *data)
 {
@@ -2547,6 +2532,10 @@ static void aft_close(void)
 {
 	osl_close_table(audio_file_table, OSL_MARK_CLEAN);
 	audio_file_table = NULL;
+	free(status_items);
+	status_items = NULL;
+	free(parser_friendly_status_items);
+	parser_friendly_status_items = NULL;
 }
 
 /**
@@ -2563,7 +2552,7 @@ static int aft_open(const char *dir)
 	int ret;
 
 	audio_file_table_desc.dir = dir;
-	ret = osl_open_table(&audio_file_table_desc, &audio_file_table);
+	ret = osl(osl_open_table(&audio_file_table_desc, &audio_file_table));
 	if (ret >= 0) {
 		unsigned num;
 		osl_get_num_rows(audio_file_table, &num);
@@ -2572,7 +2561,7 @@ static int aft_open(const char *dir)
 	}
 	PARA_INFO_LOG("failed to open audio file table\n");
 	audio_file_table = NULL;
-	if (ret >= 0 || is_errno(-ret, ENOENT))
+	if (ret >= 0 || ret == -OSL_ERRNO_TO_PARA_ERROR(E_OSL_NOENT))
 		return 1;
 	return ret;
 }
@@ -2580,7 +2569,7 @@ static int aft_open(const char *dir)
 static int aft_create(const char *dir)
 {
 	audio_file_table_desc.dir = dir;
-	return osl_create_table(&audio_file_table_desc);
+	return osl(osl_create_table(&audio_file_table_desc));
 }
 
 static int clear_attribute(struct osl_row *row, void *data)