Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix line jump when started as editor #2344

Closed
mc-butler opened this issue Sep 9, 2010 · 7 comments
Closed

Fix line jump when started as editor #2344

mc-butler opened this issue Sep 9, 2010 · 7 comments
Assignees
Labels
area: mcedit mcedit, the built-in text editor prio: low Minor problem or easily worked around
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2344
Reporter zaytsev (@zyv)

Two common line jump protocols supported by mc are:

  • mcedit +lineno file
  • mcedit file:lineno

Unfortunately, in the implementation of the first protocol, if line
number equals to zero, which is often used to position the cursor at the beginning of the file, it is completely ignored and file "+0" is opened instead.

This patch fixes aforementioned issue.

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 9, 2010 at 8:41 UTC (comment 1)

  • Priority changed from major to minor
  • Severity changed from no branch to on review

Branch: 2344_fix_line_jump (parent: master)
Changeset: [73ee1d0]

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 9, 2010 at 8:42 UTC (comment 2)

  • Keywords set to stable-candidate

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 9, 2010 at 8:43 UTC

Patch for the stable branch

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 9, 2010 at 8:54 UTC (comment 3)

  • Votes set to andrew_b

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Sep 9, 2010 at 10:36 UTC (comment 4)

  • Votes changed from andrew_b to andrew_b angel_il
  • Severity changed from on review to approved

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 9, 2010 at 10:41 UTC (comment 5)

  • Severity changed from approved to merged
  • Votes changed from andrew_b angel_il to committed-master

Committed to master. Merge changeset: [7133f4b].

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 9, 2010 at 10:44 UTC (comment 6)

  • Keywords stable-candidate deleted
  • Votes changed from committed-master to committed-master committed-stable
  • Status changed from new to closed
  • Resolution set to fixed

Cherry-picked to 4.7.0-stable, changeset: [7c3d8ef].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: mcedit mcedit, the built-in text editor prio: low Minor problem or easily worked around
Development

No branches or pull requests

2 participants