plugins/rm: Parse PROP header

This header contains the following interesting fields: duration and
avg_bit_rate.

plugins/rm: Simplify loop, searching for CONT header

Call _read_header_type_and_size() in only one place and make
_parse_cont_header() return the number of bytes read, so we can seek to
the right place later.

TODO: Add entry about buffered, unbuffered, mmap for plugins

plugins/ogg: Add support to new fields

New fields are supported for ogg vorbis:
* container: Always the same: ogg
* codec: Although ogg may contain streams encoded by other
  codecs we never supported them. They will fail to parse when
  we try to create the vorbis structs. Therefore for now this is
  statically set to "vorbis"
* bitrate, sampling rate, channels: As extracted from the
  vorbis header

plugins/ogg: Fix leaking comment on error path

We need to call vorbis_comment_clear() on error path.

Do not use {0} struct initializers

Change code all over the tree with the following command, and fix up the
one false positive:

git ls-files | while read f; do sed -i 's/ = { *0.*}/ = { }/g' $f; done

Get rid of "if (bla) free(bla);"

Patch generated with the following coccinelle semantic patch:

@r@
expression E;
@@

- if (E)
- free(E);
+ free(E);

plugins/asf: Add codec support

ASF as other containers may contain streams encoded in several other
codecs. Maybe we should have a global list/hash-table with all the
possible { ID: name }? Meanwhile, put the more important codecs there
for this specific format.

plugins/asf: Add channels, bitrate, sampling_rate

plugins/asf: Fix overriding type - parse multiple streams

ASF files may contain more than one stream, with more than one type.
Previously if there was a video stream after an audio stream, we would
overwrite the type.

Prepare to support multiple streams and parse all of them. If there's
any video stream, register it. Otherwise, take the first stream.

plugins/asf: Add container

And change a littler bit the coding style:
 * else on the same line as the previous "}"
 * no need to to "if (bla) free bla;"

plugins/asf: No need to overly complicate skipping attributes

We have no interest in the extension header. Skip it entirely instead of
parsing field by field and skipping them individually.

plugins/asf: Add length field

plugins/flac: Add new fields

New fields: length, bitrate, codec, container, sampling_rate, channels.

Bitrate may be a little bit higher than the actual bitrate since we
calculate it as (filesize * 8)/length. I.e. header/coverart are also
accounted.

We could start using the lower level decoder APIs of libflac, so we
don't have to open the file twice to parse metadata + streaminfo.
However benchmarks show it's not that relevant performance-wise.

plugins/id3: we parse adts files as well

.adts files are the same as .aac, but usually without ID3.

plugins: Turn on warnings

Turn on the same compiler warnings that were on for the library and fix
the issues.

  * signed/unsigned comparison
  * Missing function prototype
  * constify lms_plugin_info
  * Missing struct initializers (actually remove all of them)
  * Fix LMS_STATIC_STRING_SIZE to cast to char* since lms_string_size is
    declared with 'char *str' and we can't really change it (otherwise
    in other places it would not be able to change the
    string in place

Warnings like this in mp4 plugin is not fixed:
mp4/mp4.c:120:5: warning: implicit declaration of function
'MP4GetMetadataName' [-Wimplicit-function-declaration]
     MP4GetMetadataName(mp4_fh, &info.title.str);

This happens because MP4v2 lib deprecated the use of these functions in
favor to another MP4Tags*() API. Let this warning as is so we remember
to change this code.

gitignore: Add cscope.{out,files}

build: Unrecurse plugins' build

Make build warnings-aware

Enable common warnings by default for the library. In order to enable it
for plugins it's better to first convert the build-sys to make it not
recursive.

plugins/id3: Add new fields

Add codec, sampling rate and channels. Container is left empty on
purpose since .mp3 and .aac files are not inside a container per se. We
may want to split the _codecs table later and put the first part as the
container.

We also have basic support for AAC file in a MPEG 4 file, since it's
almost the same of an MPEG2 one.

