greg/mentat
1
0
Fork 0
Code Issues 294 Pull requests Projects Releases 9 Packages Wiki Activity

Review comment.

Browse source
This commit is contained in:
Richard Newman 2018-05-28 18:50:07 -07:00
parent 23a616db22
commit a0e99b131d
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
edn/src/namespaceable_name.rs
View file

@ -36,17 +36,16 @@ pub struct NamespaceableName {
// the two parts. // the two parts.
components: String, components: String,
// The index (in bytes) into `components` of the dividing solidus — the place // The index (in bytes) into `components` of the dividing solidus — the character
// where the namespace ends and the name begins. // between the namespace and the name.
// //
// If this is zero, it means that this is _not_ a namespaced value! // If this is zero, it means that this is _not_ a namespaced value!
// //
// Important: The following invariants around `boundary` must be maintained // Important: The following invariants around `boundary` must be maintained:
// for memory safety.
// //
// 1. `boundary` must always be less than or equal to `components.len()`. // 1. `boundary` must always be less than or equal to `components.len()`.
// 2. `boundary` must be byte index that points to a character boundary, // 2. `boundary` must be a byte index that points to a character boundary,
// and not point into the middle of a utf8 codepoint. That is, // and not point into the middle of a UTF-8 codepoint. That is,
// `components.is_char_boundary(boundary)` must always be true. // `components.is_char_boundary(boundary)` must always be true.
// //
// These invariants are enforced by `NamespaceableName::namespaced()`, and since // These invariants are enforced by `NamespaceableName::namespaced()`, and since