Fix a memory leak in TableData::operator=

[WeiqunZhang]

X-Ref: ECP-WarpX/WarpX#4779

[ax3l]

Thank you! I think the copy constructor might need the same logic usually, does it?

[WeiqunZhang]

You mean copy assignment? It is deleted.

…

On Thu, Mar 21, 2024, 6:40 AM Axel Huebl ***@***.***> wrote: ***@***.**** approved this pull request. Thank you! I think the copy constructor might need the same logic usually, does it? — Reply to this email directly, view it on GitHub <#3807 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB37TYLCRIEQIUHOICON6XLYZLPN3AVCNFSM6AAAAABEYZTEBSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNJSGI2TKMJSHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[ax3l]

@WeiqunZhang I actually meant the move constructor, but I think that's not needed there after all (nothing to clear from a previous allocation).

[WeiqunZhang]

Yes for constructor there is nothing to clear and no alias to check.

…

On Sun, Mar 24, 2024, 6:21 PM Axel Huebl ***@***.***> wrote: @WeiqunZhang <https://github.com/WeiqunZhang> I actually meant the move constructor, but I think that's not needed there after all (nothing to clear from a previous allocation). — Reply to this email directly, view it on GitHub <#3807 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB37TYMZGMIMWC5E5NS3KGDYZ532NAVCNFSM6AAAAABEYZTEBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJXGA2DKNZQHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>