Skip to content

Update the XML spec #72

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.

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

Merged
merged 8 commits into from
Mar 3, 2016
Merged

Update the XML spec #72

merged 8 commits into from
Mar 3, 2016

Conversation

zhengbli
Copy link
Contributor

This PR updates the XML spec with the Windows 10 TH2 release version (10586).

Review on Reviewable

if Seq.contains "any" allTypes then "any" else String.concat " | " allTypes
else
// Check if is array type, which looks like "sequence<DOMString>"
let genericMatch = Regex.Match(objDomType, @"^(\w+)<(\w+)>$")
let unescaped = objDomType.Replace("&lt;", "<").Replace("&gt;", ">")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mhegazy
Copy link
Contributor

mhegazy commented Mar 1, 2016

LGTM

@falsandtru
Copy link
Contributor

👍

@@ -383,7 +388,7 @@ interface IDBObjectStore {
deleteIndex(indexName: string): void;
get(key: any): IDBRequest;
index(name: string): IDBIndex;
openCursor(range?: any, direction?: string): IDBRequest;
openCursor(range?: IDBKeyRange | IDBValidKey, direction?: string): IDBRequest;
put(value: any, key?: any): IDBRequest;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also should be put(value: any, key?: IDBValidKey): IDBRequest;. Other methods are same.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed now

zhengbli added a commit that referenced this pull request Mar 3, 2016
@zhengbli zhengbli merged commit baf84a4 into master Mar 3, 2016
@zhengbli zhengbli deleted the updateXML branch March 3, 2016 19:04
@zhengbli zhengbli mentioned this pull request Mar 21, 2016
orta pushed a commit to orta/TSJS-lib-generator that referenced this pull request Jun 23, 2021
Bumps [webidl2](https://github.com./w3c/webidl2.js) from 23.13.0 to 23.13.1.
- [Release notes](https://github.com./w3c/webidl2.js/releases)
- [Changelog](https://github.com./w3c/webidl2.js/blob/gh-pages/CHANGELOG.md)
- [Commits](w3c/webidl2.js@v23.13.0...v23.13.1)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants