> + if (ops->existing_ctrl) { > + if (!opts->duplicate_connect && ops->existing_ctrl(opts)) { Nitpick, but the logic would read easier at least to me as: if (!opts->duplicate_connect && ops->existing_ctrl && ops->existing_ctrl(opts)) { I can fix this up if we get some more positive reviews and this gets applied.