cfrg / draft-irtf-cfrg-hash-to-curve

Hashing to Elliptic Curves
Other
78 stars 27 forks source link

Address Colin Perkins review #337

Closed chris-wood closed 2 years ago

chris-wood commented 2 years ago

@kwantam suggestions applied -- thanks for the review!

kwantam commented 2 years ago

Just checked, if you search the document for "(see discussion above)" you will find a lot of those indirections. I guess we should sweep for them...

(In fact, most of the code blocks seem to have one! D'oh...)

kwantam commented 2 years ago

Hrm, now I'm getting worried. Are we sure we don't want to just keep "(see discussion above)" or maybe "(see discussion immediately above)" and indirect from there? Seems safer, even if it's slightly awkward.

chris-wood commented 2 years ago

I wouldn't worry too much about this :) The document is sufficient stable at this point that section references won't change without us noticing. I'll do a sweep and fix the others where "immediately above" isn't the same section/paragraph.