Hi Jeff, On 12/2/19 6:52 PM, Jeff Smith wrote: > - if(!locinfo->lconv) { > - free_locinfo(locinfo); > - return NULL; > - } > + if(!locinfo->lconv) > + goto failed; I'm not sure if it's an improvement. I think I would prefer the code to stay as it is. Thanks, Piotr