Sign upLogin

abzcoding/hget

View on GitHub
Star

Showing 6 of 8 total issues

Function Execute has 61 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func Execute(url string, state *State, conn int, skiptls bool, proxy string, bwLimit string) {
    // Capture OS interrupt signals
    signalChan := make(chan os.Signal, 1)
    signal.Notify(signalChan,
        syscall.SIGHUP,
Severity: Minor
Found in main.go - About 1 hr to fix

    Function NewHTTPDownloader has 52 lines of code (exceeds 50 allowed). Consider refactoring.
    Open

    func NewHTTPDownloader(url string, par int, skipTLS bool, proxyServer string, bwLimit string) *HTTPDownloader {
        var resumable = true
        client := ProxyAwareHTTPClient(proxyServer, skipTLS)
     
        parsed, err := stdurl.Parse(url)
    Severity: Minor
    Found in http.go - About 1 hr to fix

      Avoid deeply nested control flow statements.
      Open

                          if err := s.Save(); err != nil {
                              Errorf("%v\n", err)
                          }
      Severity: Major
      Found in main.go - About 45 mins to fix

        Function Execute has a Cognitive Complexity of 22 (exceeds 20 allowed). Consider refactoring.
        Open

        func Execute(url string, state *State, conn int, skiptls bool, proxy string, bwLimit string) {
            // Capture OS interrupt signals
            signalChan := make(chan os.Signal, 1)
            signal.Notify(signalChan,
                syscall.SIGHUP,
        Severity: Minor
        Found in main.go - About 35 mins to fix

        Your code does not pass gofmt in 1 place. Go fmt your code!
        Open

        package main
        Severity: Minor
        Found in joiner.go by gofmt

        exported function Resume should have comment or be unexported
        Open

        func Resume(task string) (*State, error) {
        Severity: Minor
        Found in resume.go by golint
        Severity
        Category
        Status
        Source
        Language