pkgsrc-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint pkgtools/pkglint: remove mkalign code



details:   https://anonhg.NetBSD.org/pkgsrc/rev/95a1429637e8
branches:  trunk
changeset: 345586:95a1429637e8
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Sun Dec 15 01:29:06 2019 +0000

description:
pkgtools/pkglint: remove mkalign code

This is an early prototype that is not used in the pkglint code. It will
be included when it is ready and tested.

diffstat:

 pkgtools/pkglint/PLIST                 |    4 +-
 pkgtools/pkglint/files/mkalign.go      |  233 --------------------------------
 pkgtools/pkglint/files/mkalign_test.go |  236 ---------------------------------
 3 files changed, 1 insertions(+), 472 deletions(-)

diffs (truncated from 495 to 300 lines):

diff -r 72786a11a04e -r 95a1429637e8 pkgtools/pkglint/PLIST
--- a/pkgtools/pkglint/PLIST    Sun Dec 15 00:10:29 2019 +0000
+++ b/pkgtools/pkglint/PLIST    Sun Dec 15 01:29:06 2019 +0000
@@ -1,4 +1,4 @@
-@comment $NetBSD: PLIST,v 1.21 2019/12/13 01:39:23 rillig Exp $
+@comment $NetBSD: PLIST,v 1.22 2019/12/15 01:29:06 rillig Exp $
 bin/pkglint
 gopkg/pkg/${GO_PLATFORM}/netbsd.org/pkglint.a
 gopkg/pkg/${GO_PLATFORM}/netbsd.org/pkglint/getopt.a
@@ -49,8 +49,6 @@
 gopkg/src/netbsd.org/pkglint/lineslexer_test.go
 gopkg/src/netbsd.org/pkglint/logging.go
 gopkg/src/netbsd.org/pkglint/logging_test.go
-gopkg/src/netbsd.org/pkglint/mkalign.go
-gopkg/src/netbsd.org/pkglint/mkalign_test.go
 gopkg/src/netbsd.org/pkglint/mkassignchecker.go
 gopkg/src/netbsd.org/pkglint/mkassignchecker_test.go
 gopkg/src/netbsd.org/pkglint/mkcondchecker.go
