mark next/prev with peer text widgets

Brought to you by: andreas_kupries, dgp, hobbs, kennykb, and 2 others

#320 mark next/prev with peer text widgets

Status: closed-fixed

Owner: Jan Nijtmans

Labels: 18. [text] (26)

Priority: 7

Updated: 2012-01-22

Created: 2012-01-10

Creator: Francois VOGEL

Private: No

The attached patch fixes items 3288113 and 3288121:

Missing marks when searching through mark next from peers:
https://sourceforge.net/tracker/?func=detail&aid=3288113&group_id=12997&atid=112997

Endless loop with mark prev from peer text widget:
https://sourceforge.net/tracker/?func=detail&aid=3288121&group_id=12997&atid=112997

The patch is against the latest Fossil sources.

The existing textMark.test suite passes with no failure.
Plus I have added one test item for each issue above that the patch is fixing.

Discussion

Francois VOGEL - 2012-01-10

Patch for items 3288113 and 3288121

3288113_3288121_V1.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Francois VOGEL - 2012-01-10

assigned_to: hobbs --> vincentdarley
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Francois VOGEL - 2012-01-18

assigned_to: vincentdarley --> nijtmans
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Jan Nijtmans - 2012-01-19

Patch applied in branch patch-3471873.

After adding test textmark-7.9, various tests textmark-8.x started
failing due to leftover marks, so converted those to new-style
tktest format and added cleanup code. With this change,
all tests now pass.

As far as I can see, everthing is OK to be merged to the
core-8-5-branch and to trunk.

François, agreed?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Francois VOGEL - 2012-01-19

priority: 5 --> 7
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Francois VOGEL - 2012-01-19

Sure, please proceed.
Thanks a lot again for your help!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Jan Nijtmans - 2012-01-22

status: open --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Jan Nijtmans - 2012-01-22

Merged to trunk and to core-8-5-branch. Everything looks OK to me.

Thanks! François!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link: