Grokbase
Topics Posts Groups | in
x
[ help ]

Re: Warn on occluded global - question [PATCH]

View PostFlat  Thread  Threaded | < Prev - Next >
Rafael Garcia-Suarez Re: Warn on occluded global - question [PATCH]
| +1 vote
[ Profile | Reply to group ] [ Flat  Thread  Threaded ]
2008/11/17 Peter Scott <Peter@psdt.com>:
> At 06:57 AM 11/17/2008, demerphq wrote:
>>
>> Im wondering tho, isnt part of your education that anything named with
>> ALLCAPS, or "punctuation style" is "special" and should be treated
>> specially and carefully?
>
> Except that I cite ALLCAPS early on as a best practice for naming
> constants...
>
> So, I return to the original question: If I beef this patch up to industrial
> strength, will it be accepted?

I must admit that I see no compelling reason to add this warning. It's
not extremely consistent, and not extremely useful.

Thread : Warn on occluded global - question [PATCH]
1)
Peter Scott The patch below does not include tests or documentation, which I will happily add if there is...
2)
Rafael Garcia-Suarez 2008/11/15 Peter Scott <Peter@psdt.com>: I'll be neutral about the idea, until further comments,...
3)
Ben Morrow Quoth rgarciasuarez@gmail.com ("Rafael Garcia-Suarez"): I don't really like it (if I'm entitled to...
4)
Abigail But the mentioned globals *aren't* lexical. Triggering your suggested warning would be quite an...
5)
Peter Scott Exactly, which is why I stuck to the basic ones in perlvar. I was very hesitant about putting in...
6)
Paul Johnson Well, if you're looking for consistency ... I don't know though - is this a real problem? Reusing...
7)
Peter Scott It's not like people are going, "I need a variable... I know, I'll call it %INC.' It's that after...
8)
Gabor Szabo I am personally quite in favor making Perl more beginner safe. That said, there are tons of things...
9)
Chip Salzenberg I am tempted to cite mjd channeling Klortho the Magnificent: "If you don't know what it does, why...
10)
demerphq 2008/11/16 Peter Scott <Peter@psdt.com>: Im wondering tho, isnt part of your education that...
11)
Peter Scott Except that I cite ALLCAPS early on as a best practice for naming constants... So, I return to the...
12)
Rafael Garcia-Suarez 2008/11/17 Peter Scott <Peter@psdt.com>: I must admit that I see no compelling reason to add this...
13)
Peter Scott Global bareword filehandles? Seems to be already covered: $ perl -wle 'my ARGVOUT' No such class...
14)
Moritz Lenz I'm in favor for that warning, under two conditions: 1) could we please use a word that a...
15)
Ronald J Kimball I don't have an opinion either way on whether the warning should be added, but I was thinking the...
spacer
View PostFlat  Thread  Threaded | < Prev - Next >