Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename from iterator to iterable and split the functionality #19831

Merged
merged 9 commits into from
Sep 22, 2020
Merged

Rename from iterator to iterable and split the functionality #19831

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ca2d02d
rename from iterator to iterable and split the functionality
todortotev Sep 14, 2020
60b8bb5
brought back the iterator variable name due to tests failing
todortotev Sep 18, 2020
2cf0346
fix the issue with failing CI
todortotev Sep 18, 2020
ce81af1
remove parentheses before data
todortotev Sep 18, 2020
2558946
move the check in the iterator case
todortotev Sep 18, 2020
c629822
check if executing the function returns an object
todortotev Sep 18, 2020
cf19623
remove symbol import
todortotev Sep 22, 2020
f7052e8
modify the check once again
todortotev Sep 22, 2020
28bb959
Added tests and fixed edge cases for DevTools iterator handling
Sep 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 8 additions & 3 deletions packages/react-devtools-shared/src/utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,8 @@ export type DataType =
| 'html_all_collection'
| 'html_element'
| 'infinity'
| 'iterator'
| 'iterable'
| 'opaque_iterable'
| 'nan'
| 'null'
| 'number'
Expand Down Expand Up @@ -435,8 +436,10 @@ export function getDataType(data: Object): DataType {
// If it doesn't error, we know it's an ArrayBuffer,
// but this seems kind of awkward and expensive.
return 'array_buffer';
} else if (data[Symbol.iterator]() === 'data') {
todortotev marked this conversation as resolved.
Show resolved Hide resolved
return 'opaque_iterable';
} else if (typeof data[Symbol.iterator] === 'function') {
return 'iterator';
return 'iterable';
} else if (data.constructor && data.constructor.name === 'RegExp') {
return 'regexp';
} else {
Expand Down Expand Up @@ -613,7 +616,7 @@ export function formatDataForPreview(
} else {
return shortName;
}
case 'iterator':
case 'iterable':
const name = data.constructor.name;
if (showFormattedValue) {
// TRICKY
Expand Down Expand Up @@ -653,6 +656,8 @@ export function formatDataForPreview(
} else {
return `${name}(${data.size})`;
}
case 'opaque_iterable':
return `${name}(${data.size})`;
case 'date':
return data.toString();
case 'object':
Expand Down