Adding whitespace? to clojure.string #2

Closed
wants to merge 1,214 commits into
from

Projects

None yet
@tebeka
tebeka commented Nov 17, 2010

I had the need for whitespace? a few times now, thought it'll make a good addition to clojure.string

stuartsierra and others added some commits Jan 20, 2010
@stuartsierra stuartsierra Exclude clojure.contrib.pprint from compilation
Other c.c.pprint.* namespaces get compiled, but can't load
c.c.pprint itself until dependencies are AOT-compiled.
952c55e
@stuartsierra stuartsierra Add release and SNAPSHOT distributionManagement to POM 66e35b0
@stuartsierra stuartsierra Add "dist" assembly for distribution ZIP/tar files. cecccb4
@stuartsierra stuartsierra Add assembly configuration to POM. e3bdcf3
@stuartsierra stuartsierra Reformat README to 70-chars-per-line f3826fa
@stuartsierra stuartsierra Clean up URLs and add link to package downloads in README 889f591
@stuartsierra stuartsierra Add additional mvn build commands to README 1bffd84
@stuartsierra stuartsierra Clean up headings in README 2984bb0
@stuartsierra stuartsierra Add source .clj files to JAR 78df570
@stuartsierra stuartsierra Correct README typo 5bed152
@stuartsierra stuartsierra Describe version numbering corresponding to Clojure versions 9baad3c
@stuartsierra stuartsierra Make clojure.version a property in POM 3f433eb
@stuartsierra stuartsierra Describe setting clojure.version in README c45e453
@stuartsierra stuartsierra Add instructions for building with local Clojure 18584a4
@stuartsierra stuartsierra Add str-utils3 with tests
 * most significant argument last, for ->>
 * 'contains?' renamed 'substring?'
 * removed 'partial'
18c6d21
@stuartsierra stuartsierra str-utils3: substitute type-specific fns for 'replace' multimethod
'replace' doesn't need runtime dispatch, so multimethods are
unnecessary performance overhead.

New functions: replace-char, replace-str, replace-re, and replace-by
ed4faa4
@stuartsierra stuartsierra Add offline build instructions to README df49997
@stuartsierra stuartsierra Add "local" profile to POM 86803a4
@stuartsierra stuartsierra Describe "local" build in POM db9bedf
@stuartsierra stuartsierra Describe where to get development snapshots in README d52942f
@stuartsierra stuartsierra Add source encoding UTF-8 to POM 436fef0
@stuartsierra stuartsierra Simplify POM, only AOT-compile namespaces that gen-class
Binary incompatibility of AOT-compiled namespaces makes
it desirable to avoid compilation except when needed
for gen-class.

Also fixes problem of the compilation order being
non-deterministic, depending on the underlying filesystem.
66870bf
@stuartsierra stuartsierra Reenable builds with local clojure.jar
Corrected POM still has Clojure as a dependency,
with an optional profile that allows specifying
the location of clojure.jar
4e8696e
@stuartsierra stuartsierra pprint: require ColumnWriter in PrettyWriter
This is an attempt to fix compilation-order
issues with gen-class.

See http://groups.google.com/group/clojure-maven-plugin/browse_thread/thread/40150bb45270b322
0294e79
@stuartsierra stuartsierra Revert "Reenable builds with local clojure.jar"
This reverts commit 4e8696e.

Attempted fix doesn't work.
75eaf57
@stuartsierra stuartsierra Do not AOT-compile pprint
Need to fix order-of-compilation issues
with PrettyWriter and ColumnWriter
a0f80d1
@stuartsierra stuartsierra pprint: Move gen-class expressions to separate file; refs #65
To avoid order-of-compilation issues,
this patch puts the gen-class expressions
for ColumnWriter and PrettyWriter in a
separate file, rather than in the ns
declarations.
3275810
@stuartsierra stuartsierra Reenable compiling of test namespaces in POM; refs #65
But disable tests of pprint, which fail currently
aa34fb9
@stuartsierra stuartsierra Fix mis-named ns in pprint.gen-class; refs #66 5b3beae
Alexander Taggart fix for aotc, ticket 44; moved impl selection code from macro to runt…
…ime eval

Signed-off-by: Stuart Sierra <mail@stuartsierra.com>
655060b
@stuartsierra stuartsierra c.c.json.read: accept any Reader type; fixes #67 8d5f9fa
@stuartsierra stuartsierra c.c.json.read: use transients f72d665
@stuartsierra stuartsierra Add c.c.json; replacement for c.c.json.read & c.c.json.write
 * New library uses protocols.
 * read-json accepts any String or Reader.
 * read-json keywordizes keys by default.
