<div dir="ltr">Hi Rafael,<div><br></div><div>Good point.</div><div>There's only one usage in clang's ToolingTest.cpp: <a href="https://reviews.llvm.org/source/clang/browse/cfe/trunk/unittests/Tooling/ToolingTest.cpp;327962$322">https://reviews.llvm.org/source/clang/browse/cfe/trunk/unittests/Tooling/ToolingTest.cpp;327962$322</a>.</div><div>And it should be easy to remove it. Will come up with a change that removes those functions, we shouldn't need them.</div></div><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 19, 2018 at 10:16 PM Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ilya Biryukov via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> writes:<br>
<br>
> +/// @brief Get a unique name, not currently exisiting in the filesystem. Subject<br>
> +/// to race conditions, prefer to use createUniqueFile instead.<br>
> +///<br>
> +/// Similar to createUniqueFile, but instead of creating a file only<br>
> +/// checks if it exists. This function is subject to race conditions, if you<br>
> +/// want to use the returned name to actually create a file, use<br>
> +/// createUniqueFile instead.<br>
> +std::error_code getPotentiallyUniqueFileName(const Twine &Model,<br>
> +                                             SmallVectorImpl<char> &ResultPath);<br>
> +<br>
> +/// @brief Get a unique temporary file name, not currently exisiting in the<br>
> +/// filesystem. Subject to race conditions, prefer to use createTemporaryFile<br>
> +/// instead.<br>
> +///<br>
> +/// Similar to createTemporaryFile, but instead of creating a file only<br>
> +/// checks if it exists. This function is subject to race conditions, if you<br>
> +/// want to use the returned name to actually create a file, use<br>
> +/// createTemporaryFile instead.<br>
> +std::error_code<br>
> +getPotentiallyUniqueTempFileName(const Twine &Prefix, StringRef Suffix,<br>
> +                                 SmallVectorImpl<char> &ResultPath);<br>
> +<br>
<br>
Do we have any use for these?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>