Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

vet fixes #1212

Merged
merged 9 commits into from
Dec 2, 2014
Merged

vet fixes #1212

merged 9 commits into from
Dec 2, 2014

Conversation

ceh
Copy link
Contributor

@ceh ceh commented Dec 1, 2014

No description provided.

ceh added 9 commits December 1, 2014 19:57
Fix the following vet reports:

logstreamer/filehandling.go:327: arg l.logfiles for printf verb %s of wrong type: logstreamer.Logfiles
logstreamer/filehandling.go:481: arg newLogfiles for printf verb %s of wrong type: logstreamer.Logfiles
logstreamer/reader.go:94: arg l.SeekPosition for printf verb %s of wrong type: int64
Fix the following vet report:

pipeline/config.go:560: struct field tag `toml:use_framing"` not compatible with reflect.StructTag.Get
Fix the following vet report:

plugins/graphite/whisper.go:159: wrong number of args for format in Errorf call: 1 needed but 2 args
Fix the following vet reports:

plugins/irc/irc_output.go:476: arg output.TimeBeforeRejoin for printf verb %s of wrong type: uint
plugins/irc/irc_output_test.go:382: arg ircOutput.TimeBeforeRejoin for printf verb %s of wrong type: uint
Fix the following vet reports:

plugins/kafka/kafka_input_test.go:104: possible formatting directive in Fatal call
plugins/kafka/kafka_input_test.go:196: possible formatting directive in Fatal call
Fix the following vet report:

plugins/process/process_input.go:104: struct field tag `toml: trim` not compatible with reflect.StructTag.Get
Fix the following vet reports:

plugins/statsd/stat_filter.go:122: arg stat for printf verb %s of wrong type: github.com/mozilla-services/heka/pipeline.Stat
plugins/statsd/statsd_input.go:136: arg stat for printf verb %s of wrong type: github.com/mozilla-services/heka/pipeline.Stat
Fix the following vet reports:

plugins/udp/udp_input.go:83: no formatting directive in Errorf call
plugins/udp/udp_input.go:167: no formatting directive in Errorf call
Fix the following vet report:

sandbox/lua/lua_sandbox_test.go:756: arg 1e9 for printf verb %s of wrong type: float64
@ceh ceh changed the title Vet fixes vet fixes Dec 1, 2014
rafrombrc added a commit that referenced this pull request Dec 2, 2014
@rafrombrc rafrombrc merged commit cf86fa7 into mozilla-services:dev Dec 2, 2014
@ceh ceh deleted the vet-fixes branch December 3, 2014 06:03
relistan pushed a commit to Nitro/heka that referenced this pull request Jan 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants