Re: [PATCH] gitweb fix validating pg (page) parameter

Matthias Lederhofer [off-list ref] writes:

quoted

quoted

+	if ($page =~ m/[^0-9]/) {

...
The first page seems to have number 0.  After removing the 1 <= $page
this code should do the same as my patch, apply whatever you think is
easier to read (I prefer if over unless most of the time).

Ah, I did not notice we call the first page 0; barfing on any
non-digit in the string like you do sounds quite sane.

Thanks.