[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint/files/intqa pkgtools/pkglint: remove ...

[rillig <rillig%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/cdf4c372b7f9
branches:  trunk
changeset: 347030:cdf4c372b7f9
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Mon Jan 06 20:39:35 2020 +0000

description:
pkgtools/pkglint: remove outdated ideas.go

diffstat:

 pkgtools/pkglint/files/intqa/ideas.go |  17 -----------------
 1 files changed, 0 insertions(+), 17 deletions(-)

diffs (21 lines):

diff -r e5692cd69e1b -r cdf4c372b7f9 pkgtools/pkglint/files/intqa/ideas.go
--- a/pkgtools/pkglint/files/intqa/ideas.go     Mon Jan 06 20:39:00 2020 +0000
+++ /dev/null   Thu Jan 01 00:00:00 1970 +0000
@@ -1,17 +0,0 @@
-package intqa
-
-// XXX: It might be nice to check all comments of the form "See XYZ"
-// to see whether XYZ actually exists. The scope should be the current type,
-// then the current package, then a package-qualified identifier.
-// As if there were a "_ = XYZ" at the beginning of the function.
-
-// XXX: If there is a constructor for a type, only that constructor may be used
-// for constructing objects. All other forms (var x Type; x := &Type{}) should be forbidden.
-
-// Each test must call its testee, if the testee is callable at all.
-//
-// If it doesn't, the name of the test is misleading. A typical case where
-// this happens is copy-and-paste mistakes combined with incomplete test
-// cases.
-//
-// To check this, every testee must be instrumented.