Usman Akinyemi <usmanakinyemi202@xxxxxxxxx> writes: > Some code from "builtin/bugreport.c" uses uname(2) to get system > information. > > Let's refactor this code into a new get_uname_info() function, so > that we can reuse it in a following commit. This does two things: refactor and enhancement. Shouldn't it do pure refactoring in a single patch, with a follow-up patch that extends it to allow the caller to hide the system details?