Allow EdnReader to read foo// (CLJ-873 for EdnReader)

Description

The patch that has been applied in 1.6 for predated the introduction of EdnReader, as such it only patched LispReader.

This patch makes the same change to allow foo// in EdnReader, and removes two constants in clojure.lang.RT that are not needed anymore after this patch.

To reproduce:

Approach: copy changes from LispReader in CLJ-873. After:

Patch: 0001-Fix-CLJ-873-for-EdnReader-too.patch

Screened by: Alex Miller

Environment

None

Activity

Show:
Alex Miller
October 31, 2013, 10:51 PM

Switching back to prior patch.

Alex Miller
October 31, 2013, 10:48 PM

Andy you are uncanny - I was just on my way to update this.

Andy Fingerhut
October 31, 2013, 10:37 PM

With the undoing of the change for earlier today, the patch 0001-Fix-CLJ-873-for-EdnReader-too.patch applies cleanly again, and clj-1238-2.diff no longer does.

Alex Miller
October 25, 2013, 8:05 PM

Updated patch to apply cleanly to master as of Oct 25, 2013.

Andy Fingerhut
October 25, 2013, 6:27 PM

I've not looked into details yet, but screened 0001-Fix-CLJ-873-for-EdnReader-too.patch fails to apply cleanly as of Oct 25, 2013 latest Clojure master.

Completed

Assignee

Unassigned

Reporter

Alex Miller

Labels

Approval

Ok

Patch

Code

Fix versions

Affects versions

Priority

Major