The only user of FieldFormatter is DbaseFileWriter, which in turn cannot possibly be used by multiple thread in parallel because it keeps state about its position inside the dbase file. So these synchronized statements are not useful, and if the vm is not smart enough to get rid of them at runtime (it should be, but one never knows) they can also be harmful. Please remove them.