Add package check
authorFrank Duncan <frank@kank.net>
Sat, 11 Jul 2015 00:53:07 +0000 (19:53 -0500)
committerFrank Duncan <frank@kank.net>
Sat, 11 Jul 2015 00:53:07 +0000 (19:53 -0500)
resources/nopackage.lisp [new file with mode: 0644]
src/main/checker.asd [new file with mode: 0644]
src/main/checker.lisp [new file with mode: 0644]
src/main/package.lisp [new file with mode: 0644]

diff --git a/resources/nopackage.lisp b/resources/nopackage.lisp
new file mode 100644 (file)
index 0000000..d04af50
--- /dev/null
@@ -0,0 +1 @@
+(defun hello-world () "Hello world")
diff --git a/src/main/checker.asd b/src/main/checker.asd
new file mode 100644 (file)
index 0000000..b714aa3
--- /dev/null
@@ -0,0 +1,3 @@
+(asdf:defsystem checker :serial t
+ :components ((:file "package") (:file "checker"))
+ :depends-on (:cl-ppcre))
diff --git a/src/main/checker.lisp b/src/main/checker.lisp
new file mode 100644 (file)
index 0000000..769ec3f
--- /dev/null
@@ -0,0 +1,95 @@
+(in-package #:style-checker)
+
+; Rules
+; - Elements in each form must be indented the same amount
+; - No form longer than 50 lines
+; - Top level multiline forms must be separated by exactly one space
+; - No line longer than 120 characters
+; - No use of unexported symbols in other packages
+; - No tabs
+; - Only one space between elements in a form on a single line
+; - in-package must be first line in file unless file is package.lisp
+; - No whitespace only lines
+; - No empty lines at end of file
+;
+; Some thoughts
+; - form starting reader macros will have to be hand added to this code
+; - exceptions will eventually arise, and the rule file will have to be changed
+; - the proper formatting of "loop" is weird
+
+(define-condition check-failure nil ((msg :initarg :msg :reader check-failure-msg)
+                                     (line-no :initarg :line-no :reader check-failure-line-no)
+                                     (col-no :initarg :col-no :reader check-failure-col-no)))
+
+(defvar *state* nil)
+(defvar *line-no* nil)
+(defvar *col-no* nil)
+(defvar *evaluators* nil)
+
+(defun set-state (state)
+ (when (not (find state (list :begin ; start of file
+                              :normal
+                        )))
+  (error "Can't set state to ~A" state))
+ (setf *state* state)
+ nil)
+
+(defmacro defevaluator (state match func)
+ (let
+  ((scanner (gensym)))
+ `(let
+   ((,scanner (when (stringp ,match) (cl-ppcre:create-scanner ,match))))
+   (pushnew
+    (list
+     (lambda (state text)
+      (and
+       (eql ,state state)
+       (or
+        (and (symbolp text) (eql text ,match))
+        (and ,scanner
+             (stringp text)
+             (multiple-value-bind (start end) (cl-ppcre:scan ,scanner text)
+              (and start end (= 0 start) (/= 0 end)))))))
+     (lambda (text) (second (multiple-value-list (cl-ppcre:scan ,scanner text))))
+     ,func)
+    *evaluators*))))
+
+(defun evaluate (text)
+; (if (string= "" text)
+     (let
+      ((evaluator (find-if (lambda (f) (funcall f *state* text)) *evaluators* :from-end t :key #'car)))
+      (when (not evaluator) (error (make-condition 'check-failure :msg (format nil "Can't check in state ~S: ~S..." *state* (subseq text 0 (min (length text) 10))) :line-no *line-no* :col-no *col-no*)))
+      (let
+       ((problem (funcall (third evaluator))))
+       (when problem (error (make-condition 'check-failure :msg problem :line-no *line-no* :col-no *col-no*)))
+       (let
+        ((length-of-match (funcall (cadr evaluator) text)))
+        (incf *col-no* length-of-match)
+        (when (< 120 *col-no*) (error (make-condition 'check-failure :msg "Line longer than 120 characters" :line-no *line-no* :col-no *col-no*)))
+        (evaluate (subseq text length-of-match))))));)
+
+(defun slurp-file (filename &key (element-type 'character) (sequence-type 'string))
+ (with-open-file (str filename :element-type element-type)
+  (let ((seq (make-sequence sequence-type (file-length str)))) (read-sequence seq str) seq)))
+
+(defun check-file (file)
+ (set-state :begin)
+ (setf *line-no* 0)
+ (setf *col-no* 0)
+ (handler-case
+  (progn (evaluate (slurp-file file)) t)
+  (check-failure (cf)
+   (format t "In file ~A, Had an error: ~S at ~A:~A~%" (check-failure-msg cf) (check-failure-line-no cf) (check-failure-col-no cf))
+   nil)))
+
+(defun check-directory (dir)
+ (every #'identity (mapcar #'check-file (directory (format nil "~A/**/*.lisp" dir)))))
+
+(progn
+ (setf *evaluators* nil)
+ (defevaluator :begin "\\(in-package[^\\)]*\\)"
+  (lambda ()
+   (set-state :normal) nil))
+ (defevaluator :begin ".*"
+  (lambda ()
+   "Must begin with in-package form")))
diff --git a/src/main/package.lisp b/src/main/package.lisp
new file mode 100644 (file)
index 0000000..bb4da60
--- /dev/null
@@ -0,0 +1,2 @@
+(defpackage #:style-checker (:use :cl))
+