Inconsistent Error Handling in nat Package

[ashwek]

A minor inconsistency in error handling between two functions in the Port type.

In the Int function, there is a comment suggesting that the error should be ignored:

go-connections/nat/nat.go

Lines 80 to 87 in fa09c95

	// Int returns the port number of a Port as an int
	func (p Port) Int() int {
	portStr := p.Port()
	// We don't need to check for an error because we're going to
	// assume that any error would have been found, and reported, in NewPort()
	port, _ := ParsePort(portStr)
	return port
	}

However, in the Range function, there is no similar comment, and the error is returned, even though the same case applies here:

go-connections/nat/nat.go

Lines 89 to 92 in fa09c95

	// Range returns the start/end port numbers of a Port range as ints
	func (p Port) Range() (int, int, error) {
	return ParsePortRangeToInt(p.Port())
	}

Suggestion

For consistency and to adhere to the principle of least surprise, it would be beneficial to add a similar comment in the Range function and ignore the error.