I disagree on this point. I put the code in a standalone function so that the test method is more readable. The same applies for call_iface_info_cb(). I strongly believe that many small functions make better/cleaner code, which is also easier to maintain.

Why would you want to inline/merge the function?