Mercurial > hg > piper-cpp
changeset 102:cd438188e3f9
Fix compiler warnings
author | Chris Cannam <c.cannam@qmul.ac.uk> |
---|---|
date | Mon, 17 Oct 2016 16:39:36 +0100 |
parents | 8c449824e08d |
children | 491a4bc10a01 |
files | vamp-capnp/VampnProto.h vamp-client/PluginStub.h vamp-server/server.cpp vamp-support/PluginStaticData.h |
diffstat | 4 files changed, 39 insertions(+), 34 deletions(-) [+] |
line wrap: on
line diff
--- a/vamp-capnp/VampnProto.h Fri Oct 14 16:22:32 2016 +0100 +++ b/vamp-capnp/VampnProto.h Mon Oct 17 16:39:36 2016 +0100 @@ -143,10 +143,10 @@ b.setHasFixedBinCount(od.hasFixedBinCount); if (od.hasFixedBinCount) { - b.setBinCount(od.binCount); + b.setBinCount(int(od.binCount)); if (od.binNames.size() > 0) { - auto binNames = b.initBinNames(od.binNames.size()); - for (size_t i = 0; i < od.binNames.size(); ++i) { + auto binNames = b.initBinNames(unsigned(od.binNames.size())); + for (int i = 0; i < int(od.binNames.size()); ++i) { binNames.set(i, od.binNames[i]); } } @@ -232,8 +232,8 @@ } if (pd.valueNames.size() > 0) { - auto valueNames = b.initValueNames(pd.valueNames.size()); - for (size_t i = 0; i < pd.valueNames.size(); ++i) { + auto valueNames = b.initValueNames(unsigned(pd.valueNames.size())); + for (int i = 0; i < int(pd.valueNames.size()); ++i) { valueNames.set(i, pd.valueNames[i]); } } @@ -282,8 +282,8 @@ b.setLabel(f.label); if (f.values.size() > 0) { - auto values = b.initFeatureValues(f.values.size()); - for (size_t i = 0; i < f.values.size(); ++i) { + auto values = b.initFeatureValues(unsigned(f.values.size())); + for (int i = 0; i < int(f.values.size()); ++i) { values.set(i, f.values[i]); } } @@ -317,13 +317,13 @@ const Vamp::Plugin::FeatureSet &fs, const PluginOutputIdMapper &omapper) { - auto featureset = b.initFeaturePairs(fs.size()); + auto featureset = b.initFeaturePairs(unsigned(fs.size())); int ix = 0; for (const auto &fsi : fs) { auto fspair = featureset[ix]; fspair.setOutput(omapper.indexToId(fsi.first)); - auto featurelist = fspair.initFeatures(fsi.second.size()); - for (size_t j = 0; j < fsi.second.size(); ++j) { + auto featurelist = fspair.initFeatures(unsigned(fsi.second.size())); + for (int j = 0; j < int(fsi.second.size()); ++j) { auto feature = featurelist[j]; buildFeature(feature, fsi.second[j]); } @@ -387,32 +387,32 @@ b.setCopyright(d.copyright); b.setVersion(d.pluginVersion); - auto clist = b.initCategory(d.category.size()); - for (size_t i = 0; i < d.category.size(); ++i) { + auto clist = b.initCategory(unsigned(d.category.size())); + for (int i = 0; i < int(d.category.size()); ++i) { clist.set(i, d.category[i]); } - b.setMinChannelCount(d.minChannelCount); - b.setMaxChannelCount(d.maxChannelCount); + b.setMinChannelCount(int(d.minChannelCount)); + b.setMaxChannelCount(int(d.maxChannelCount)); const auto &vparams = d.parameters; - auto plist = b.initParameters(vparams.size()); - for (size_t i = 0; i < vparams.size(); ++i) { + auto plist = b.initParameters(unsigned(vparams.size())); + for (int i = 0; i < int(vparams.size()); ++i) { auto pd = plist[i]; buildParameterDescriptor(pd, vparams[i]); } const auto &vprogs = d.programs; - auto pglist = b.initPrograms(vprogs.size()); - for (size_t i = 0; i < vprogs.size(); ++i) { + auto pglist = b.initPrograms(unsigned(vprogs.size())); + for (int i = 0; i < int(vprogs.size()); ++i) { pglist.set(i, vprogs[i]); } b.setInputDomain(fromInputDomain(d.inputDomain)); const auto &vouts = d.basicOutputInfo; - auto olist = b.initBasicOutputInfo(vouts.size()); - for (size_t i = 0; i < vouts.size(); ++i) { + auto olist = b.initBasicOutputInfo(unsigned(vouts.size())); + for (int i = 0; i < int(vouts.size()); ++i) { auto od = olist[i]; buildBasicDescriptor(od, vouts[i]); } @@ -469,7 +469,7 @@ const PluginConfiguration &c) { const auto &vparams = c.parameterValues; - auto params = b.initParameterValues(vparams.size()); + auto params = b.initParameterValues(unsigned(vparams.size())); int i = 0; for (const auto &pp : vparams) { auto param = params[i++]; @@ -529,8 +529,8 @@ flags.push_back(piper::AdapterFlag::ADAPT_BUFFER_SIZE); } - auto f = r.initAdapterFlags(flags.size()); - for (size_t i = 0; i < flags.size(); ++i) { + auto f = r.initAdapterFlags(unsigned(flags.size())); + for (int i = 0; i < int(flags.size()); ++i) { f.set(i, flags[i]); } } @@ -608,8 +608,8 @@ const PluginHandleMapper &pmapper) { b.setHandle(pmapper.pluginToHandle(cr.plugin)); - auto olist = b.initOutputs(cr.outputs.size()); - for (size_t i = 0; i < cr.outputs.size(); ++i) { + auto olist = b.initOutputs(unsigned(cr.outputs.size())); + for (int i = 0; i < int(cr.outputs.size()); ++i) { auto od = olist[i]; buildOutputDescriptor(od, cr.outputs[i]); } @@ -637,8 +637,8 @@ auto t = b.initTimestamp(); buildRealTime(t, timestamp); - auto vv = b.initInputBuffers(buffers.size()); - for (size_t ch = 0; ch < buffers.size(); ++ch) { + auto vv = b.initInputBuffers(unsigned(buffers.size())); + for (int ch = 0; ch < int(buffers.size()); ++ch) { const int n = int(buffers[ch].size()); vv.init(ch, n); auto v = vv[ch]; @@ -739,8 +739,8 @@ const ListResponse &resp) { auto r = b.getResponse().initList(); - auto p = r.initAvailable(resp.available.size()); - for (size_t i = 0; i < resp.available.size(); ++i) { + auto p = r.initAvailable(unsigned(resp.available.size())); + for (int i = 0; i < int(resp.available.size()); ++i) { auto pd = p[i]; buildExtractorStaticData(pd, resp.available[i]); }
--- a/vamp-client/PluginStub.h Fri Oct 14 16:22:32 2016 +0100 +++ b/vamp-client/PluginStub.h Mon Oct 17 16:39:36 2016 +0100 @@ -110,9 +110,9 @@ throw std::logic_error("Plugin has already been initialised"); } - m_config.channelCount = inputChannels; - m_config.stepSize = stepSize; - m_config.blockSize = blockSize; + m_config.channelCount = int(inputChannels); + m_config.stepSize = int(stepSize); + m_config.blockSize = int(blockSize); m_outputs = m_client->configure(this, m_config);
--- a/vamp-server/server.cpp Fri Oct 14 16:22:32 2016 +0100 +++ b/vamp-server/server.cpp Mon Oct 17 16:39:36 2016 +0100 @@ -17,6 +17,11 @@ using namespace piper_vamp; using namespace Vamp; +//!!! This could be faster and lighter: +// - Use Capnp structures directly rather than converting to vamp-support ones +// - Use Vamp C API (vamp.h) directly rather than converting to C++ +//!!! Doing the above for process() and finish() alone would be a good start + void usage() { string myname = "piper-vamp-server";
--- a/vamp-support/PluginStaticData.h Fri Oct 14 16:22:32 2016 +0100 +++ b/vamp-support/PluginStaticData.h Mon Oct 17 16:39:36 2016 +0100 @@ -73,8 +73,8 @@ std::string copyright; int pluginVersion; std::vector<std::string> category; - int minChannelCount; - int maxChannelCount; + size_t minChannelCount; + size_t maxChannelCount; Vamp::PluginBase::ParameterList parameters; Vamp::PluginBase::ProgramList programs; Vamp::Plugin::InputDomain inputDomain;