Looks like a good set to me. The one patch is a little bit incongruous,
but the total volume is quite manageable so it's not really an issue.
Also, having a patch that may or may not fit into a set is far less of a
concern to me than having changes that may or may not belong in a
patch. Each individual patch is clear and consistent, so don't worry.
Also, the (inst == NULL) check is a good catch. I wouldn't be surprised
if we need that in a few more spots, especially since we've been a bit
inconsistent when it comes to the "Is a NULL instance an error or just a
NULL instance?" question.
--
-Jay