We only parse audio parameter fields for the ones with ADTS header.
Adding support for ADIF shall be trivial in the future. The current code
find the ID3v2 tag is actually doing more than it needed. The only case
in which the ID3 tag would not be in the start of the file is if there
was an ADIF tag first. But then the header has a fixed size and we
shouldn't need to use a ring buffer of 3 chars. When adding support to
ADIF we may want to kill useless code.

Regarding the supported codecs, we support all the
mpeg{1,2,2.5,3}layer{1,2,3} values. For AAC there's MAIN, LC, SSR and
LTP. Both for MPEG 2 and 4.

plugins/id3: use designated initializers

It's a pain to add more fields to structs that initialized their fields
like this.

audio: use inttypes.h and reorder fields

We don't have versioning in plugins' structs, so plugins must be
updated.

audio: use a macro to bind values

When adding new fields it's too easy to mistakenly call a
lms_db_bind_*() function and forget to update the column which were are
binding to.

audio: Allow plugins to update new fields

We don't have versioning in plugins' structs, so plugins must be
updated.

audio: Add new fields in its table

Set update_id when checking files

Set update_id when processing files

Pass the entire pinfo to the slave

The worker process already needs 2 fields that comes from pinfo when the
process is forked. There's no reason lms_create_slave() and
lms_restart_slave() should keep the rest of the information private.
Since the worker process will require the update_id that also comes from
this struct, pass the entire struct.

Add helper method to update the global update_id

update_id is saved in lms_internal, but it's not a table versioning like
other entries.

Allow to set update_id field in files table

Add update_id to files table

Fix treating up-to-date files as processed

If the file didn't change its mtime and size we shouldn't treat it
otherwise. Previously we would:

  1) Call the callback with reply==PROCESSED
  2) Increment the number of commits to the database. This could
     end with situations like: "BEGIN TRANSACTION; COMMIT;", without
     any operation if no file changed at all.

Fix using wrong stmt size column

_update_finfo_from_stmt() was using the wrong size column as the
information about file size. This makes lms_check* always considering
the file as changed, since the size will try to match with the wrong
column.

build: Hide plugins symbols by default

Only export the symbols that are meant to be exported with the proper
use of API

build: Avoid building and installing static plugin

There's no need to build and install a static plugin. So, disable it
(only for plugins).

build: modernize build

  - Use subdir-objects
  - Don't keep INSTALL file on tree. It's a pain that it's updated
    everytime we build the source
  - ChangeLog is more like a NEWS file. ChangeLog can be retrieved from
    git
  - Don't dist autogen.sh since it's intended for devs

release 0.4.5 with mp4v2 2.0 api support.

mp4: support 2.0 mp4v2 API changes.

The function signature changed:
 * MP4Read(path, flag) -> MP4Read(path)
 * MP4Close(handle) -> MP4Close(handle, flags)

Then detect this and add a new define to choose how to use functions.

release 0.4.4 with minor build improvements.

INSTALL: update autofoo provided install instructions file.

Ignore generated files.

improve autofoo handling of m4 and generation.

Use recommended macro in configure.ac and Makefile.am to define "m4/"
as our storage for macro files.

Use autoreconf instead of manually running all the commands on our
own.

ship config.rpath to shut up newer automake.

AM_ICONV requires config.rpath that is provided by gettext stuff, but
we don't use gettext/autopoint so it's not created. However seems
newer automake will complain about this missing file.

I'm not happy to copy this non-sense script into repository, but let's
go the easy route.

use AM_ICONV to allow external libiconv to be used.

Update my copyright and lgpl to 2.1

Fixed AM_CONDITIONAL(USE_TREMOR) if OGG was not in use.

0.4.2 release with RPM spec inside.

ignore generated RPM spec file.

Add RPM spec so rpm-build works

Signed-off-by: Rui Miguel Silva Seabra <rms@1407.org>

New release with mp4v2.h fixes.

Support mp4v2.h as well.

mp4v2 does not ship with mp4.h anymore, so let's detect that.

detect color in more $TERM (urxvt and screen)

Release 0.4

Mention rakuco's contributions

Mention jprvita's contributions

Update etrunko's email on authors file

Fix double call of lms_parsers_finish()

If an error ocurred in _db_and_parsers_setup() called from the single
process version of lms_process, lms_parsers_finish() was called twice,
leading to a double free.

Change the dummy plugin not to write a log file.

Fix lms_parser_del().

When memmove'ing the parsers array, the pointer to the array was being
overwritten with the destination of the memmove.

Fix by Rafael Antognolli <antognolli@profusion.mobi>.

Adds insert time (itime) field to table files.

This field represents the time each file was inserted in the database.
It's updated on INSERT statements and also when dtime is set to zero
(representing a re-insertion of a file with the same path).
It does not cover the case where a path is overwritten by a different
file without prior deletion. In this case the itime remains the time
of the insertion of the first file.

Fix binding in lms_db_set_file_dtime().

The id value was being binded to the statement with index one, which
is the index of the dtime field in the statement.

Ogg: Add "ogv" extension

Allow building from a separate directory.

Remove compiler warnings.

Fix small typo on README file.

Add length field to table videos.

Add length field to table audios.

Fix doxygen of dtime field in files DB table.

Added m4v extension to the mp4 plugin.

LMS was mistakingly treating Composer field as Genre/Content.

Version bump to reflect API changes.

Fix sqlite3 dependencies.

New status code for progress callback (affects lms_process only).

Changed the semantics of the LMS_PROGRESS_STATUS_PROCESSED status code,
which is now triggered only for files which are actually parsed (do a
match). Files which don't get a parser match send the
*new* LMS_PROGRESS_STATUS_SKIPPED status code.

Also updating TODO file.

Do not modify or free FLAC library-owned strings.

lms_string_size_strip_and_free() and lms_charset_conv() will modify
string contents and pointers, do not handle it FLAC library strings,
this will screw memory. Instead we duplicate memory, operate on it and
then free.

add path to parser_info.

This way we can use lms_parser_info_find() to retrieve module path as well.

improve test to use getopt.

Test is now much more user friendly using getopt, one can choose to
set or not various parameters that before were forced. See --help.

export parsers and their informations.

It is now possible to list all known parsers and retrieve information
about them, like category and description.

Release 0.2

Remove compiler warnings.

New column on audios table: artist_id.

In the old schema, audio files with no album tags didn't get
their artist tags inserted on the DB, also.

Bugfix: using correct column.

Introducing new API function: lms_check_single_process().

It behaves as lms_check(), but without forking into a child process.
The test program was updated in order to choose which checking API
function to call.

Changes on (lms_check's) functions to facilitate the choice between
single process/original version mode of operation of LMS.

Now a custom _check_row() function may be used. Also preparing for
introduction of a new db struct.

Factoring out code from (lms_check's) lms_check() and some related functions.

Introducing new API function: lms_process_single_process().

It behaves as lms_process(), but without forking into a child process.
The test program was updated in order to choose which processing API
function to call.

Adapting _process_trigger() to call a custom process_file() function.

The modification on struct pinfo was made because a new info struct is to
be introduced for the single-process version of LMS API functions. They
have struct cinfo as common fields.

Factoring out code from (lms_process's) lms_process().

One of the created functions will manage the choice between
single process/original version mode of operation of LMS.

Factoring out code from (lms_process's) _slave_work().

The two created functions are going to be, later, used by
file processing functions for the single version lms API.

Add callback to report progress.

Get missing data from id3v1 when they are not available on id3v2.

ASF: Get album artist from extended content description.

Update my mail, add Etrunko.

Proper handling of DT_UNKNOWN directory entry type.

Some file systems like ReiserFS and NFS use DT_UNKNOWN for both
files and directories, so in this case on must stat(2) the path
to get the real type of the file.

Thanks to Etrunko, Aloisio, Lauro and MoRpHeUz for point it.

OGG: Avoid segfault when value returned by vorbis_comment_query is NULL.

Created new function to avoid code duplication.

Added example in test.c.

OGG: Make ogg plugin work with both libvorbisidec (aka Tremor) and libvorbis.

configure.ac: Better check for mp4 libraries.

Fixed FSF address.

Added flv to video-dummy.

Removed dead code.