It looks reasonable. Just having the same thoughts as Tim on the attributional side. Having a proper reference in the patch itself helps a lot later if you get asked about it. -Stefan