implementation of Text.dump method

[johnw42]

BPO	546244
Nosy	@gvanrossum, @tim-one, @loewis
Files	patch.diff: patch for Tkinter.py patch2.diff: corrected patch for Tkinter.py

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/gvanrossum'
closed_at = <Date 2004-08-19.11:41:49.000>
created_at = <Date 2002-04-19.18:10:48.000>
labels = ['expert-tkinter']
title = 'implementation of Text.dump method'
updated_at = <Date 2004-08-19.11:41:49.000>
user = 'https://bugs.python.org/johnw42'

bugs.python.org fields:

activity = <Date 2004-08-19.11:41:49.000>
actor = 'loewis'
assignee = 'gvanrossum'
closed = True
closed_date = None
closer = None
components = ['Tkinter']
creation = <Date 2002-04-19.18:10:48.000>
creator = 'johnw42'
dependencies = []
files = ['4173', '4174']
hgrepos = []
issue_num = 546244
keywords = ['patch']
message_count = 5.0
messages = ['39659', '39660', '39661', '39662', '39663']
nosy_count = 4.0
nosy_names = ['gvanrossum', 'tim.peters', 'loewis', 'johnw42']
pr_nums = []
priority = 'low'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue546244'
versions = []

[johnw42]

This is a fairly robust implementation of the dump
command for the text widget. It supports all the
options of the underlying Tk command. This patch is
relative to version 2.2.1.

[gvanrossum]

Logged In: YES
user_id=6380

A quick test suggets that you don't parse the dump output
correctly. I entered "The quick brown fox jumps over the
lazy dog.\n" into a text widget, and then called
t.dump("1.0", "2.0"). It returned the following garbage:

[('text', '{The', 'quick'), ('brown', 'fox', 'jumps'),
('over', 'the', 'lazy'), ('dog.}', '1.0', 'mark'),
('current', '1.44', 'mark'), ('anchor', '1.44', 'mark'),
('insert', '1.44', 'text'), ('{', '}', '1.44')]

It looks like you don't parse the curly braces at all.

Maybe you can find someone on c.l.py to help you fix this?

[johnw42]

Logged In: YES
user_id=44174

What an embarassing mistake! I've attached a new version
that uses a callback function to build the result list
instead of trying to parse a string with Tcl quoting in it.
It seems to work correctly when there are spaces and other
magic characters in the results.

[tim-one]

Logged In: YES
user_id=31435

Guido, care to try this again? Looks like you're the only one
to try in 2 years!

[loewis]

Logged In: YES
user_id=21627

Guido had already committed this as Tkinter.py 1.162, on
2002/04/23. Closing it as accepted.