Added 'const' or '&' to arguments when needed.
Changes according @daschuer comments.
Rewrited CrateDAO::addTracksToCrate.
If track ID List is not used by the callers afterwards, use pointer;
output parameters are at the end of the parameter list.
Rewrite AnalysisDao::loadAnalysesFromQuery arguments.
Cppcheck found some issues:
 - Member variables `ControlDoublePrivate::m_trackType` and
`ControlDoublePrivate::m_trackFlags` was not initialized in the
constructor.
 - `SoundSourceProxy::ParseHeader`: return `OK` instead of `0`.
 - `TrackInfoObject::parse`: omitted implicit cast.
 - `TrackCollection`: added references to arguments in methods
`getTrackSource...`
 - `chromaprinter.cpp`: Memory leak: `pData` (return without freeing
`pData`).
 - In `GLSLWaveformRendererSignal::createFrameBuffers` was strange
expression: `m_framebuffer->isValid() && m_framebuffer->isValid()`.
 - Lots of postfix ++/-- changed to prefix form.
Added spaces around =
Corrected typo in GLSLWaveformRendererSignal::createFrameBuffers
Merge pull request #32 from troyane/DAO_cleanup

Added 'const' or '&' to arguments when needed.
Merge pull request #33 from troyane/code_cleanup

Cppcheck found some issues. Solved some of them. And fixed typo in GLSLWaveformRendererSignal::createFrameBuffers