For some additional context (and perhaps an extension to this proposal):

• computeBBox and computeKernelBBox are equivalent to computeKernelImage().getBBox()
• computeImageBBox is equivalent to computeImage().getBBox().
• computeBBox is a nonvirtual method that delegates to doComputeBBox (allowing the base class to cache).
• computeImageBBox is a nonvirtual method that delegates to doComputeImagBBox (allowing the base class to cache).

I think that means we should at least deprecate doComputeBBox as well, in favor of (adding) doComputeKernelBBox.

We could also consider renaming (with deprecation, of course):

• computeKernelBBox -> computeKernelImageBBox
• doComputeKernelBBox -> doComputeKernelImageBBox
  to better match the actual method they correspond to: computeKernelImage (not computeKernel).

Other than that, strong from me (Josh created this RFC as a direct result of my confusion about what computeBBox did).

Joshua Meyers, this is ready to be adopted whenever you have the time to write an implementation ticket for it.

Joshua Meyers are you still intending to adopt this RFC or withdraw it?

Joshua Meyers please add a triggering ticket and adopt this RFC or else withdraw it if you've changed your mind.

Adopting all of Jim's suggestions above.