should this be preceded by an instanceof assertion, or are we confident it would never be called without a FileSink instance as item?