Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1525)

Unified Diff: third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h

Issue 2385283002: reflow comments in platform/{testing,text} (Closed)
Patch Set: idunnolol Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h
diff --git a/third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h b/third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h
index ea4f8ff21a7584d2c739eaee3c9afa2d19fa5081..8d5277b6af95a3f5fe945c14c15b88c519d3861a 100644
--- a/third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h
+++ b/third_party/WebKit/Source/platform/text/hyphenation/HyphenatorAOSP.h
@@ -36,41 +36,45 @@ class Hyphenator {
// Note: this will also require a locale, for proper case folding behavior
static Hyphenator* load(const uint16_t* patternData, size_t size);
- // Compute the hyphenation of a word, storing the hyphenation in result vector. Each
- // entry in the vector is a "hyphen edit" to be applied at the corresponding code unit
- // offset in the word. Currently 0 means no hyphen and 1 means insert hyphen and break,
- // but this will be expanded to other edits for nonstandard hyphenation.
- // Example: word is "hyphen", result is [0 0 1 0 0 0], corresponding to "hy-phen".
+ // Compute the hyphenation of a word, storing the hyphenation in result
+ // vector. Each entry in the vector is a "hyphen edit" to be applied at the
+ // corresponding code unit offset in the word. Currently 0 means no hyphen and
+ // 1 means insert hyphen and break, but this will be expanded to other edits
+ // for nonstandard hyphenation. Example: word is "hyphen", result is [0 0 1 0
+ // 0 0], corresponding to "hy-phen".
void hyphenate(std::vector<uint8_t>* result,
const uint16_t* word,
size_t len);
- // pattern data is in binary format, as described in doc/hyb_file_format.md. Note:
- // the caller is responsible for ensuring that the lifetime of the pattern data is
- // at least as long as the Hyphenator object.
+ // pattern data is in binary format, as described in doc/hyb_file_format.md.
+ // Note: the caller is responsible for ensuring that the lifetime of the
+ // pattern data is at least as long as the Hyphenator object.
- // Note: nullptr is valid input, in which case the hyphenator only processes soft hyphens
+ // Note: nullptr is valid input, in which case the hyphenator only processes
+ // soft hyphens
static Hyphenator* loadBinary(const uint8_t* patternData);
private:
// apply soft hyphens only, ignoring patterns
void hyphenateSoft(uint8_t* result, const uint16_t* word, size_t len);
- // try looking up word in alphabet table, return false if any code units fail to map
- // Note that this methor writes len+2 entries into alpha_codes (including start and stop)
+ // Try looking up word in alphabet table, return false if any code units fail
+ // to map. Note that this methor writes len+2 entries into alpha_codes
+ // (including start and stop).
bool alphabetLookup(uint16_t* alpha_codes, const uint16_t* word, size_t len);
- // calculate hyphenation from patterns, assuming alphabet lookup has already been done
+ // calculate hyphenation from patterns, assuming alphabet lookup has already
+ // been done
void hyphenateFromCodes(uint8_t* result, const uint16_t* codes, size_t len);
- // TODO: these should become parameters, as they might vary by locale, screen size, and
- // possibly explicit user control.
+ // TODO: these should become parameters, as they might vary by locale, screen
+ // size, and possibly explicit user control.
static const int MIN_PREFIX = 2;
static const int MIN_SUFFIX = 3;
- // See also LONGEST_HYPHENATED_WORD in LineBreaker.cpp. Here the constant is used so
- // that temporary buffers can be stack-allocated without waste, which is a slightly
- // different use case. It measures UTF-16 code units.
+ // See also LONGEST_HYPHENATED_WORD in LineBreaker.cpp. Here the constant is
+ // used so that temporary buffers can be stack-allocated without waste, which
+ // is a slightly different use case. It measures UTF-16 code units.
static const size_t MAX_HYPHENATED_SIZE = 64;
const uint8_t* patternData;

Powered by Google App Engine
This is Rietveld 408576698