Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
Changes/Update to efm_perl.pl question(s).
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  4 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
HarleyPig  
View profile  
 More options Apr 19 2012, 6:40 pm
From: HarleyPig <alansyoung...@gmail.com>
Date: Thu, 19 Apr 2012 15:40:58 -0700 (PDT)
Local: Thurs, Apr 19 2012 6:40 pm
Subject: Changes/Update to efm_perl.pl question(s).

I submitted a pull request to update/change efm_perl.pl
( http://goo.gl/oJfqs ).

Andy asked me what kind of justification I could make for it so he could
pass that on to Bram.

This is how I responded, but I'd like other's input as well.  Am I out in
left field here?  Any additions to what I've got would be nice.

=======

At its core, efm_perl.pl reformats any error/warning messages from 'perl -c
%'. These changes don't improve on that, and I don't really see how that
could change unless perl changes the way it reports errors and warnings.

My changes handle some annoyances I have had with the output and what is
checked. These changes include ignoring the 'compilation aborted' message,
and a way to easily add additional messages to be skipped (like the
DB::single used once warning).

Also, if any of the following modules are installed; circular::require,
indirect, warnings::method or warnings::unused, it is assumed you want to
use them to check your code, and are included in the call to 'perl -c'.

Finally, when checking packages inheriting from a base package in the same
directory structure, and the library hasn't been installed on the system, a
spurious 'could not locate in @inc <https://github.com/inc>' error is
reported. Though, the way I check for this is naive, I'm not really sure
how to improve it.

I, obviously, think this would be a good change to vim. However, I realize
others will have different ideas. Perhaps I need to submit this to vim-perl
community and see what they have to say?

=======


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Andy Lester  
View profile  
 More options Apr 19 2012, 7:03 pm
From: Andy Lester <a...@petdance.com>
Date: Thu, 19 Apr 2012 18:03:19 -0500
Local: Thurs, Apr 19 2012 7:03 pm
Subject: Re: [vim-perl] Changes/Update to efm_perl.pl question(s).

On Apr 19, 2012, at 5:40 PM, HarleyPig wrote:

> This is how I responded, but I'd like other's input as well.  Am I out in left field here?  Any additions to what I've got would be nice.

And if emf_perl.pl is something that's been woefully underutilized, what can we do to help it become more useful?

Personally, I've never do syntax checking inside vim, and maybe I should.  Maybe it were butt simple I would.  Maybe it IS butt simple but I don't know it because it's an underappreciated secret.

So how can we leverage this work to be even more awesome to vim's Perl users?

xoa

--
Andy Lester => a...@petdance.com => www.petdance.com => AIM:petdance


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Paul  
View profile  
 More options Apr 20 2012, 7:54 am
From: Paul <google01...@rainslide.net>
Date: Fri, 20 Apr 2012 12:54:40 +0100
Local: Fri, Apr 20 2012 7:54 am
Subject: Re: [vim-perl] Changes/Update to efm_perl.pl question(s).

On Thursday, 19 April, 2012 at 23:40:58 BST, HarleyPig wrote:
>Also, if any of the following modules are installed; circular::require,
>indirect, warnings::method or warnings::unused, it is assumed you want to
>use them to check your code, and are included in the call to 'perl -c'.

I wouldn't assume that just because something's installed, it should always be used. That could get very annoying very quickly.

--

.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
HarleyPig  
View profile  
 More options Apr 20 2012, 10:08 am
From: HarleyPig <alansyoung...@gmail.com>
Date: Fri, 20 Apr 2012 07:08:15 -0700 (PDT)
Local: Fri, Apr 20 2012 10:08 am
Subject: Re: [vim-perl] Changes/Update to efm_perl.pl question(s).

On Friday, April 20, 2012 5:54:40 AM UTC-6, Paul wrote:

> I wouldn't assume that just because something's installed, it should
> always be used. That could get very annoying very quickly.

Right.  This is what works for me at the moment.

What I'm not sure of, is how to handle what modules to use in terms of
being setup in vim.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »