Bogus use after move in DataStorm/config test

[pepone]

datastorm/cpp/test/DataStorm/config/Writer.cpp

Line 304 in fba29e6

w.emplace_back(move(holder), move(node), move(topic), makeSingleKeyWriter(topic, "elem"));

Unless I'm mistaken the call to move(topic) is bogus, as it is also used in makeSingleKeyWriter(topic, "elem")

[pepone]

we can do

auto singleKeyWriter = makeSingleKeyWriter(topic, "elem");
w.emplace_back(
    std::move(holder),
    std::move(node),
    std::move(topic),
    std::move(singleKeyWriter));