Differential D1950

Map result values instead to use a match block
ClosedPublic
Actions

Authored by dereckson on Oct 21 2018, 21:47.

Tags

None

Referenced Files

	F3794505: D1950.diff
	Wed, Nov 27, 23:34

	F3792735: D1950.id4932.diff
	Wed, Nov 27, 18:06

	Unknown Object (File)
	Mon, Nov 18, 02:54

	Unknown Object (File)
	Mon, Nov 18, 02:21

	Unknown Object (File)
	Wed, Nov 13, 06:52

	Unknown Object (File)
	Wed, Nov 13, 06:52

	Unknown Object (File)
	Wed, Nov 13, 06:27

	Unknown Object (File)
	Wed, Nov 13, 06:06

Subscribers

None

Details

Reviewers

Commits

rLFebe3916f84d9: Map result values instead to use a match block

Summary

To map a result, ie a success T or an error E, into another result,
ie a success U or an error F means the code focus on T -> U, without
regard it's an Ok, and E -> F, without regard it's an Err value.

As such, it's seems more natural to express the idea with
methods manipulating tbe values than to recreate the enum container too.

The Serde library uses those map and map_err methods.

Test Plan

#[get("/player/<name>")]
pub fn get_player(connection: DatabaseConnection, name: String) -> ApiJsonResponse<Player> {
    players
        .filter(username.eq(&name))
        .first::<Player>(&*connection)
        .into_json_response()
}

Diff Detail

Repository

rLF Limiting Factor

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Oct 21 2018, 21:47

dereckson created this revision.

Harbormaster completed remote builds in B3038: Diff 4932.Oct 21 2018, 21:47

dereckson accepted this revision.Oct 21 2018, 21:50

This revision is now accepted and ready to land.Oct 21 2018, 21:50

Closed by commit rLFebe3916f84d9: Map result values instead to use a match block (authored by dereckson). · Explain WhyOct 21 2018, 21:51

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

api/

18 lines

Diff 4933

src/api/replies.rs

Loading...