https://github.com/PiotrZSL requested changes to this pull request.

**Missing tests.**
Lot of issues.
Personally i would consider modernize-use-std-copy-fill as an name, and cover 
also memset, as this would be the same code. And I would convert this into 
std::copy_n or std::full_n.

The only thing when std::copy should eb used would be if call to memset would 
be:
memset(other_begin, begin, end-begin);

https://github.com/llvm/llvm-project/pull/74663
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to