Tags:
create new tag
, view all tags
This is a minor issue that I don't have strong feelings about, but I wanted to mention.

Take a look at the diff between r1.71 and r1.70 of the BeijingRelease page. Notice that the first two changes (first four table rows) show only part of the row, which includes only, for example:

Changed: < <

PeterThoeny 0% 0% 0%
>
>
PeterThoeny 100% 100% 95%

It would be much easier to find the specific row changed if the entire row were displayed in the diff. (I realize TWiki depends on the standard features of RCS, and I suspect they don't support something like this -- it's just a "nice to have" kind of thing.)

(Maybe there is some kind of workaround -- maybe TWiki could insert a "dummy change" of some sort in the first cell of a row if any cell in the row has been changed?)

-- RandyKramer - 18 Jul 2002

ok - I'm going to call this a bug, but i'm not going to rush doing it yet

if you look at the debug diff output (http://twiki.org/cgi-bin/rdiff/Codev/BeijingRelease?rev1=1.71&rev2=1.70&context=10&render=debug), this is due to using \ to make a row span multiple lines

type: u
| | 50% | 10% | 0% | | VerbatimExpandsVariables \ | Main.JohnTalintyre | 100% | 100% | 100% | | NestedVerbatimTags \ | Main.JohnTalintyre | 100% | 100% | 100% | | ProposalForNewlineReplacement \ | Main.PeterThoeny | 100% | 100% | 100% | | [[Telnet URL Support]] \ | Main.PeterThoeny | 100% | 100% | 100% | | RenameTestWebToSandboxWeb \
| | 50% | 10% | 0% | | VerbatimExpandsVariables \ | Main.JohnTalintyre | 100% | 100% | 100% | | NestedVerbatimTags \ | Main.JohnTalintyre | 100% | 100% | 100% | | ProposalForNewlineReplacement \ | Main.PeterThoeny | 100% | 100% | 100% | | [[Telnet URL Support]] \ | Main.PeterThoeny | 100% | 100% | 100% | | RenameTestWebToSandboxWeb \
type: c
| Main.PeterThoeny | 0% | 0% | 0% |
| Main.PeterThoeny | 100% | 100% | 95% |

so the diff interpreter needs to re-combine \'ed lines before it hist the renderer (i think)

-- SvenDowideit - 09 May 2004

Yes, I'm going to slack off on this and leave it for DakarRelease -- SvenDowideit - 27 Jun 2004

Edit | Attach | Watch | Print version | History: r7 < r6 < r5 < r4 < r3 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r7 - 2005-04-24 - WillNorris
 
  • Learn about TWiki  
  • Download TWiki
This site is powered by the TWiki collaboration platform Powered by Perl Hosted by OICcam.com Ideas, requests, problems regarding TWiki? Send feedback. Ask community in the support forum.
Copyright © 1999-2017 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.