Curly braces inside q// match against real curly braces #86

Closed
shlomif opened this Issue Sep 12, 2012 · 4 comments

2 participants

@shlomif

In this file:

http://www.shlomifish.org/Files/files/code/vim-perl-issue-curly-braces-inside-q.txt

One cannot match the very first curly brace (at the start of line 2) because the ones inside the q/../ qualify as such. I am using commit b94578e .

Please look into it.

Regards,

-- Shlomi Fish

P.S: I discovered that one cannot attach files to issues of GitHub's issue tracker:

http://stackoverflow.com/questions/10963205/how-to-attach-file-to-a-github-issue

I guess it can suck a lot less.

@hoelzro
vim-perl member

@shlomif I noticed the file you linked to is gone. Could you please restore it?

@shlomif

@hoelzro : I am including it below:

# Test the { ? and { [command] commands.
{
    my $wrapper = DebugWrap->new(
        {
            cmds =>
            [
                q/{ ?/,
                q/{ l/,
                q/{ ?/,
                q/b 5/,
                q/c/,
                q/q/,
            ],
            prog => '../lib/perl5db/t/disable-breakpoints-1',
        }
    );

    $wrapper->contents_like(qr#
        ^No\ pre-debugger\ actions\.\n
        .*?
        ^pre-debugger\ commands:\n
        \s+\{\ --\ l\n
        .*?
        ^5==>b\s+\$x\ =\ "FirstVal";\n
        6\s*\n
        7:\s+\$dummy\+\+;\n
        8\s*\n
        9:\s+\$x\ =\ "SecondVal";\n

        #msx,
        'Test the pre-prompt debugger commands',
    );
}

Call it "vim.pl" and edit it using gvim.

By the way, why did it take so long to respond?

Regards,

-- @shlomif .

@hoelzro
vim-perl member

@shlomif Sorry for the long response time; I joined this group to make improvements to the Perl 6 syntax file. However, the 7.4 release is coming up, and I'd like to resolve large outstanding bugs before we send Bram the files to include in the release.

@hoelzro
vim-perl member

This is now fixed, as long as you're using the matchit plugin. Unfortunately, Vim doesn't support this kind of thing for non-double quoted strings by default.

@hoelzro hoelzro closed this May 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment