Skip to content

Conversation

hexihaohao
Copy link

add support to VBNet(Visual Basic .NET)

@hhatto hhatto self-requested a review May 9, 2021 03:15
@@ -157,6 +157,9 @@ var Exts = map[string]string{
"jai": "JAI",
"java": "Java",
"jsp": "JSP",
"vb": "VBNet",
"resx": "VBNet",
"vbproj": "VBNet",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am thinking that vbproj should be MSBuild script instead of VBNet. What do you think?
This was based on other cloc-like tools (like cloc, scc, tokei).

@@ -157,6 +157,9 @@ var Exts = map[string]string{
"jai": "JAI",
"java": "Java",
"jsp": "JSP",
"vb": "VBNet",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of VBNet, I want to use Visual Basic as a programming language.

@@ -157,6 +157,9 @@ var Exts = map[string]string{
"jai": "JAI",
"java": "Java",
"jsp": "JSP",
"vb": "VBNet",
"resx": "VBNet",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that should use XML or XML Resource or .NET Resource instead of VBNet. Because .resx files is resource definition for .NET framework.

ref: https://docs.microsoft.com/en-us/dotnet/framework/resources/creating-resource-files-for-desktop-apps

@hexihaohao hexihaohao closed this May 10, 2021
@hhatto hhatto mentioned this pull request May 13, 2021
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.

2 participants