Skip to content

feat: Implement header#5

Open
iykekings wants to merge 5 commits into
masterfrom
new-features
Open

feat: Implement header#5
iykekings wants to merge 5 commits into
masterfrom
new-features

Conversation

@iykekings

Copy link
Copy Markdown
Member

Implemented LasCs.header()

@todo

todo Bot commented Feb 22, 2020

Copy link
Copy Markdown

Throw Error instead of printing to the console

las-cs/src/LasCs.cs

Lines 158 to 163 in a0d0012

//TODO: Throw Error instead of printing to the console
Console.WriteLine("There is no header in the file");
}
return uncommentSth.Split("\n").Select(m => Regex.Split(m.Trim(), @"\s+|[.]")[0]).ToArray();
}


This comment was generated by todo based on a TODO comment in a0d0012 in #5. cc @laslibs.

@iykekings

Copy link
Copy Markdown
Member Author

@Khelechy I've almost implemented everything.... Now you can just join stuffs and the work will be done. But you will need to implemnt property method. A good example can be found here at common.go in las-go


https://github.com/laslibs/las-go/blob/7e07e77fb688f280c6e12be6456929815c63e0b7/common.go#L137-L178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant