[webkit-reviews] review denied: [Bug 23501] Overlapping memcpy in TestDecoder::reset : [Attachment 27074] Patch attempt for skipping assignment in reset().

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Dean McNamee <deanm at chromium.org>'s
request for review:
Bug 23501: Overlapping memcpy in TestDecoder::reset
https://bugs.webkit.org/show_bug.cgi?id=23501

Attachment 27074: Patch attempt for skipping assignment in reset().
https://bugs.webkit.org/attachment.cgi?id=27074&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
This is the wrong level to fix this at. The TextEncoding class's assignment
operator needs to deal with this. Generally, C++ allows you to self-assign, and
it's the assignment operator's responsibility to deal with that.

I don't see how memcpy is involved at all in the compiler-generated assignment
operator. A TextEncoding has a single data member, a const char*, so I can't
imagine how memcpy gets involved. Is this something specific to some particular
platform? Perhaps you're using a misconfigured development tool that is
complaining about something that's not a real problem.