From: SourceForge.net <no...@so...> - 2008-10-06 08:29:39
|
Bugs item #1934200, was opened at 2008-04-04 10:23 Message generated for change (Settings changed) made by oehhar You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=1934200&group_id=10894 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: 53. Configuration and Build Tools Group: current: 8.5.4 >Status: Open Resolution: Fixed Priority: 5 Private: No Submitted By: Harald Oehlmann (oehhar) Assigned to: Joe Mistachkin (mistachkin) Summary: man2tcl fails with \N'xx' sequence->ParseCmd.3 fails Initial Comment: on windows hlp file build, the program man2tcl.c is compiled and executed for each help file. A sequence of \N'xx' is translated to: text\u00yytext ' (yy: hex of xx) and should be translated to: text \u00yy This bug makes the help build fail on the file ParseCmd.3, which contains the sequence: \N'34' to get a double-quote. The position of this text is on the page of Tcl_ParseCommand, section Tcl_ParseQuotedString A patch against 8.5.2 is attached. ---------------------------------------------------------------------- >Comment By: Harald Oehlmann (oehhar) Date: 2008-10-06 10:28 Message: Row 355: p++ of the patch was not committed. This results in additional unwanted "'"-characters in the output. Example: man2tcl.exe ParseCmd.3 > ParseCmd_86a2.out ParseCmd.3 line 140: "The first character of \fIstart\fR must be \fB\N'34'\fR. " Translates to: ---START--- text The\ first\ character\ of\ font I text start font R text \ must\ be\ font B text \u0022 text ' font R text .\ newline ---END--- The command: "\N'34'" translates to: ---START--- text \u0022 text ' ---END--- and should translate to: ---START--- text \u0022 ---END--- The output of the example is on the Tcl_ParseCommand-Page, section "Description", "Tcl_ParseQuotedString" in bold at the beginning of the paragraph, 2nd sentence, last character. Copy and paste results in: "The first character of start must be "'." ---------------------------------------------------------------------- Comment By: Joe Mistachkin (mistachkin) Date: 2008-10-02 21:25 Message: The fix has now been committed to core-8-5-branch. ---------------------------------------------------------------------- Comment By: Harald Oehlmann (oehhar) Date: 2008-09-04 13:56 Message: Logged In: YES user_id=444601 Originator: YES Bug stil present in release 8.5.4, patch still solves issue. ---------------------------------------------------------------------- Comment By: Harald Oehlmann (oehhar) Date: 2008-06-30 17:39 Message: Logged In: YES user_id=444601 Originator: YES IMHO, the bug is still pending in tcl8.5.3rc1. Row 141 in ParseCmd.3 states: The first character of \fIstart\fR must be \fB\N'34'\fR. It is translated by man2tcl.exe to (Row 513): ... font B text \u0022text ' font R ... and should be: ... font B text \u0022 font R The patch still applies. ---------------------------------------------------------------------- Comment By: SourceForge Robot (sf-robot) Date: 2008-06-28 04:20 Message: Logged In: YES user_id=1312539 Originator: NO This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: Joe Mistachkin (mistachkin) Date: 2008-06-13 08:04 Message: Logged In: YES user_id=113501 Originator: NO This should now be fixed in HEAD. Still needs to be backported to 8.5. ---------------------------------------------------------------------- Comment By: Harald Oehlmann (oehhar) Date: 2008-04-04 12:02 Message: Logged In: YES user_id=444601 Originator: YES Please honor also bug reports with patches Tk 1934265 and Tk 1934272. They appear when building Tk8.5.2. The concerned files are in the Tcl8.5.2 sources. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=1934200&group_id=10894 |