forall seems to break syntax

[ghost]

Example:

Looks like it's somehow related to the parentheses? Wrapping the return value in parens "fixes" it:

[faustinoaq]

Good catch!

I think this issue is related to #9

BTW, Do you know if sublime text have this issue?

[Proximyst]

I would like to add how this works fine:

    def convert?(input : String) : T? forall T
      converter = ConversionManager.global_converters![T.name]
      converter ||= convert
    end

This however only cursives the method in question instead of everything:

      def inner! : TypeConverter(T) forall T
        @inner.as TypeConverter(T)
      end

[faustinoaq]

Hi @Proximyst Thank you for your comment!, To fix this issue I need to do some changes in current syntax file

An alternative is replacing current json file and return back to tmLanguage file, but I don't know if ST has this issue as well 😅

[Proximyst]

The fact there are no issues on it so far indicates it most likely isn't there. You could however try to type out the snippet yourself.

I by the way found out why the inner! method had cursive, and that's as the initialize method uses forall as well, but is an empty method.

module Cropt
  module Internal
    struct TypeConverter
      def initialize(@inner : Cropt::TypeConverter(T)) forall T
      end

      def inner! : TypeConverter(T) forall T
        @inner.unsafe_as TypeConverter(T)
      end
    end
  end
end

[faustinoaq]

An alternative is replacing current json file and return back to tmLanguage file, but I don't know if ST has this issue as well sweat_smile

The fact there are no issues on it so far indicates it most likely isn't there.

So, I think I gonna replace my custom crystal.json syntax by official TextMate Crystal.tmLanguage or Sublime Crystal.tmLanguage 😅 WDYT?

[faustinoaq]

Looks like is working fine with TextMate Crystal.tmLanguage 🎉

Will be available on next release 😉

[Proximyst]

That looks great. Thanks for the look into the issue ^^

[faustinoaq]

next minor version 0.3.11 (available soon on vscode marketplace) fixes this (thanks to @icyleaf)