Ticket #2126 (closed enhancement: fixed) — at Version 11
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
comment:1 Changed 14 years ago by zaytsev
- severity changed from no branch to on review
Branch: 2126_ini_syntax (parent: master)
Changeset: f25d1fbd2f53bcc618691be33022650b82ddce73
comment:2 Changed 14 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 14 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 14 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 14 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 14 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 14 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 14 years ago by slavazanko
Shall I merge this or you want to use this branch for something else?
Just merge.
comment:9 Changed 14 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:11 Changed 14 years ago by andrew_b
- Description modified (diff)
Don't forget remove the merged branch.