8b512d8
@stuartsierra stuartsierra new c.c.json: fix 1 reflective call 1c75488
@stuartsierra stuartsierra c.c.json: reorganize around extend
Simpler & less repetitive than extend-protocol
in this case.
23ef042
@stuartsierra stuartsierra c.c.json: use PrintWriter directly; even faster 687df96
@stuartsierra stuartsierra Set clojure-maven-plugin version in POM
Older versions may fail to run the build correctly
7e1746e
@stuartsierra stuartsierra Rename popular libs with shorter names. 4e7a55d
@stuartsierra stuartsierra Delete obsolete versions of popular libs. 6b68e48
@stuartsierra stuartsierra Change ns names for all renamed libs except str-utils ec6a757
@stuartsierra stuartsierra Rename uses of str-utils/2/3 to string 3a14dce
@stuartsierra stuartsierra Rename test namespaces to match new names. e9269d2
@stuartsierra stuartsierra Remove old str-utils lib tests. 1ab0a22
@stuartsierra stuartsierra Remove redundant test-contrib ns name part. 075f770
@stuartsierra stuartsierra Rename test sources to clojure/contrib/test_*.clj naming pattern. 69ccca4
@stuartsierra stuartsierra Rename namespace declarations in test sources. 4e62c17
@stuartsierra stuartsierra Add test namespaces to be compiled in POM
Current clojure-maven-plugin does not run tests
unless they are AOT-compiled.
2c07978
@stuartsierra stuartsierra Fix remaining ns name issues in tests. ac67da7
@stuartsierra stuartsierra Re-enable local build with -Dclojure.jar in POM 9b194df
@pjt pjt c.c.string migration changes; fixes #70
Signed-off-by: Stuart Sierra <mail@stuartsierra.com>
af788ed
@stuartsierra stuartsierra README: remove -o from maven command line for local build 9aecba5
@Chouser Chouser lazy-xml: Experimental emit based on xml Transformer 1fe4b40
@stuartsierra stuartsierra fix bogus Replacement class name in c.c.string e65fe64
@stuartsierra stuartsierra json: factor out remaining printer functions as defns 724e767
@stuartsierra stuartsierra c.c.json: add pretty printing 319057d
@stuartsierra stuartsierra Add test for c.c.io/file-str 2cae517
@stuartsierra stuartsierra c.c.io: fix backslash handling in file-str c8be496
@stuartsierra stuartsierra c.c.prxml: Fix bad refers from lazy-xml 7a0e744
@stuartsierra stuartsierra Add c.c.reflect, taking 2 functions from c.c.java 91582e1
@stuartsierra stuartsierra Refactor c.c.java into c.c.io, c.c.string, and c.c.properties bacf492
@stuartsierra stuartsierra prxml: remove references to c.c.java 4cca961
@stuartsierra stuartsierra c.c.json: change references from c.c.java to c.c.string 55cc53e
@stuartsierra stuartsierra c.c.http.connection: replace references to c.c.java b1ef935
@Chouser Chouser Stop using c.c.java in pprint examples. 5d30e63
@stuartsierra stuartsierra c.c.jmx: change from c.c.java to c.c.string 5ec001a
@stuartsierra stuartsierra c.c.sql: change references to c.c.java 6a0483d
@stuartsierra stuartsierra Remove c.c.java reference from miglayout.internal b79b246
@stuartsierra stuartsierra Remove test_java.clj 65363bd
@stuartsierra stuartsierra Move tests from old test_java.clj to test_io.clj 3093440
@stuartsierra stuartsierra Move tests from test_java.clj to test_string.clj faa392b
@stuartsierra stuartsierra Add test_properties.clj from old test_java.clj 3174885
@Chouser Chouser trace: Allow dotrace to work on private var fns. 7362d7b
@Chouser Chouser trace: fix previous commit -- no longer generate infinite recursion. 0a1bfc9
@stuarthalloway stuarthalloway :environment option for jmx/with-connection 6f2b1d7
@stuarthalloway stuarthalloway allow custom :jndi-path for JMX URLs 6cfd3b2
@Chouser Chouser Remove CPL.TXT -- contrib is currently licensed under EPL 1.0, see ep…
…l-v10.html
8e3fca7
@khinsen khinsen generic.comparison: fixed typo in <=, added pos? neg? min max f7a454f
The Stuarts fix PushbackReader bug 1e33333
@stuartsierra stuartsierra Update POM to clojure-maven-plugin 1.3.2
No longer need list of test namespaces to AOT-compile
1923b54
@stuartsierra stuartsierra Merge branch 'newplugin' d1e831b
@stuartsierra stuartsierra Disable test-jmx temporarily
Tests fail on build.clojure.org, see
http://build.clojure.org/job/clojure-contrib/46/console
2507183
@stuarthalloway stuarthalloway troubleshooting build box: another ad hoc exception for read-supported ffcfbf9
@stuarthalloway stuarthalloway once more, with spelling a0178ae
@stuarthalloway stuarthalloway simplify jmx/read-supported and improve test output d157d64
@Chouser Chouser update lazy-xml for reify with explicit 'this' on every method bbe248f
@stuarthalloway stuarthalloway added apropos (per Michel Salim, plus re support) 4f9a78d
@stuarthalloway stuarthalloway windows-safe test for relative-path-string ebf0d21
@stuarthalloway stuarthalloway fixed corner case in c.c.profile print-summary 6dd033d
@stuarthalloway stuarthalloway -?>> per Chas Emerick 9f44842
@stuarthalloway stuarthalloway strint per Chas Emerick b9db280
@stuarthalloway stuarthalloway test-load-all tries to load all nondeprecated namespaces
 - fixed bug: misspelling in pom
 - updated gen-html-docs to track c.c.string name changes
