I see -- that make sense.  I didn't see the rest of the method in GH's diff view, so I thought the method ended with the `fill()` call.