diff -r 72786a11a04e -r 95a1429637e8 pkgtools/pkglint/files/mkalign.go
--- a/pkgtools/pkglint/files/mkalign.go Sun Dec 15 00:10:29 2019 +0000
+++ /dev/null   Thu Jan 01 00:00:00 1970 +0000
@@ -1,233 +0,0 @@
-package pkglint
-
-type MkAlignFile struct {
-       Paras []*MkAlignPara
-}
-
-func (f *MkAlignFile) AlignParas() {
-       //  #.  Ein einzelner Absatz, der einen Tab weniger eingerückt ist als die übrigen,
-       //      darf auf die Einrückung der anderen Absätze angeglichen werden,
-       //      sofern der Absatz dadurch nicht zu breit wird.
-       panic("implement me")
-}
-
-type MkAlignPara struct {
-}
-
-func (p *MkAlignPara) IsAligned() bool {
-       //  #.  Das Ausrichten mit mehr als 1 Tab ist erlaubt, wenn die Ausrichtung einheitlich ist.
-
-       //  #.  Wenn VarOp über die Ausrichtung hinausragt (Ausrei�er),
-       //      darf zwischen VarOp und Wert statt der Ausrichtung 1 Leerzeichen sein.
-
-       //  #.  Das Verhältnis zwischen Tab-Zeilen und hinausragenden Zeilen muss ausgewogen sein.
-       //      Nicht zu viele hinausragende Zeilen. (Noch zu definieren.)
-       //      Möglicher Ansatz: Anteil der Leerfläche?
-
-       panic("implement me")
-}
-
-func (p *MkAlignPara) IsOutlier(line *MkAlignLine) bool {
-       //  #.  Wenn VarOp über die Ausrichtung hinausragt (Ausrei�er),
-       //      darf zwischen VarOp und Wert statt der Ausrichtung 1 Leerzeichen sein.
-       panic("implement me")
-}
-
-// ValueAlignment returns the column at which all values of the paragraph
-// are aligned, or false if they aren't.
-//
-//  #.  Die Werte aller Zeilen sind mit Tabs an einer gemeinsamen vertikalen Linie
-//      (Ausrichtung) ausgerichtet.
-func (p *MkAlignPara) ValueAlignment() (bool, int) {
-       panic("implement me")
-}
-
-func (p *MkAlignPara) MinValueAlignment() int {
-       //  #.  Die minimale Ausrichtung ergibt sich aus der maximalen Breite von # und VarOp
-       //      aller Zeilen, gerundet zum nächsten Tabstopp.
-       //      Dabei zählen auch Zeilen mit, die rechts von VarOp komplett leer sind.
-       panic("implement me")
-}
-
-func (p *MkAlignPara) MaxValueAlignment() int {
-       //  #.  Die maximale Ausrichtung ergibt sich aus der maximalen Breite von Wert
-       //      und Kommentar, abgezogen vom maximalen rechten Rand (in Spalte 73).
-       panic("implement me")
-}
-
-func (p *MkAlignPara) MayAlignValuesTo(column int) bool {
-       panic("implement me")
-}
-
-func (p *MkAlignPara) AlignValuesTo(column int) {
-       //  #.  Beim Umformatieren darf die Zeilenbreite die 73 Zeichen nicht überschreiten,
-       //      damit am rechten Rand eindeutig ist, wo jede Zeile aufhört.
-       //      Zeilen, die bereits vorher breiter waren, dürfen ruhig noch breiter werden.
-
-       panic("implement me")
-}
-
-type MkAlignMkLine struct {
-}
-
-func (l *MkAlignMkLine) RightMargin() int {
-       //  #.  Jede MkZeile hat für alle ihre Zeilen einen gemeinsamen rechten Rand.
-
-       //  #.  Um den gemeinsamen rechten Rand zu bestimmen, werden alle Zeilen ignoriert,
-       //      in denen die Fortsetzung durch 1 Leerzeichen abgetrennt ist.
-
-       //  #.  Einzelne Fortsetzungen dürfen über den rechten Rand hinausragen.
-       //      Die Fortsetzung wird dann durch 1 Leerzeichen abgetrennt.
-
-       panic("implement me")
-}
-
-func (l *MkAlignMkLine) IsCanonical() bool {
-       //  #.  Eine leere Erstzeile mit 1 fortgesetzer Zeile ist nur zulässig,
-       //      wenn die kombinierte Zeile breiter als 73 Zeichen wäre.
-       //      Sonst werden die beiden Zeilen kombiniert.
-
-       //  ### Mehrzeilig, fortgesetzte Zeilen
-       //
-       //  #.  Nach einer leeren Erstzeile ist die erste fortgesetzte Zeile an der
-       //      Ausrichtung aller Zeilen eingerückt, wenn die Erstzeile über die
-       //      Ausrichtung ragt und der Platz aller Zeilen es zulässt, andernfalls
-       //      mit 1 Tab.
-       //
-       //  #.  Bei mehrzeiligen einrückbaren Werten (AWK, Shell, Listen aus Tupeln)
-       //      dürfen die weiteren Fortsetzungszeilen weiter eingerückt sein als die erste.
-       //      Ihre Einrückung besteht aus Tabs, gefolgt von 0 bis 7 Leerzeichen.
-
-       // In the continuation lines, each follow-up line is indented with at least
-       // one tab, to avoid confusing them with regular single-lines. This is
-       // especially true for CONFIGURE_ENV, since the environment variables are
-       // typically uppercase as well.
-
-       //  MULTI_LINE= \
-       //          The value starts in the second line.
-       //
-       // The backslash in the first line is usually aligned to the other variables
-       // in the same paragraph. If the variable name is longer than the indentation
-       // of the paragraph, it may be indented with a single space.
-       //
-       // In multi-line shell commands or AWK programs, the backslash is
-       // often indented to column 73, as are the backslashes from the follow-up
-       // lines, to act as a visual guideline.
-
-       // Continuation lines may or may not have their value in the first line.
-
-       // In general, all values should be aligned using tabs.
-       // As an exception, a single very long line (called an outlier) may be
-       // aligned with a single space.
-       // A typical example is a SITES.very-long-file-name.tar.gz variable
-       // between HOMEPAGE and DISTFILES.
-
-       panic("implement me")
-}
-
-func (l *MkAlignMkLine) HasCanonicalRightMargin() bool {
-       //  #.  Die Fortsetzungen jeder MkZeile sind entweder alle durch je 1 Leerzeichen abgetrennt,
-       //      oder alle Fortsetzungen sind am rechten Rand.
-
-       //  #.  Einzelne Fortsetzungen dürfen über den rechten Rand hinausragen.
-       //      Die Fortsetzung wird dann durch 1 Leerzeichen abgetrennt.
-
-       //  #.  Die Fortsetzung der Erstzeile mit Wert ist durch 1 Leerzeichen abgetrennt,
-       //      wenn sie rechts von der Ausrichtung steht,
-       //      andernfalls durch Tabs an der Ausrichtung.
-
-       panic("implement me")
-}
-
-func (l *MkAlignMkLine) CurrentValueAlign() int {
-       // The indentation of the first value of the variable determines the minimum
-       // indentation for the remaining continuation lines.
-       //
-       // To allow long variable
-       // values to be indented as little as possible, the follow-up lines only need
-       // to be indented by a single tab, even if the other
-       // variables in the paragraph are aligned further to the right.
-       //
-       // If the
-       // indentation is not a single tab, it must match the indentation of the
-       // other lines in the paragraph.
-       panic("implement me")
-}
-
-type MkAlignLine struct {
-       Comment          string
-       VarOp            string
-       SpaceBeforeValue string
-       Value            string
-       SpaceAfterValue  string
-       Continuation     string
-}
-
-func (l *MkAlignLine) HasCanonicalRightMargin(valueAlignColumn int) bool {
-       panic("implement me")
-}
-
-func (l *MkAlignLine) IsCanonicalSingle() bool {
-       panic("implement me")
-}
-
-func (l *MkAlignLine) IsCanonicalLeadEmpty() bool {
-       //  SHELL_CMD=                                                              \
-       //          if ${PKG_ADMIN} pmatch ${PKGNAME} ${dependency}; then           \
-       //                  ${ECHO} yes;                                            \
-       //          else                                                            \
-       //                  ${ECHO} no;                                             \
-       //          fi
-       //
-       //
-       panic("implement me")
-}
-
-func (l *MkAlignLine) IsCanonicalLeadValue() bool {
-       //  MULTI_LINE=     The value starts in the first line \
-       //                  and continues in the second line.
-       //
-       // In lists or plain text, like in the example above, all values are
-       // aligned in the same column. Some variables also contain code, and in
-       // these variables, the line containing the first word defines how deep
-       // the follow-up lines must be indented at least.
-
-       panic("implement me")
-}
-
-func (l *MkAlignLine) IsCanonicalFollowLead() bool {
-       //  SHELL_CMD=                                                              \
-       // -->      if ${PKG_ADMIN} pmatch ${PKGNAME} ${dependency}; then           \
-       //                  ${ECHO} yes;                                            \
-       //          else                                                            \
-       //                  ${ECHO} no;                                             \
-       //          fi
-       //
-       //  MULTI_LINE=     The value starts in the first line \
-       //                  and continues in the second line.
-       //
-       // In lists or plain text, like in the example above, all values are
-       // aligned in the same column. Some variables also contain code, and in
-       // these variables, the line containing the first word defines how deep
-       // the follow-up lines must be indented at least.
-
-       panic("implement me")
-}
-
-func (l *MkAlignLine) IsCanonicalFollow() bool {
-       // In the continuation lines, each follow-up line is indented with at least
-       // one tab, to avoid confusing them with regular single-lines. This is
-       // especially true for CONFIGURE_ENV, since the environment variables are
-       // typically uppercase as well.
-
-       //  SHELL_CMD=                                                              \
-       //          if ${PKG_ADMIN} pmatch ${PKGNAME} ${dependency}; then           \
-       //                  ${ECHO} yes;                                            \
-       //          else                                                            \
-       //                  ${ECHO} no;                                             \
-       //          fi
-       //
-       //
-
-       panic("implement me")
-}
diff -r 72786a11a04e -r 95a1429637e8 pkgtools/pkglint/files/mkalign_test.go
--- a/pkgtools/pkglint/files/mkalign_test.go    Sun Dec 15 00:10:29 2019 +0000
+++ /dev/null   Thu Jan 01 00:00:00 1970 +0000
@@ -1,236 +0,0 @@
-package pkglint
-
-import "gopkg.in/check.v1"
-
-func (s *Suite) Test_MkAlignFile_AlignParas(c *check.C) {
-       t := s.Init(c)
-
-       test := func() {
-               // TODO
-               t.CheckEquals(true, true)
-       }
-
-       test()
-}
-
-func (s *Suite) Test_MkAlignPara_IsAligned(c *check.C) {
-       t := s.Init(c)
-
-       test := func() {
-               // TODO
-               t.CheckEquals(true, true)
-       }
-
-       test()
-}
-
-func (s *Suite) Test_MkAlignPara_IsOutlier(c *check.C) {
-       t := s.Init(c)
-
-       test := func() {
-               // TODO
-               t.CheckEquals(true, true)
-       }
-
-       test()
-}
-
-func (s *Suite) Test_MkAlignPara_ValueAlignment(c *check.C) {
-       t := s.Init(c)
-
-       test := func() {



Home | Main Index | Thread Index | Old Index