-
Notifications
You must be signed in to change notification settings - Fork 1
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
go fmt #94
Conversation
} | ||
|
||
|
||
type SipEventObject interface{ | ||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
@@ -351,10 +351,11 @@ const FIELD_QOP = 22 | |||
const FIELD_NONCE = 23 | |||
const FIELD_REALM = 24 | |||
const FIELD_ALGORITHM = 25 | |||
const FIELD_CNONCE = 26 | |||
const FIELD_CNONCE = 26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't use ALL_CAPS in Go names; use CamelCase
exported const FIELD_CNONCE should have comment or be unexported
} | ||
|
||
func NewContactHeader(){ | ||
func NewContactHeader() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported function NewContactHeader should have comment or be unexported
@@ -1,9 +1,9 @@ | |||
package message | |||
|
|||
func NewViaHeader(){ | |||
|
|||
func NewViaHeader() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported function NewViaHeader should have comment or be unexported
|
||
func (se * SipEvent) GetSipMessage() *message.SipMsg { | ||
return se.sipmsg | ||
func (se *SipEvent) GetSipMessage() *message.SipMsg { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported method SipEvent.GetSipMessage should have comment or be unexported
} | ||
|
||
type SipEventObject interface{ | ||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
} | ||
|
||
|
||
type SipEventObject interface{ | ||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
@@ -13,14 +13,13 @@ type ListeningPoint interface { | |||
Send(string, string, string) error | |||
} | |||
|
|||
type SipEventObject interface{ | |||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
@@ -14,20 +13,18 @@ type ListeningPoint interface { | |||
Send(string, string, string) error | |||
} | |||
|
|||
|
|||
type SipEventObject interface{ | |||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
@@ -351,10 +351,11 @@ const FIELD_QOP = 22 | |||
const FIELD_NONCE = 23 | |||
const FIELD_REALM = 24 | |||
const FIELD_ALGORITHM = 25 | |||
const FIELD_CNONCE = 26 | |||
const FIELD_CNONCE = 26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't use ALL_CAPS in Go names; use CamelCase
exported const FIELD_CNONCE should have comment or be unexported
} | ||
|
||
func NewContactHeader(){ | ||
func NewContactHeader() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported function NewContactHeader should have comment or be unexported
@@ -1,9 +1,9 @@ | |||
package message | |||
|
|||
func NewViaHeader(){ | |||
|
|||
func NewViaHeader() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported function NewViaHeader should have comment or be unexported
|
||
func (se * SipEvent) GetSipMessage() *message.SipMsg { | ||
return se.sipmsg | ||
func (se *SipEvent) GetSipMessage() *message.SipMsg { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported method SipEvent.GetSipMessage should have comment or be unexported
} | ||
|
||
type SipEventObject interface{ | ||
type SipEventObject interface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exported type SipEventObject should have comment or be unexported
No description provided.