search

kazuho / draft-kazuho-httpbis-priority

Other
6 stars 4 forks source link

Update reprioritization example #74

Closed rmarx closed 4 years ago

rmarx commented 4 years ago

Fixes #53

rmarx commented 4 years ago

I'm not sure what you mean... I can't seem to see the "stylistic suggestion" using the github interface for some reason. However, I'm sure it's fine and you can just merge afaic.

LPardue commented 4 years ago

@rmarx Lol Github ate my suggestion. To sumamrise, the paragraph mixes priority level names with values e.g. it uses background and then a reprioritiziation to urgency=0. I think it would help to use both in this example. I.e. after background add "(urgency=6)", and then similarly have "default (urgency=1)

rmarx commented 4 years ago

Aha, excellent suggestion! Changed in f6604a1

kazuho commented 4 years ago

Thank you all!!!