GNOME Bugzilla – Bug 501733
Syntax highlighting for IDL wanted
Last modified: 2012-12-22 10:24:34 UTC
This report has been filled here: https://bugs.launchpad.net/ubuntu/+source/gedit/+bug/172906 "When I open up an IDL script (*.pro) I don't get offered syntax highlighting. I've tried to enable it in the gedit options and still nothing. Using Kate (KDE Editor) I get the syntax highlighting to work straight away. I'm using 7.10 but I had the same problem with 7.04." IDL script: http://launchpadlibrarian.net/10728620/cpt.pro Thanks,
Which is the mime-type of the file according to Nautilus? Note that gtksourceview supports CORBA IDL but not the IDL language used in the sample.
As far as I can tell, f.d.o. database doesn't have a mime type for this IDL language. But it's not such a big deal comparing to the fact that there is no lang file, or is there?
the mime-type according to nautilus is text/plain
Closing... if someone creates a lang file we'll consider it, but until then no point keeping this open.
On http://www.ipgp.fr/~lucas/IDL_on_Gedit.html there's a language file for idl which is not too bad. It highlights at least much more than the current idl highlighting.
I am using the above file now, it is great finally having some highlighting, would be cool if it could be included!
Created attachment 228738 [details] IDL Lang file Attached the lang file as I have requested via the mailing list that it be added. Also I renamed it from idl-itt to idl-exelis as the company has had a name change.
Created attachment 228740 [details] IDL Lang file v2 ok I've fixed some a couple of errors in the old file and added a whole heap of missing functions and keywords
Created attachment 228743 [details] IDL Lang file v3 A few more fixs and additions
Created attachment 228744 [details] [review] IDL Lang file v4 Ok, I've cleaned up the formatting and turned it into a patch
Created attachment 228746 [details] [review] IDL Lang file v5 Fixed escaping of strings
Reopening or it will get lost (feel free to reopen in these cases)
What else needs to be provided to get this patch accepted?
Review of attachment 228746 [details] [review]: What is missing is time and strength to review :-) A couple of comments below and we are good to go ::: data/language-specs/idl-exelis.lang @@ +21,3 @@ +<language id="idl-exelis" _name="IDL-Exelis" version="2.0" _section="Scientific"> + <metadata> + <property name="mimetypes">text/x-idl</property> is this an official mime type included in shared-mime-info? If not, let's remove it, this metadata is not mandatory @@ +77,3 @@ + <context id="operators" style-ref="keyword"> + <keyword>TRUE</keyword> + <keyword>FALSE</keyword> are TRUE and FALSE operators?
Created attachment 232108 [details] [review] IDL Lang file v6 Removed mime type. Remove TRUE/FALSE operators they should not have been there, IDL does not even have Boolean types.
Thanks for your patience. I committed with some amendments (noted here if you happen to submit lang files in the future) - added the file to Makefile.am - added the file to POTFILES.in - removed the "types" style that as far as I can see was not used