ClosedPublic
Actions

Authored by dereckson on May 30 2020, 23:08.

Tags

None

Referenced Files

	F45395764: D2290.id5768.diff
	Sat, Aug 22, 01:17

	F45394969: D2290.id5768.diff
	Sat, Aug 22, 01:12

	F45382798: D2290.id5771.diff
	Fri, Aug 21, 23:47

	F45382708: D2290.id5770.diff
	Fri, Aug 21, 23:46

	F45374050: D2290.id5771.diff
	Fri, Aug 21, 22:57

	F45351377: D2290.diff
	Fri, Aug 21, 20:46

	F45350666: D2290.id5770.diff
	Fri, Aug 21, 20:40

	F45320548: D2290.id5771.diff
	Fri, Aug 21, 17:26

View All Files

Subscribers

None

Details

Reviewers

amj

Commits

rLFfe6f26f02319: Avoid Box<Vec<_>>

Summary

As Vec is already a smart pointer, there isn't any need or benefit
to encapsulate it into another smart pointer.

Test Plan

Run a microservice, the routes are mounted as before.

Diff Detail

Repository

rLF Limiting Factor

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-box-vec

Build Status

Buildable 3559
Build 3808: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.May 30 2020, 23:08

dereckson created this revision.

Harbormaster completed remote builds in B3557: Diff 5768.May 30 2020, 23:08

for DefaultService too

Harbormaster completed remote builds in B3559: Diff 5770.May 31 2020, 00:00

amj accepted this revision.May 31 2020, 00:26

This revision is now accepted and ready to land.May 31 2020, 00:26

Closed by commit rLFfe6f26f02319: Avoid Box<Vec<_>> (authored by dereckson). · Explain WhyJun 1 2020, 17:31

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rLFfe6f26f02319: Avoid Box<Vec<_>>.

Revision Contents
Changeset List

Path

Size

src/

config.rs

4 lines

kernel.rs

4 lines

Diff 5770

View Options

src/config.rs