]> gitweb.factorcode.org Git - factor.git/commitdiff
More lint cleanups.
authorJohn Benediktsson <mrjbq7@gmail.com>
Mon, 17 Oct 2011 02:33:16 +0000 (19:33 -0700)
committerJohn Benediktsson <mrjbq7@gmail.com>
Mon, 17 Oct 2011 02:33:16 +0000 (19:33 -0700)
14 files changed:
basis/compiler/codegen/labels/labels.factor
basis/compiler/tree/propagation/simd/simd.factor
basis/furnace/auth/login/login.factor
basis/gobject-introspection/ffi/ffi.factor
basis/lists/lazy/lazy.factor
basis/math/primes/primes.factor
basis/tools/deploy/shaker/shaker.factor
basis/ui/gadgets/editors/editors.factor
basis/ui/gadgets/incremental/incremental.factor
basis/unicode/data/data.factor
basis/xml/syntax/inverse/inverse.factor
core/strings/parser/parser.factor
extra/constructors/constructors.factor
unmaintained/alien/fortran/fortran.factor

index c3eacfd38d83d5bf2a93fb349d01a3275e6b0a8c..f15b30ccb5a79bcde723ef98ce57a291e659387f 100644 (file)
@@ -47,9 +47,7 @@ SYMBOL: binary-literal-table
     [ add-binary-literal ] dip label-fixup ;
 
 : emit-data ( obj label -- )
-    over length align-code
-    resolve-label
-    building get push-all ;
+    over length align-code resolve-label % ;
 
 : emit-binary-literals ( -- )
     binary-literal-table get [ emit-data ] assoc-each ;
index 909ab931da033b22407d64f52f0e87a3960655c3..6b72743374f0b2d10f7af4647fb50bc1473b0d21 100644 (file)
@@ -2,8 +2,8 @@
 ! See http://factorcode.org/license.txt for BSD license.
 USING: accessors assocs byte-arrays combinators compiler.cfg.builder
 continuations fry sequences compiler.tree.propagation.info
-cpu.architecture kernel words make math math.intervals
-math.vectors.simd.intrinsics namespaces ;
+compiler.tree.propagation.nodes cpu.architecture kernel words
+make math math.intervals math.vectors.simd.intrinsics namespaces ;
 IN: compiler.tree.propagation.simd
 
 CONSTANT: vector>vector-intrinsics
