
API REVIEW PROCEDURE

- all public methods are documented (arguments, return value, exceptions)
  Important: check that @raises are present if a method is likely to throw an
             exception
  Important: add "MT safe." if the method can be called from multiple threads
             at the same time
- all public instance variables and class variables are documented
- Epydoc's generated documentation looks fine (in docs/api/index.html)
- code style complies with the one explained in docs/coding_style.txt
- unit tests code coverage is reasonable (above 80% is a good indicator even
  though a manual checking of what is covered is important)
- no DOCME should be left; FIXME should have been looked over and fixed if
  possible in a very short amount of time; for big issues, tickets should be
  opened


When finished, check the maintainers listed are aware that they are
maintainers and put your name next in a maintainer variable: __maintainerX__

Finally, remove the file from the list.


All the following files need to be reviewed:

CORE

./core/observers/dict.py

./core/utils/deferred_action.py (Florian)

./core/media_scanner.py (Philippe)
./core/db_backend.py (Philippe)
[DOC ok, UNIT TESTS hard] ./core/interface_controller.py
./core/application.py
./core/config.py
./core/media_manager.py
./core/manager.py
./core/thumbnailer.py
./core/media_file.py
[IN PROGRESS] ./core/frontend.py (Lionel)
./core/media_db.py
./core/plugin_registry.py
[IN PROGRESS] ./core/backend.py (Lionel)


BASE COMPONENTS

./base_components/input_provider.py (Florian)
./base_components/service_provider.py (Florian)
./base_components/activity.py (Florian)
./base_components/action.py (Lionel)
./base_components/playlist.py (Philippe)
./base_components/theme.py (Lionel)
./base_components/context.py (Florian)
[IN PROGRESS] ./base_components/controller.py (Florian)
./base_components/view.py (Florian)