b52f0b6
@bpsm bpsm c.c.io provides input-stream and output-stream for byte-oriented I/O
clojure.contrib.io already supports character-oriented I/O through the
multi-methods reader and writer.  This patch adds support for
byte-oriented I/O by providing the multi-methods input-stream and
output-stream.

* input-stream knows how to open InputStreams for reading bytes.
* reader has been refactored to build on input-stream.

* output-stream knows how to open OutputStreams for writing bytes.
* writer has been refactored to build on output-stream (where sensible)

  By recognizing that output-stream will throw exceptions for us if it's
  unable to open the underlying resource, we were able to use the
  :default method to cover URL, URI and Socket.

  The String writer has not been touched. (Writing it in terms of
  output-stream would have made it longer and more complex.)

* *append-to-writer* has been renamed to *append* for use with
  output-streams without souding foolish.

  This is a breaking change for clients that bind *append-to-writer*
  instead of calling append-writer or append-split as recommended in
  the docsting.

* Like writer, output-stream consults the current binding of *append*.

* append-output-stream is analagous to append-writer.

  Like append-writer, the implementation looks very general, but in
  reality it only works for files.

Signed-off-by: Stuart Halloway <stu@thinkrelevance.com>
9cd7b15
@stuarthalloway stuarthalloway some tests for c.c.io byte-level support af2a730
@stuarthalloway stuarthalloway #78 relaxing test expectations to eliminate spurious errors on IBM JDK aa4142c
@stuartsierra stuartsierra Copy deleted/renamed namespaces from 1.1 release; refs #79
* For backward compatibility with 1.1 release.
* Namespaces copied: duck-streams, java-utils, seq-utils,
  shell-out, str-utils, str-utils2.
* These namespaces can be marked as deprecated.
* They are copied rather than simply aliased because
  some behavior has changed.