@@ -105,7 +105,7 @@ vector>vector-intrinsics [ { byte-array } "default-output-classes" set-word-prop
 ] "outputs" set-word-prop
 
 : clone-with-value-infos ( node -- node' )
-    clone dup in-d>> [ dup value-info ] H{ } map>assoc >>info ;
+    clone dup in-d>> extract-value-info >>info ;
 
 : try-intrinsic ( node intrinsic-quot -- ? )
     '[
index 74ff38141c7ac495b908580ba34353b68cc3b829..2295f61ea27cd692990bba1c6fab91f560db0331 100644 (file)
@@ -1,7 +1,7 @@
 ! Copyright (c) 2008 Slava Pestov\r
 ! See http://factorcode.org/license.txt for BSD license.\r
 USING: kernel accessors namespaces sequences math.parser\r
-calendar validators urls logging html.forms\r
+calendar checksums validators urls logging html.forms\r
 http http.server http.server.dispatchers\r
 furnace.auth\r
 furnace.asides\r
@@ -16,8 +16,7 @@ IN: furnace.auth.login
 SYMBOL: permit-id\r
 \r
 : permit-id-key ( realm -- string )\r
-    [ >hex 2 CHAR: 0 pad-head ] { } map-as concat\r
-    "__p_" prepend ;\r
+    hex-string "__p_" prepend ;\r
 \r
 : client-permit-id ( realm -- id/f )\r
     permit-id-key client-state dup [ string>number ] when ;\r
index 40561da87f706cff00c4126a7fcd7d0cb21c993e..badd24e19fd000f98e151b1326893ba5a5ad9575 100644 (file)
@@ -79,16 +79,16 @@ M: utf8-type parse-const-value drop ;
     [ value>> ] [ type>> ] bi parse-const-value ;
 
 : def-const ( const -- )
-    [ c-identifier>> create-in dup reset-generic ]
-    [ const-value ] bi define-constant ;
+    [ c-identifier>> create-function ] [ const-value ] bi
+    define-constant ;
 
 : def-consts ( consts -- )
     [ def-const ] each ;
 
 : define-enum-member ( member -- )
-    [ c-identifier>> create-in dup reset-generic ]
-    [ value>> ] bi define-constant ;
-           
+    [ c-identifier>> create-function ] [ value>> ] bi
+    define-constant ;
+
 : def-enum-type ( enum -- )
     [ members>> [ define-enum-member ] each ]
     [ c-type>> int def-c-type ] bi ;
index 122a2205dd27664e73877879862cfde7c8daf908..b430df4e81b44505c6890db46fb6674960656543 100644 (file)
@@ -281,7 +281,7 @@ M: lazy-concat nil? ( lazy-concat -- ? )
     dup nil? [
         drop nil
     ] [
-        [ car ] [ cdr ] bi
+        uncons
         [ car lcartesian-product ] [ cdr ] bi
         list>array swap [
             swap [ swap [ suffix ] with lazy-map  ] with lazy-map  lconcat
index a4f0973f295f03dc0f8d752252125e431a8ee24b..6be21371352784383d1ad11f5a4b73872ef4a40e 100644 (file)
@@ -28,7 +28,7 @@ IN: math.primes
     108 max 10000 min <vector> ] keep
     3 < [ [ 2 swap push ] keep ] when ;
 
-: simple? ( n -- ? ) { [ even? ] [ 3 mod 0 = ] [ 5 mod 0 = ] } 1|| ;
+: simple? ( n -- ? ) { [ even? ] [ 3 divisor? ] [ 5 divisor? ] } 1|| ;
 
 PRIVATE>
 
index 7fa68b41ff787a3a6e813ab0900c6a4b46d8a75c..0f838926e0c56211acd42d1d797dfb50c50cf0dc 100755 (executable)
@@ -521,7 +521,7 @@ SYMBOL: deploy-vocab
 : write-vocab-manifest ( vocab-manifest-out -- )
     "Writing vocabulary manifest to " write dup print flush
     vocabs "VOCABS:" prefix
-    deploy-libraries get [ libraries get at path>> ] map members "LIBRARIES:" prefix append
+    deploy-libraries get [ library path>> ] map members "LIBRARIES:" prefix append
     swap utf8 set-file-lines ;
 
 : prepare-deploy-libraries ( -- )
index aef79ec92baca23ca7ca34e3d91307395b22f4b8..ac636855ab884e34769564ad5500dd8fbbf1233b 100644 (file)
@@ -543,20 +543,14 @@ PRIVATE>
     [ last-line# ] dip = ;
 
 : prev-line-and-this ( document line -- start end )
-    swap
-    [ drop 1 - 0 2array ]
-    [ [ drop ] [ doc-line length ] 2bi 2array ]
-    2bi ;
+    swap [ drop 1 - 0 2array ] [ line-end ] 2bi ;
 
 : join-with-prev ( document line -- )
     [ prev-line-and-this ] [ drop ] 2bi
     [ join-lines ] change-doc-range ;
 
 : this-line-and-next ( document line -- start end )
-    swap
-    [ drop 0 2array ]
-    [ [ 1 + ] dip [ drop ] [ doc-line length ] 2bi 2array ]
-    2bi ;
+    swap [ drop 0 2array ] [ [ 1 + ] dip line-end ] 2bi ;
 
 : join-with-next ( document line -- )
     [ this-line-and-next ] [ drop ] 2bi
index 767b1a57b1e26fafe6aaaaa94140b7328a4364c1..f46a4371d4ce6f2c98b1e4f2d03760e2b55971a8 100644 (file)
@@ -31,7 +31,7 @@ M: incremental pref-dim*
     >>loc drop ;
 
 : prefer-incremental ( gadget -- )
-    dup forget-pref-dim dup pref-dim >>dim drop ;
+    dup forget-pref-dim prefer ;
 
 M: incremental dim-changed drop ;
 
index cb1b0ce032e33140016f43010f38b038418fecd9..22828286810527d0c5a855237b53793f547e09f4 100644 (file)
@@ -116,7 +116,7 @@ PRIVATE>
     [ 0 = not ] filter ;
 
 : remove-exclusions ( alist -- alist )
-    exclusions [ dup ] H{ } map>assoc assoc-diff ;
+    exclusions unique assoc-diff ;
 
 : process-canonical ( data -- hash hash )
     (process-decomposed) [ first* ] filter
index 0d757b378cfdc5b2d574a56cb43fc4b9e3f5db2d..a4fcb8c70173fc2f8785267d1eb3d72638630737 100644 (file)
@@ -49,8 +49,7 @@ M: tag [undo-xml] ( tag -- quot: ( tag -- ) )
     } cleave '[ _ _ _ tri ] ;
 
 : firstn-strong ( seq n -- ... )
-    [ swap length =/fail ]
-    [ firstn ] 2bi ; inline
+    [ assure-length ] [ firstn ] 2bi ; inline
 
 M: sequence [undo-xml] ( sequence -- quot: ( seq -- ) )
     remove-blanks [ length ] [ [ [undo-xml] ] { } map-as ] bi
index d3dc72005abbfe32eb76e801ae1391c79f7fd5ce..16fab82db8afaab21d52fc4d246a6255dd4b4519 100644 (file)
@@ -103,9 +103,7 @@ ERROR: escaped-char-expected ;
     ] if ;
 
 : lexer-head? ( string -- ? )
-    [
-        lexer get [ line-text>> ] [ column>> ] bi tail-slice
-    ] dip head? ;
+    [ lexer get rest-of-line ] dip head? ;
 
 : advance-lexer ( n -- )
     [ lexer get ] dip [ + ] curry change-column drop ; inline
index 6701e613249d2fec2f15b7eeb130cb4442c5cf7e..cd5e3566d49299c21e525bd87761742759c3b2da 100644 (file)
@@ -1,6 +1,6 @@
 ! Copyright (C) 2009 Slava Pestov, Doug Coleman.
 ! See http://factorcode.org/license.txt for BSD license.
-USING: accessors arrays assocs classes classes.tuple
+USING: accessors alien.parser arrays assocs classes classes.tuple
 effects.parser fry generalizations sequences.generalizations
 generic.standard kernel lexer locals macros parser sequences
 sets slots vocabs words ;
@@ -60,7 +60,7 @@ ERROR: unknown-constructor-parameters class effect unknown ;
     '[ @ _ [ execute( obj -- obj ) ] each ] effect define-declared ;
 
 : scan-constructor ( -- word class )
-    scan-word [ name>> "<" ">" surround create-in dup reset-generic ] keep ;
+    scan-word [ name>> "<" ">" surround create-function ] keep ;
 
 : parse-constructor ( -- class word effect def )
     scan-constructor complete-effect ensure-constructor-parameters
index f17e91b90ce4e907aab131668acb05778647fc1d..5d2bfe0142a7e98aa2ded207b4088ff09100eb3f 100755 (executable)
@@ -435,7 +435,7 @@ MACRO: fortran-invoke ( return library function parameters -- )
     bi* <effect> ;
 
 :: define-fortran-function ( return library function parameters -- )
-    function create-in dup reset-generic 
+    function create-function
     return library function parameters return [ c:void ] unless* parse-arglist
     [ \ fortran-invoke 5 [ ] nsequence ] dip define-declared ;