[issue27396] Change default filecmp.cmp shallow option

2016-06-26 Thread Raymond Hettinger
Raymond Hettinger added the comment: The docs are already pretty clear. In general, there isn't a defense against a naive use of anything in the computing world. We can write docs but can't make a person read them and realize that their pre-existing mental model is off base. Sorry, but I'm g

[issue27396] Change default filecmp.cmp shallow option

2016-06-26 Thread Gabriel Hackebeil
Gabriel Hackebeil added the comment: All good points. Perhaps further emphasis on this in the documentation would be helpful to. As it stands, this module is a dangerous one for a naive user (like me) to stumble across. Maybe introducing an “exact” or “slow" diff function to the module would h

[issue27396] Change default filecmp.cmp shallow option

2016-06-26 Thread Raymond Hettinger
Raymond Hettinger added the comment: Changing an existing API is painful, especially so for cases like this that have existed for a very, very long time. Such as a change would likely have a detrimental effect on long-standing code relying on the existing behavior. The module itself is all ab

[issue27396] Change default filecmp.cmp shallow option

2016-06-26 Thread Gabriel Hackebeil
New submission from Gabriel Hackebeil: I would like to propose changing the default setting for the shallow compare option in filecmp.cmp to False (or providing access an exact comparison function that does not use various performance optimizations). I think many users will turn to this module