[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