Fix comments according to best practices by effective go

This commit is contained in:
xuri 2018-12-05 00:27:19 +08:00 committed by Shane Mulligan
parent b89f75c896
commit 81b43da7b6
3 changed files with 5 additions and 5 deletions

2
col.go
View File

@ -337,7 +337,7 @@ func (f *File) RemoveCol(sheet, column string) {
f.adjustHelper(sheet, col, -1, -1)
}
// Completion column element tags of XML in a sheet.
// completeCol; Completion column element tags of XML in a sheet.
func completeCol(xlsx *xlsxWorksheet, row, cell int) {
buffer := bytes.Buffer{}
for r := range xlsx.SheetData.Row {

View File

@ -90,7 +90,7 @@ func julianDateToGregorianTime(part1, part2 float64) time.Time {
return time.Date(year, time.Month(month), day, hours, minutes, seconds, nanoseconds, time.UTC)
}
// By this point generations of programmers have repeated the algorithm sent
// doTheFliegelAndVanFlandernAlgorithm; By this point generations of programmers have repeated the algorithm sent
// to the editor of "Communications of the ACM" in 1968 (published in CACM,
// volume 11, number 10, October 1968, p.657). None of those programmers seems
// to have found it necessary to explain the constants or variable names set

View File

@ -118,7 +118,7 @@ func trimCell(column []xlsxC) []xlsxC {
return col[0:i]
}
// Read and update property of contents type of XLSX.
// setContentTypes; Read and update property of contents type of XLSX.
func (f *File) setContentTypes(index int) {
content := f.contentTypesReader()
content.Overrides = append(content.Overrides, xlsxOverride{
@ -127,7 +127,7 @@ func (f *File) setContentTypes(index int) {
})
}
// Update sheet property by given index.
// setSheet; Update sheet property by given index.
func (f *File) setSheet(index int, name string) {
var xlsx xlsxWorksheet
xlsx.Dimension.Ref = "A1"
@ -209,7 +209,7 @@ func (f *File) setAppXML() {
f.saveFileList("docProps/app.xml", []byte(templateDocpropsApp))
}
// Some tools that read XLSX files have very strict requirements about the
// replaceRelationshipsNameSpaceBytes; Some tools that read XLSX files have very strict requirements about the
// structure of the input XML. In particular both Numbers on the Mac and SAS
// dislike inline XML namespace declarations, or namespace prefixes that don't
// match the ones that Excel itself uses. This is a problem because the Go XML