8000 Core - Update CefErrorCodes by campersau · Pull Request #3785 · cefsharp/CefSharp · GitHub
[go: up one dir, main page]

Skip to content

Core - Update CefErrorCodes#3785

Merged
amaitland merged 1 commit intocefsharp:masterfrom
campersau:errorcodes
Sep 3, 2021
Merged

Core - Update CefErrorCodes#3785
amaitland merged 1 commit intocefsharp:masterfrom
campersau:errorcodes

Conversation

@campersau
Copy link
Contributor

Related: #3784

Summary:

Changes:

var text = document.documentElement.textContent,
    result = [],
    comment = [];

text.split("\n").forEach(line => {
    if (!line) {
        if (comment.length > 0) {
            result.push("", ...comment);
            comment = [];
        }
    }
    else if (line.startsWith("//")) {
        comment.push(line);
    }
    else if (line.startsWith("NET_ERROR")) {
        result.push(
            "",
            "/// <summary>",
            ...comment.map(c => "/" + c),
            "/// </summary>",
            line.replace(/NET_ERROR\((.*?), (.*?)\)/, (match, name, code) => name.split("_").map(n => n[0] + n.substring(1).toLowerCase()).join("") + " = " + code + ",")
        );
        comment = [];
    }
});

copy(result.join("\n"))
  • Manually update Icann to ICANN

There are some breaking changes, most notably the rename from Spdy to Http2.

I also compared it to the 93 branch and it looks like there are no changes there.

How Has This Been Tested?
Build works.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Updated documentation

Checklist:

  • Tested the code(if applicable)
  • Commented my code
  • Changed the documentation(if applicable)
  • New files have a license disclaimer
  • The formatting is consistent with the project (project supports .editorconfig)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0