[svn:DBD-Pg] r14722 - in DBD-Pg/trunk: . t

[email protected]
Newsgroups gmane.comp.db.postgresql.dbdpg.cvs
Message-ID <[email protected]>
Author: turnstep
Date: Thu Feb 24 13:01:17 2011
New Revision: 14722

Modified:
   DBD-Pg/trunk/.perlcriticrc
   DBD-Pg/trunk/t/03smethod.t

Log:
Minor P:C adjustments.


Modified: DBD-Pg/trunk/.perlcriticrc
==============================================================================
--- DBD-Pg/trunk/.perlcriticrc	(original)
+++ DBD-Pg/trunk/.perlcriticrc	Thu Feb 24 13:01:17 2011
@@ -40,6 +40,7 @@
 [-Modules::ProhibitMultiplePackages]
 [-Modules::RequireBarewordIncludes]
 [-Modules::RequireEndWithOne]
+[-Modules::RequireExplicitInclusion]
 [-NamingConventions::Capitalization]
 [-References::ProhibitDoubleSigils]
 [-RegularExpressions::ProhibitCaptureWithoutTest]

Modified: DBD-Pg/trunk/t/03smethod.t
==============================================================================
--- DBD-Pg/trunk/t/03smethod.t	(original)
+++ DBD-Pg/trunk/t/03smethod.t	Thu Feb 24 13:01:17 2011
@@ -668,18 +668,20 @@
 eval {
 	# This statement will block indefinitely because of the 'FOR UPDATE' clause,
 	# so we set up an alarm to cancel it after 2 seconds.
-	my $sth = $dbh2->prepare('SELECT * FROM dbd_pg_test WHERE id = ? FOR UPDATE');
-	$sth->{RaiseError} = 1;
+	my $sthl = $dbh2->prepare('SELECT * FROM dbd_pg_test WHERE id = ? FOR UPDATE');
+	$sthl->{RaiseError} = 1;
 
-	my $action = POSIX::SigAction->new(sub {$sth->cancel},POSIX::SigSet->new(SIGALRM));
+	my $action = POSIX::SigAction->new(
+		sub {$sthl->cancel},POSIX::SigSet->new(SIGALRM));
 	$oldaction = POSIX::SigAction->new;
-	POSIX::sigaction(SIGALRM,$action,$oldaction);
+	POSIX::sigaction(SIGALRM,$action,$oldaction);  ## no critic (ProhibitCallsToUnexportedSubs)
 
 	alarm(2); # seconds before alarm
-	$sth->execute(1);
+	$sthl->execute(1);
 	alarm(0); # cancel alarm (if execute didn't block)
 };
-POSIX::sigaction(SIGALRM,$oldaction); # restore original signal handler
+# restore original signal handler
+POSIX::sigaction(SIGALRM,$oldaction); ## no critic (ProhibitCallsToUnexportedSubs)
 like ($@,qr/canceling statement due to user request/,'cancel');
 $dbh2->disconnect();
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.