giraffe-fsharp / Giraffe

A native functional ASP.NET Core web framework for F# developers.

Home Page:https://giraffe.wiki

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Follow-up from "Minor code optimisation #567"

64J0 opened this issue · comments

Description:

@Thorium suggested:

Another modification to consider, would be adding [<Struct>] attribute to Giraffe's minor enum-types like HttpVerb, MatchMode, Precondition, CacheDirective and RangeBoundary. But if there are proper performance tests somewhere, could be first tested if they'd have any effect or not. But I guess that goes around questions like should some Option values changed to ValueOption values.

Reference to the PR: #567