From 56f28f7aade8d22e3c882bfb42bfb289267e491c Mon Sep 17 00:00:00 2001
From: Stefan Sauer <ensonic@users.sf.net>
Date: Mon, 7 Oct 2013 10:59:39 +0200
Subject: [PATCH] mi-todo: planning and todo comment update

---
 mediainfo/TODO             | 10 +++++++++-
 mediainfo/src/mi-info.vala |  1 -
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/mediainfo/TODO b/mediainfo/TODO
index f62ed01fee..b5588e165f 100644
--- a/mediainfo/TODO
+++ b/mediainfo/TODO
@@ -52,9 +52,13 @@ http://www.headbands.com/gspot/v26x/index.htm
     - famfamfam-flag-png: locale/usr/share/flags/countries/
   - geo-tags: map-widget?, link to google-maps?, rev-geocoding
   - artist: links to {last.fm,wikipedia}
-  - format dates nicely
+  - format dates nicely (current locale, xx days ago?)
 - more codec details
   - caps have profile/level strings, can we also turn them into wikilinks?
+- bitrate and encode_size are related, calculate one from the other if missing
+  - encoded_size = duration * bitrate;
+  - bitrate = encoded_size / duration;
+  - needed in: quicktime, ogg/theora
 
 == deep scan mode ==
 - play the file by using fakesinks and gather statistics:
@@ -92,6 +96,10 @@ http://www.headbands.com/gspot/v26x/index.htm
     - one row per element
     - each row contains colored segments
 
+= TODO for gstreamer-plugins =
+- wav: no bitrate for uncompressed files
+- mp4: no bitrate for many formats
+
 = TODO for discoverer =
 - add deep-scan mode (see above)
   - add a mode property: quick-scan, deep-scan
diff --git a/mediainfo/src/mi-info.vala b/mediainfo/src/mi-info.vala
index 792fafb341..14124dcf8a 100644
--- a/mediainfo/src/mi-info.vala
+++ b/mediainfo/src/mi-info.vala
@@ -655,7 +655,6 @@ public class MediaInfo.Info : Box
     add_table_row_for_string (table, row, "PixelAspect:", str);
     row++;
 
-    // TODO(ensonic): this seems to be always 0
     str = "%u bits/pixel".printf (vinfo.get_depth());
     add_table_row_for_string (table, row, "Bitdepth:", str);
     row++;