refactor(ivy): update TNodeType (#29343)
- Remove an extra type `ViewOrElement`, which even had the same numeric value as `View`. - Updates comment to remove part about alleged bit-masking that we could be doing here. We aren't using this with bitmasks, and if we were, everything would be a `NodeType.Container`, because it's value was `0`. - Updates the number values to be simple, human-readable integers, since we're not using these with any kind of bit-manipulation. - Add comments about each type. PR Close #29343
This commit is contained in:
parent
604f37b679
commit
e8df000e97
|
@ -12,17 +12,33 @@ import {LView, TView} from './view';
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* TNodeType corresponds to the TNode.type property. It contains information
|
* TNodeType corresponds to the {@link TNode} `type` property.
|
||||||
* on how to map a particular set of bits in TNode.flags to the node type.
|
|
||||||
*/
|
*/
|
||||||
export const enum TNodeType {
|
export const enum TNodeType {
|
||||||
Container = 0b000,
|
/**
|
||||||
Projection = 0b001,
|
* The TNode contains information about an {@link LContainer} for embedded views.
|
||||||
View = 0b010,
|
*/
|
||||||
Element = 0b011,
|
Container = 0,
|
||||||
ViewOrElement = 0b010,
|
/**
|
||||||
ElementContainer = 0b100,
|
* The TNode contains information about an `<ng-content>` projection
|
||||||
IcuContainer = 0b101,
|
*/
|
||||||
|
Projection = 1,
|
||||||
|
/**
|
||||||
|
* The TNode contains information about an {@link LView}
|
||||||
|
*/
|
||||||
|
View = 2,
|
||||||
|
/**
|
||||||
|
* The TNode contains information about a DOM element aka {@link RNode}.
|
||||||
|
*/
|
||||||
|
Element = 3,
|
||||||
|
/**
|
||||||
|
* The TNode contains information about an `<ng-container>` element {@link RNode}.
|
||||||
|
*/
|
||||||
|
ElementContainer = 4,
|
||||||
|
/**
|
||||||
|
* The TNode contains information about an ICU comment used in `i18n`.
|
||||||
|
*/
|
||||||
|
IcuContainer = 5,
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in New Issue