Switch to OmniString::explode implementation
ClosedPublic
Actions

Authored by dereckson on Mar 16 2020, 01:35.

Tags

None

Referenced Files

	F47303479: D2283.id5754.diff
	Sun, Aug 30, 11:15

	F47303464: D2283.id5753.diff
	Sun, Aug 30, 11:15

	F47270434: D2283.diff
	Sun, Aug 30, 07:59

	Unknown Object (File)
	Sun, Aug 23, 13:23

	Unknown Object (File)
	Sun, Aug 23, 10:06

	Unknown Object (File)
	Thu, Aug 20, 10:02

	Unknown Object (File)
	Wed, Aug 19, 03:18

	Unknown Object (File)
	Mon, Aug 17, 19:00

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rKOT9e4c82e02a41: Switch to OmniString::explode implementation

Summary

230faccc628f has introduced a correct reference implementation
of explode, without any "returns false" surprise if the delimiter
is "".

This change switches OmniArray::explode implementation to use that.

Test Plan

Unit tests provided

Diff Detail

Repository

rKOT Keruald OmniTools

Lint

Lint Passed

Unit

Tests Passed

Branch

master

Build Status

Buildable 3548
Build 3797: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 16 2020, 01:35

dereckson created this revision.

Harbormaster completed remote builds in B3548: Diff 5753.Mar 16 2020, 01:35

dereckson accepted this revision.Mar 16 2020, 01:36

dereckson edited the summary of this revision. (Show Details)

This revision is now accepted and ready to land.Mar 16 2020, 01:36

Closed by commit rKOT9e4c82e02a41: Switch to OmniString::explode implementation (authored by dereckson). · Explain WhyMar 16 2020, 01:37

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rKOT9e4c82e02a41: Switch to OmniString::explode implementation.

Revision Contents
Changeset List

Path

Size

src/

Collections/

OmniArray.php

6 lines

tests/

Collections/

OmniArrayTest.php

12 lines

Diff 5753

View Options

src/Collections/OmniArray.php