about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2020-09-09 06:26:09 +0000
committerEric Wong <e@80x24.org>2020-09-10 19:45:18 +0000
commit38226be5fd4c24885658e00d4ce87ccee663063d (patch)
tree1cb97f932a099655877ed07a3b3df44f22859128
parent6b13c85d2b1b94973c374fa3444148b49d1d8a19 (diff)
downloadpublic-inbox-38226be5fd4c24885658e00d4ce87ccee663063d.tar.gz
With Perl upstream preparing to deprecate things, we'll move
towards only enabling warnings during development via shebang
and stop enabling them via "use".

We'll also favor "use v5.10.1" over the Perl 5.6-compatible "use
5.010_001", since our code base never worked on 5.6.

Finally, were also importing SEEK_SET without using it, just use it
for readability since we can't avoid loading Fcntl in other
places and it'll get constant-folded, anyways.
-rw-r--r--lib/PublicInbox/SolverGit.pm5
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/PublicInbox/SolverGit.pm b/lib/PublicInbox/SolverGit.pm
index ff8a4946..dd95f400 100644
--- a/lib/PublicInbox/SolverGit.pm
+++ b/lib/PublicInbox/SolverGit.pm
@@ -9,8 +9,7 @@
 # local filesystem layouts in the process.
 package PublicInbox::SolverGit;
 use strict;
-use warnings;
-use 5.010_001;
+use v5.10.1;
 use File::Temp 0.19 (); # 0.19 for ->newdir
 use Fcntl qw(SEEK_SET);
 use PublicInbox::Git qw(git_unquote git_quote);
@@ -270,7 +269,7 @@ sub prepare_index ($) {
         my $in = tmpfile("update-index.$oid_full") or die "tmpfile: $!";
         print $in "$mode_a $oid_full\t$path_a\0" or die "print: $!";
         $in->flush or die "flush: $!";
-        sysseek($in, 0, 0) or die "seek: $!";
+        sysseek($in, 0, SEEK_SET) or die "seek: $!";
 
         dbg($self, 'preparing index');
         my $rdr = { 0 => $in };