Ticket #2126 (closed enhancement: fixed) — at Version 11

Opened 12 years ago

Last modified 12 years ago

ini.syntax

Reported by: zaytsev Owned by: zaytsev
Priority: trivial Milestone: 4.7.2
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master committed-stable

Description (last modified by andrew_b) (diff)

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=567046

From: Karsten Hilbert <karsten.hilbert@gmx.net>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: mc: INI syntax file proposition
Date: Tue, 26 Jan 2010 22:49:38 +0100
[Message part 1 (text/plain, inline)]
Package: mc
Version: 3:4.7.0.1-1
Severity: wishlist
Tags: patch
I am attaching a proposed syntax highlighting definition for
- gasp - DOS style INI files. You,
	[groups]
with
	option = value
and such. It would probably need to be tied to *.ini files.
Personally I also use it for *.conf files but that is by no
means safe to assume to be an INI style file.
I'd love to set the
	option
part in
	option = value
to some other color, too, but didn't find the proper syntax
for that.
Karsten

Change History

Changed 12 years ago by zaytsev

comment:1 Changed 12 years ago by zaytsev

  • severity changed from no branch to on review

Branch: 2126_ini_syntax (parent: master)
Changeset: f25d1fbd2f53bcc618691be33022650b82ddce73

comment:2 Changed 12 years ago by andrew_b

  • Owner set to zaytsev
  • Status changed from new to assigned
  • Votes for changeset set to andrew_b

comment:3 Changed 12 years ago by zaytsev

Could someone clever please tell me how to set option = value to some other color as per original request? I'd be happy to apply the patch and squash the branch.

comment:4 Changed 12 years ago by slavazanko

  • Votes for changeset changed from andrew_b to andrew_b slavazanko
  • severity changed from on review to approved

comment:5 Changed 12 years ago by zaytsev

Did anyone see my question above about highlighting of the first part? I have no clue how to implement this correctly, but if someone comes up with the solution after merge I'll have to reopen and merge again for just few lines... Egh?

comment:6 Changed 12 years ago by slavazanko

See changeset:a25431452305fdc471bcccd3bdbcbe9b5965113f

Also, no need to create separate branch (and tickect) for syntax-highlight related patches: http://midnight-commander.org/wiki/WorkingGuideLines#Directcommits

comment:7 Changed 12 years ago by zaytsev

  • Status changed from assigned to accepted

Hi Slava!

Looks cool! Shall I merge this or you want to use this branch for something else? Sorry for creating the branch, I'm out of Jabber till at least next week. I would have asked there otherwise.

Z.

comment:8 Changed 12 years ago by slavazanko

Shall I merge this or you want to use this branch for something else?

Just merge.

comment:9 Changed 12 years ago by zaytsev

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b slavazanko to committed-master committed-stable
  • Resolution set to fixed
  • severity changed from approved to merged

Committed to master. Merge changeset: bfd0b2c9953dfa5a0591ee243a1532ad44138463.
Cherry-picked to 4.7.0-stable, changeset: 03fdddfdccfa6eba501b66da208db032bdb44572.

comment:10 Changed 12 years ago by zaytsev

  • Status changed from testing to closed

comment:11 Changed 12 years ago by andrew_b

  • Description modified (diff)

Don't forget remove the merged branch.

Note: See TracTickets for help on using tickets.