[PATCH] D158254: Fixing the memory leak using split() instead of strtok
Abhina Sree via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 21 10:15:52 PDT 2023
abhina.sreeskantharajan accepted this revision.
abhina.sreeskantharajan added a comment.
This revision is now accepted and ready to land.
In D158254#4603765 <https://reviews.llvm.org/D158254#4603765>, @hchilama wrote:
> Hi @abhina.sreeskantharajan I have verified the fix aganist the clang-format, seems fine. (FYI: I saw clean build on Friday)
>
> I have modified this object with the edit policy "All Users"
>
> Please let me know if I need to do anything else here?
I think the format should be good then, the CI might be failing due to another commit. LGTM, thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158254/new/
https://reviews.llvm.org/D158254
More information about the cfe-commits
mailing list