MISRA23_13.1 flagging non initializer lists for having persistent side effects

[kylejacksonb]

someType status = rBuffer.ReadData(readBuf.data(), byteData);

[jasonbquinn]

I am unable to replicate this. This is what I tried and didn't get any violations from the check:

#include <vector>

enum class someType : int
{
    OK = 0,
    ERROR = 1
};

class RBuffer
{
public:
    someType ReadData(std::uint8_t *dest, std::size_t count) noexcept
    {
        if (!dest || count == 0u) {
            return someType::ERROR;
        }

        for (std::size_t i = 0u; i < count; ++i) {
            dest[i] = static_cast<std::uint8_t>(i & 0xFFu);
        }
        return someType::OK;
    }
};

int main()
{
    RBuffer rBuffer;
    std::vector<std::uint8_t> readBuf(16u);
    std::size_t byteData = readBuf.size();

    someType status = rBuffer.ReadData(readBuf.data(), byteData);
}

[kylejacksonb]

Following up with user.

[groke]

User said to disregard for now