Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Comparing changes

Choose two branches to see what's changed or to start a new pull request. If you need to, you can also compare across forks.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also compare across forks.
base: 0fc30f82eb0695908b813644402a780edeaba23d
...
compare: c6ff768d3afe5cc60f5982d4ed4266e1f9a089c1
Checking mergeability… Don't worry, you can still create the pull request.
  • 1 commit
  • 1 file changed
  • 0 commit comments
  • 1 contributor
Showing with 1 addition and 1 deletion.
  1. +1 −1  rejected/prog_perl_4th.mdwn
View
2  rejected/prog_perl_4th.mdwn
@@ -1,6 +1,6 @@
Programming Perl 4th Edition is a good book, however it is also a book with a lot of history. Huge parts of its contents stem from 1991 and have a very outdated style of Perl. 4th Edition tries to upgrade this somewhat, but did not to so with enough diligence to permit me to recommend it. Some specific issues:
* strict and warnings are not higlighted at all. In the preface chapter there are two code examples without strict and one is written as if it was not in effect. In fact, strict first occurs on the 14th page of the first chapter on the third page of a sub-chapter at the bottom of a page.
-* On later pages in the book code examples consistently act as if there was no strict present.
+* <s>On later pages in the book code examples consistently act as if there was no strict present.</s> This was incorrect, $a and $b were being used and are always declared since they're [[magical variables|https://metacpan.org/module/perlvar]]. I'm leaving this here because i think it presents an issue in itself.
* given/when and smartmatching are one of the first things highlighted. That is unfortunate since these features are rife with issues, inconsistencies and their future is pretty contentious.
* The first filehandles encountered are of the bareword kind. Others have written mounds on why this is not a wise choice and i do not think this should be one of the first things a newbie sees.

No commit comments for this range

Something went wrong with that request. Please try again.