Stefan Beller <sbeller@xxxxxxxxxx> writes: > To explain, you can either have: > struct git_attr_result result[2]; > or > struct git_attr_result *result = git_attr_result_alloc(check); > and both are running just fine in a thread. However you should not > make that variable static. But maybe that is too much common sense > and hence confusing. Yup, if you spelled it out that does make sense, but the description given in the patch was just misleading. Thanks.