Clojure

Audit IReduce usages for proper Reduced handling

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: Release 1.7
  • Fix Version/s: Release 1.7
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code

Description

Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

Approach: I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

Example: Before the patch:

user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>

Patch: clj-1537-v3.diff
Screened by: Alex Miller

  1. audit-ireduce.diff
    26/Sep/14 11:16 AM
    2 kB
    Timothy Baldridge
  2. clj-1537-gvec-ArraySeq.patch
    14/Nov/14 1:00 PM
    7 kB
    Nicola Mometto
  3. clj-1537-v2.diff
    30/Sep/14 5:59 PM
    2 kB
    Alex Miller
  4. clj-1537-v3.diff
    03/Oct/14 2:23 PM
    2 kB
    Timothy Baldridge

Activity

Timothy Baldridge made changes -
Field Original Value New Value
Description Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Patch:" audit-ireduce.diff
Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Patch:* audit-ireduce.diff
Alex Miller made changes -
Issue Type Enhancement [ 4 ] Defect [ 1 ]
Fix Version/s Release 1.7 [ 10250 ]
Priority Minor [ 4 ] Major [ 3 ]
Approval Vetted [ 10003 ]
Alex Miller made changes -
Attachment clj-1537-v2.diff [ 13374 ]
Alex Miller made changes -
Attachment clj-1537-v2.diff [ 13374 ]
Alex Miller made changes -
Attachment clj-1537-v2.diff [ 13375 ]
Alex Miller made changes -
Description Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Patch:* audit-ireduce.diff
Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Patch:* clj-1537-v2.diff
Alex Miller made changes -
Approval Vetted [ 10003 ] Incomplete [ 10006 ]
Timothy Baldridge made changes -
Description Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Patch:* clj-1537-v2.diff
Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Example:* Before the patch:

{noformat}
user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>
{noformat}

*Patch:* clj-1537-v2.diff
Alex Miller made changes -
Approval Incomplete [ 10006 ] Vetted [ 10003 ]
Timothy Baldridge made changes -
Attachment clj-1537-v3.diff [ 13387 ]
Description Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Example:* Before the patch:

{noformat}
user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>
{noformat}

*Patch:* clj-1537-v2.diff
Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Example:* Before the patch:

{noformat}
user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>
{noformat}

*Patch:* clj-1537-v3.diff
Alex Miller made changes -
Description Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Example:* Before the patch:

{noformat}
user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>
{noformat}

*Patch:* clj-1537-v3.diff
Rich asked that we make sure that all usages of IReduce properly handle Reduced semantics.

*Approach:* I did a "Find Usages" in InteliJ and updated usages of IReduce as needed.

*Example:* Before the patch:

{noformat}
user=> (transduce (take 1) conj (seq (subvec [1 2 3 4 5] 1)))
#<Reduced@13df2a8c: #<Reduced@1ebea008: #<Reduced@72d6b3ba: #<Reduced@1787f2a0: [2]>>>>

user=> (transduce (take 1) conj '(1 2 3 4))
#<Reduced@51bd8b5c: #<Reduced@7b50df34: #<Reduced@1b410b60: #<Reduced@2462cb01: [1]>>>>
{noformat}

*Patch:* clj-1537-v3.diff
*Screened by:* Alex Miller
Approval Vetted [ 10003 ] Screened [ 10004 ]
Rich Hickey made changes -
Approval Screened [ 10004 ] Ok [ 10007 ]
Stuart Halloway made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Completed [ 1 ]
Nicola Mometto made changes -
Resolution Completed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Approval Ok [ 10007 ]
Nicola Mometto made changes -
Attachment clj-1537-gvec-ArraySeq.patch [ 13518 ]
Nicola Mometto made changes -
Status Reopened [ 4 ] Closed [ 6 ]
Resolution Completed [ 1 ]

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: