[issue2659] textwrap handling of hyphenation

Georg Brandl <[EMAIL PROTECTED]> added the comment: Thanks for the patch! Reviewed and committed as r63053. -- resolution: -> accepted status: open -> closed __ Tracker <[EMAIL PROTECTED]> __

[issue2659] textwrap handling of hyphenation

Changes by Sylvain Fourmanoit <[EMAIL PROTECTED]>: Added file: http://bugs.python.org/file10068/textwrap_hyphens_doc.patch __ Tracker <[EMAIL PROTECTED]> __ ___

[issue2659] textwrap handling of hyphenation

Changes by Sylvain Fourmanoit <[EMAIL PROTECTED]>: Removed file: http://bugs.python.org/file10061/textwrap_hyphens_doc.patch __ Tracker <[EMAIL PROTECTED]> __ _

[issue2659] textwrap handling of hyphenation

Sylvain Fourmanoit <[EMAIL PROTECTED]> added the comment: Thanks for the feedback. Enclosed patch: - moves the added "break_on_hyphens" argument at the end of the argument list - adds a unit test for the new attribute Added file: http://bugs.python.org/file10067/textwrap_break_on_hyphens_2.patc

[issue2659] textwrap handling of hyphenation

Antoine Pitrou <[EMAIL PROTECTED]> added the comment: Two things: - I think the added argument should come at the end of the argument list, so that code currently passing `drop_whitespace` positionally isn't broken by the change - it would be nice to have unit tests -- nosy: +pitrou ___

[issue2659] textwrap handling of hyphenation

Changes by Benjamin Peterson <[EMAIL PROTECTED]>: -- assignee: -> georg.brandl nosy: +georg.brandl __ Tracker <[EMAIL PROTECTED]> __ ___ Python

[issue2659] textwrap handling of hyphenation

Sylvain Fourmanoit <[EMAIL PROTECTED]> added the comment: I also think it would be nice if a public interface to completely avoid breaking hyphenated words would be included; patch included: it's pretty light, and should have no impact on any code currently using textwrap. Added file: http://bug

[issue2659] textwrap handling of hyphenation

New submission from Sylvain Fourmanoit <[EMAIL PROTECTED]>: The textwrap module in standard library breaks hyphenated words given the opportunity; I don't think that's absolutely obvious from the current doc, and it's something worth mentioning. Here is a short addition to the library reference