Thread (26 messages) flat view 26 messages, 3 authors, 2016-06-15

Re: [PATCH 0/17] removing questionable uses of git_path

From: Duy Nguyen <hidden>
Date: 2016-06-15 23:06:12

On Mon, Aug 10, 2015 at 4:27 PM, Jeff King [off-list ref] wrote:
The problem is that git_path uses a static buffer that gets overwritten
by subsequent calls.
resolve_ref() was in the same boat, then we renamed it to
resolve_ref_unsafe(), I believe with an intention to eventually kill
it. But it lives on anyway. I wanted to suggest renaming git_path() to
git_path_unsafe(). But I'm not sure if that will catch reviewer's eyes
when new call sites are added. Probably not..
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help