>> Now I'm really mystified. When I added those location-tracing functions >> and macros, they were only ever enabled with SVN_DEBUG turned on (i.e., >> in maintainer-mode). Now I see that those #ifdef wrappers are gone, and >> can't recall an explanation as to why that's a good thing. >> >> Can anyone remember what that change was good for?
>Found it, r843793 and I agree with the change. So, effectively, >svn_error__locate has been public since then, and r1053469 should be >reverted anyway because it breaks the ABI. Not sure why r1053469 need to be reverted. IIUC it still keeps the svn_error__locate by changing it to NOOP. Which I think is enough to keep ABI. With regards Kamesh Jayachandran