[issue30270] Remove sqlite3.Cache display method

2018-07-22 Thread Berker Peksag
Berker Peksag added the comment: Closing this as 'rejected' for the reasons given by Brett and David. Thanks for working on this! -- resolution: -> rejected stage: -> resolved status: open -> closed ___ Python tracker

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Berker Peksag
Berker Peksag added the comment: FWIW, I agree with Brett and David. Did you try to use it? I prefer to fix it even if it's broken at the moment. -- ___ Python tracker ___ _

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread R. David Murray
R. David Murray added the comment: If it is there for debugging, why would we want to remove it? -- nosy: +r.david.murray ___ Python tracker ___ _

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Brett Cannon
Brett Cannon added the comment: With over 10 years of existence for that method, I can guarantee you someone relies on it, and so we should do a reasonable job to avoid breaking them without at least some warning first. -- ___ Python tracker

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Aviv Palivoda
Aviv Palivoda added the comment: >From looking at the git blame this is there since 2006. Maybe this should be >dependent on issue #30262. Maybe at first there should be a deprecation >warning and then removing this later. I understand that this might break someone's code but this is an undocum

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Brett Cannon
Brett Cannon added the comment: How long has the method been there? And does it hurt anything for it to be exposed? Removing it at this point may break code if it predates Python 3.7. -- nosy: +brett.cannon ___ Python tracker

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Aviv Palivoda
Changes by Aviv Palivoda : -- pull_requests: +1559 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Aviv Palivoda
New submission from Aviv Palivoda: The display method is there for debugging and should not be in the released code. -- components: Extension Modules messages: 293005 nosy: berker.peksag, ghaering, palaviv priority: normal severity: normal status: open title: Remove sqlite3.Cache displa