I like being able to fold functions out of the way - but I don't love it more than life itself.  If it annoys others it is fine to remove.<br><br>I'll look at the tuple matching problem if I get a chance.  Same with class parameters since those should probably be a variable of some sort.
<br><br>constant.language.boolean would be the appropriate scope (constant.boolean isn't allowed)<br><br><br><div><span class="gmail_quote">On 8/15/06, <b class="gmail_sendername">William D. Neumann</b> <<a href="mailto:wneumann@cs.unm.edu">
wneumann@cs.unm.edu</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">FIXED:<br>Moved the class type part of the grammar above the class section to keep class types
<br>from being hijacked.<br><br>Added ':' as a possible end marker for methods so things are handled properly in class<br>type definitions. Note, however that this causes the method name in the type definition<br>to also appear in the symbol list, but I'm not sure that's bad.
<br><br>Corrected the spelling of tuple.<br><br>Changed the end pattern for meta.object.ocaml to allow for optional ;; after the 'end'.<br><br>Changed tag on the input and really_input snippets to inp and rinp respectively to
<br>avoid a collision with the trigger for stdin.<br><br>TODO (anyone , feel free to take these as I may be a while getting around to them):<br><br>Need to fix the tuple definition, as it is catching constructs like:<br>let f (a : int) b = ...
<br>and labeling the (a : int) as a tuple.  Also, the int is being considered a parameter<br>of the function and not a type, as it should be.  This might be fixed by matching on,<br>say (\()([^,],)+(\)), but match patterns can't have sub-patterns, so something
<br>slightly different is needed.<br><br>Folding markers are messed up for classes and modules... this needs fixing if possible.<br><br>Class parameters (e.g. the xx, yy, and zz in class blah xx yy zz = ...) aren't given<br>
any scope other than source.ocaml.  This may or may not be worth fixing.<br><br>QUESTIONS:<br>Folding markers for let function bindings are also pretty silly.  I'm wondering if<br>these shouldn't just be removed, as they are next to useless, and do little more than
<br>add extraneous fold start markers to the window.  Does anyone else agree?<br><br>Why are true and false labeled as "constant.language.pseudo-variable.ocaml"?<br>Wouldn't constant.boolean.ocaml be better?<br>
<br>William D. Neumann<br><br>Changed:<br>U   trunk/Bundles/OCaml.tmbundle/Syntaxes/OCaml.plist<br>U   trunk/Bundles/OCamlCodeCompletion.tmbundle/Snippets/input.tmSnippet<br>U   trunk/Bundles/OCamlCodeCompletion.tmbundle/Snippets/really_input.tmSnippet
<br>U   trunk/Themes/Slush & Poppies.tmTheme<br><br>_______________________________________________<br>textmate-dev mailing list<br><a href="mailto:textmate-dev@lists.macromates.com">textmate-dev@lists.macromates.com</a>
<br><a href="http://lists.macromates.com/mailman/listinfo/textmate-dev">http://lists.macromates.com/mailman/listinfo/textmate-dev</a><br></blockquote></div><br>