a1c66df
@stuarthalloway stuarthalloway update contrib to remove seq fns promoted to clojure.core 78ee9b3
@tomfaulhaber tomfaulhaber Remove gen-class requirement from clojure.contrib.pprint. See #81 661dcfd
@tomfaulhaber tomfaulhaber clojure.contrib.pprint (cl-format): Fixes to rounding and width issue…
…s in ~f and ~$. See #47.
912e567
@tomfaulhaber tomfaulhaber Remove debug prints :-(. See #47 5a703b3
@richhickey richhickey removed seq-contains? calls eb7021a
khinsen and others added some commits Sep 3, 2010
@khinsen khinsen macro-utils: don't expand symbols bound in the surrounding lexical en…
…vironment
67fccc6
@khinsen khinsen Merge branch 'master' of git://github.com/clojure/clojure-contrib bbb200f
Alexander Taggart commenting out tests that fail only on build machine 328f104
Alexander Taggart Fix arglist doc for level-specific macros 9212f64
@Chouser Chouser Fix broken repl-utils/show
Was broken since fe4ed31
Fixes #94
Also add a bit more type info in class component maps.
4083a06
@stuartsierra stuartsierra Fix broken with-ns tests; refs #95
* broken by Clojure commit 1c0468d
* Clojure's compiler exceptions changed their format
11852b9
@stuartsierra stuartsierra Fix EOF-handling in JSON reader; refs #91 5a928e2
@stuartsierra stuartsierra Base64 decoding from Teemu Antti-Poika; refs #84 5263e4b
@stuartsierra stuartsierra Add latest Maven build plugin to top-level POM 97870ec
@stuartsierra stuartsierra Set clojure.version to 1.2.0 for release 3d88184
@stuartsierra stuartsierra Remove extra xmlns declarations from POMs (breaks release plugin) b808b45
@stuartsierra stuartsierra [maven-release-plugin] prepare release 1.3.0 d6e13bc
@stuartsierra stuartsierra [maven-release-plugin] prepare for next development iteration d5529fd
@stuartsierra stuartsierra Revert "[maven-release-plugin] prepare for next development iteration"
This reverts commit d5529fd.
2d2b158
@stuartsierra stuartsierra Revert "[maven-release-plugin] prepare release 1.3.0"
This reverts commit d6e13bc.
6dcc653
@stuartsierra stuartsierra Set clojure.version to 1.3.0-master-SNAPSHOT 1eccdee
@stuartsierra stuartsierra :require clojure.set in accumulators as it's not auto-loaded anymore a9eeece
@stuartsierra stuartsierra Create clojure.contrib.version property in parent d328525
@stuartsierra stuartsierra Use clojure.contrib.version property in all modules 147c66c
@stuartsierra stuartsierra Use clojure.version property in AOT-compiled modules eb67ac8
@stuartsierra stuartsierra Add distributionManagement sections to top-level and parent POMs ce7965d
@stuartsierra stuartsierra Set release versions to 1.3.0-alpha1 89833b4
@stuartsierra stuartsierra Back to SNAPSHOT versions 6db7902
@stuartsierra stuartsierra Add distribution ZIP/TAR file assembly c826d67
@stuartsierra stuartsierra Fix dist assembly to include modules 04cd891
@stuartsierra stuartsierra Update README with instructions for building against custom Clojure 5bae316
@stuartsierra stuartsierra cartesian-product nil-arg fix; refs #96 499b9c0
@stuartsierra stuartsierra Set all version numbers to 1.3.0-alpha1 explicitly 5bc8b35
@stuartsierra stuartsierra Remove assembly from "complete" POM be8e41e
@stuartsierra stuartsierra Merge branch 'fixrelease'
Conflicts:
	modules/parent/pom.xml [resolved]

* Must hard-code contrib version number in every POM
0d69c12
@stuartsierra stuartsierra Back to version 1.3.0-SNAPSHOT on all modules 39c3822
@Chouser Chouser Restore examples lost during modules split, a6a92b9 2e7ef0b
@abrenk abrenk Fix build of complete/standalone artifact.
Dependency on single, large JAR:
 groupId: org.clojure.contrib
 artifactId: clojure-contrib
 version: 1.3.0-SNAPSHOT
 classifier: standalone

Dependency on multiple, small JARs:
 groupId: org.clojure.contrib
 artifactId: complete
 version: 1.3.0-SNAPSHOT
d6b88cb
@abrenk abrenk Add separate module for standalone uberjar.
Dependency on single, large JAR:
 groupId: org.clojure.contrib
 artifactId: standalone
 version: 1.3.0-SNAPSHOT

Dependency on multiple, small JARs:
 groupId: org.clojure.contrib
 artifactId: complete
 version: 1.3.0-SNAPSHOT
40d7ba0
@stuartsierra stuartsierra Merge branch 'abrenk-build' 0561a9b
@stuartsierra stuartsierra Fix failing Boolean test in c.c.jmx 6146f54
@Chouser Chouser Fix out of date github URLs in docs 8bb336e
@bpsm bpsm t100: revert "removed deprecated clojure.contrib.java-utils"
This reverts commit cb832f6.

Signed-off-by: Stuart Sierra <mail@stuartsierra.com>
c115db1
@bpsm bpsm t100: removed deprecated functions form c.c.java-utils
removed: relative-path-string, as-file, file, as-str, delete-file,
         delete-file-recursively, as-url, wall-hack-method,
         wall-hack-field

use instead: clojure.java.io and clojure.contrib.reflections

Signed-off-by: Stuart Sierra <mail@stuartsierra.com>
456fbaa
@bpsm bpsm t100: added c.c.java-utils to c.c.standalone
Signed-off-by: Stuart Sierra <mail@stuartsierra.com>
3eb423c
@stuartsierra stuartsierra Merge branch 'master' of github.com:clojure/clojure-contrib f6ef99e
@abrenk abrenk Simplify building against a custom Clojure JAR.
Signed-off-by: Andreas Brenk <mail@andreasbrenk.com>
21a8ba7
@abrenk abrenk Use ${project.version} to reference artifacts in standalone module 5ac0b49
@abrenk abrenk Fix URL of XMLSchema-instance namespace in pom.xml d46ae8f
@khinsen khinsen macro-utils: declared rebound vars as :dynamic 3124805
@stuartsierra stuartsierra Rename Var.getRoot to Var.getRawRoot 6b0ced9
@stuartsierra stuartsierra Move clojure.version to 1.3.0-alpha2 in prep for alpha2 contrib release 23d2e36
@khinsen khinsen generic: remove superfluous use clause 57aae4f
@stuartsierra stuartsierra Move complete & standalone modules to end of top-level build; maybe a…
…void dependency errors on release
d183b52
@stuartsierra stuartsierra Merge branch 'master' of github.com:clojure/clojure-contrib 0dfe73b
@stuartsierra stuartsierra Revert "Move complete & standalone modules to end of top-level build"
This reverts commit d183b52.

It didn't work, release still fails, probably needs separate
install step first.
9cf1984
@stuartsierra stuartsierra Update maven-release-plugin to version 2.1 d133788
@stuartsierra stuartsierra Update all versions to 1.3.0-alpha2 release cb86895
@stuartsierra stuartsierra Revert all versions to 1.3.0-SNAPSHOT for development 163c7d5
@stuartsierra stuartsierra Set all versions to 1.3.0-alpha3 6711b9e
@stuartsierra stuartsierra Reset versions to 1.3.0-SNAPSHOT c38fa23
@tebeka tebeka Adding whitespace? to clojure.string d16ac41
@tomfaulhaber tomfaulhaber Replace occurences of "as-str" with "name" since that is the new way. 53df7dc
@tomfaulhaber tomfaulhaber Refer to clojure.java.io/file since we use it 60e7162
@stuartsierra stuartsierra Adapt POMs to use new contrib parent POM a579d18
@stuartsierra stuartsierra Handle quotes in JSON keys correctly; refs #98 d6f6ccf
@stuartsierra stuartsierra parent POM to depend on pom.contrib 0.0.19 b41419b
@stuartsierra stuartsierra Add/fix SCM info in POMs 47e7898
@stuartsierra stuartsierra Remove <repository> from top-level POM c2eb801
@stuartsierra stuartsierra Add pom.contrib parent to top-level POM d3f22c5
@tomfaulhaber tomfaulhaber Moved the documentation into the namespace doc string so that autodoc…
… will pick it up.
37fba7e
@stuartsierra stuartsierra Add option to NOT escape Unicode; refs #100 CONTRIB-100 5380552
@stuartsierra stuartsierra Document JSON option; refs #100 CONTRIB-100 82c7d49
@stuartsierra stuartsierra Fix unicode characters in JSON docstrings 01036ea
@tebeka tebeka Upstream merge 6bfd926
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
672b9e9
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
86be654
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
2c6ad24
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
e2a7d03
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
ce63e7b
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Apr 27, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
cfbeb71
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request May 23, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
6f17882
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request May 24, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
3c686c7
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Jun 9, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
213e41a
@Bronsa Bronsa added a commit to Bronsa/clojure that referenced this pull request Jun 14, 2013
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
8f0f62d
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jun 23, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
927c6d4
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jun 29, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
2f9740e
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jun 30, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
af02a28
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 5, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
e4cd9b6
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 6, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
6a70f88
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 7, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
7ead5d1
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 8, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
d8f1494
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 10, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
93f96fd
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 15, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
560aa33
@MichaelBlume MichaelBlume added a commit to MichaelBlume/clojure that referenced this pull request Jul 31, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.
842ef23
@stuarthalloway stuarthalloway added a commit that referenced this pull request Jul 31, 2015
@hyPiRion hyPiRion Fix "~n@*"-directives in clojure.pprint/cl-format. Fixes #CLJ-1134.
This solves two issues as specified by #CLJ-1134. Issue #1 is solved by doing a
relative jump forward within `absolute-reposition` in cl_format.clj, line 114 by
switching `(- (:pos navigator) position)` with `(- position (:pos navigator))`.

Issue #2 is handled by changing the default `n`-parameter to `*` depending on
whether the `@`-prefix is placed or not. If it is placed, then `n` defaults to
0, otherwise it defaults to 1.

In addition, new tests have been appended to `test_cl_format.clj` to ensure the
correctness of this patch. The tests have been tested on the Common Lisp
implementation GNU CLISP 2.49, which presumably handle the `~n@*`
correctly. This patch and GNU CLISP returns the same output for each format
call, sans case for printed symbols; Common Lisp has case-insensitive symbols,
whereas Clojure has not.

Signed-off-by: Stuart Halloway <stu@cognitect.com>
8a8e1f9
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment