[ rockbox-Patches-750691 ] Per file/directory cfg file loader

Patches item #750691, was opened at 2003-06-07 16:26
Message generated for change (Comment added) made by brking
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=439120&aid=750691&group_id=44306

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Brian King (brking)
Assigned to: Nobody/Anonymous (nobody)
Summary: Per file/directory cfg file loader

Initial Comment:
Attached is a patch that allows for automatically
loading of cfg like files that can be maintained on a
per directory, per mp3, or per m3u basis. The files
handled include:

1) mp3-specific, called trackname.cfo
2) m3u-specific, called playlist.cfo
3) dir-specific, called _default.cfo 

I changed the extension (config override) because I
didn't want these files to show up in the file browser.
A user should never need to "play" one of these files
manually.

Some layering of cfg files is also allowed. You can
have a _default.cfo file in a given directory and a
trackname.cfo as well. The _default.cfo will get loaded
first, then the trackname.cfo will get loaded. 

After playing a given song with a .cfo override file
associated with it, the settings prior to loading the
override file are restored.

One feature I didn't implement was attempting to
recurse the directory structure for a given file,
loading every _default.cfo in the path... If this is a
feature anyone wants, let me know and I can add it.



----------------------------------------------------------------------

>Comment By: Brian King (brking)
Date: 2003-12-17 15:52

Message:
Logged In: YES 
user_id=624006

Attached is an update which fixes a bug in that per mp3 file
cfo files were getting loaded while the previously playing
song was still playing. It also adds a .cfg file option:
rvolume, which can be used to adjust the relative volume of
a given song by using +10 to increase volume by 10 or -10 to
decrease by 10. Documentation is now updated as well.

----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-09-30 21:38

Message:
Logged In: YES 
user_id=624006

Keeping current with cvs. No new features.

----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-07-09 21:15

Message:
Logged In: YES 
user_id=624006

Attached is a patch to fix Henrik's issue. I didn't try
applying the bookmarking patch as it was going to require a
bit of work (The patch I downloaded seemed to be a bit out
of date), but I did what you suggested. I also fixed a small
bug that prevented the patch from doing anything useful;)

----------------------------------------------------------------------

Comment By: Henrik Backe (hbacke)
Date: 2003-07-09 07:16

Message:
Logged In: YES 
user_id=714272

Your change of playlist_get_resume_info breaks the bookmark
patch. Also to me playlist_get resume_info is a query function
so I think it's not the best place to change global_settings.

Change the code in wps.c slightly instead to something like 
this (hope you can find the place)

if (!playlist_get_resume_info(&global_settings.resume_index))
{
  saved_settings.resume_index=global_settings.resume_index;
  global_settings.resume_offset=id3->offset;
  saved_settings.resume_offset = id3->offset;
  settings_save();
}

Keep up the great work/Henrik


----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-07-08 21:01

Message:
Logged In: YES 
user_id=624006

Patch 5 is an update against the current cvs. No functional
changes.

----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-06-14 14:25

Message:
Logged In: YES 
user_id=624006

I had broken the resume function with my previous patches.
This patch (4) fixes resume.

----------------------------------------------------------------------

Comment By: Steve Michelson (chilliedog)
Date: 2003-06-14 13:58

Message:
Logged In: YES 
user_id=719492

Great idea, Brian.

----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-06-11 21:49

Message:
Logged In: YES 
user_id=624006

This patch should fix the sound settings and file settings
(eg. wps file) getting messed up when switching songs. 

----------------------------------------------------------------------

Comment By: Brian King (brking)
Date: 2003-06-07 22:20

Message:
Logged In: YES 
user_id=624006

The latest patch (2) fixes a problem that when loading a
playlist, the configuration was not getting loaded properly.
It also restores the configuration when stopping the mp3
rather than waiting until the next file is selected.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=439120&aid=750691&group_id=44306