Skip to content
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

VB Delegate function conversion error #771

Closed
acrigney opened this issue Sep 29, 2021 · 3 comments
Closed

VB Delegate function conversion error #771

acrigney opened this issue Sep 29, 2021 · 3 comments
Labels
compilation error A bug where the converted output won't compile VB -> C# Specific to VB -> C# conversion

Comments

@acrigney
Copy link

Awesome product guys, a really great job thanks heaps!

I have had a few simple errors but I thought you might be interested in this one.
This did not convert
Public Delegate Function Operation(Of T)() As T
It should have come across to C# as
public delegate T Operation();

The

Erroneous output

public delegate Operation.T Operation();

version used
C# -> VB
[8.4.0] - 2021-09-05

@acrigney acrigney added the C# -> VB Specific to C# -> VB conversion label Sep 29, 2021
@GrahamTheCoder GrahamTheCoder added compilation error A bug where the converted output won't compile VB -> C# Specific to VB -> C# conversion and removed C# -> VB Specific to C# -> VB conversion labels Sep 29, 2021
@GrahamTheCoder
Copy link
Member

Thanks for the report, glad it's helpful!
The error looks likely to be an issue with the "Expand" pre-step in roslyn. Should be an easy workaround hopefully

@acrigney
Copy link
Author

acrigney commented Sep 29, 2021 via email

@GrahamTheCoder
Copy link
Member

The fix should hopefully be released quite soon

Fortunately, not that many people are using VB.NET for serious projects nowadays, but according to the last stackoverflow survey where I saw it included, the various forms of visual basic occupy three of the top five most dreaded languages from this article:
image

When I ended up needing to pick up someone's legacy VB.NET code, it seemed crazy to me that there wasn't a top quality converter available I could find (even paid for ones seemed hopeless when I tried them). So it's mainly to help the world escape from VB, but that's not to say it's never fun too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compilation error A bug where the converted output won't compile VB -> C# Specific to VB -> C# conversion
Projects
None yet
Development

No branches or pull requests

2 participants