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

Function type variable and lazy stored properties not detected #13

Closed
Arthur111 opened this issue Sep 6, 2018 · 7 comments
Closed

Function type variable and lazy stored properties not detected #13

Arthur111 opened this issue Sep 6, 2018 · 7 comments

Comments

@Arthur111
Copy link

You can define a function type variable or a lazy stored property and not using it and your script is blind.

Lazy stored properties

Function type variable

I do not think closing an active bug will solve the problem.

@PaulTaykalo
Copy link
Owner

Yep.

Again. what exact case are you referencing to?

class A {
	lazy var item: String = ""
}

image

Will leave it here.
https://opensource.guide/how-to-contribute/#how-to-submit-a-contribution

@PaulTaykalo
Copy link
Owner

@Arthur111 is there any updates on this?

@Arthur111
Copy link
Author

class A
{
static let a = A()
}

class B
{
lazy var c = A.a // you don't use c
}

c is not detected

class D
{
var e: (()->())! // you don't use e
}

e is not detected

So as i said Lazy stored properties and Function type variable are not detected.

@Arthur111
Copy link
Author

Issue not solved.
Some var are used and they are detected.
Some lazy var are not used and not detected.
Some functions var are used and detected.

In other words this script is not usable.

@PaulTaykalo
Copy link
Owner

@Arthur111 Sure, as soon as there'll be other examples, and some free time I will or will not try or not try to fix. If ever there were a chance for you to fix it. Wait...

@PaulTaykalo
Copy link
Owner

@Arthur111 https://github.com/PaulTaykalo/swift-scripts#known-issues
If you find any bug - feel free to open pull request with a fix.

@PaulTaykalo
Copy link
Owner

By the way, there are Hacktoberfest challenge at the moment, so there even more reasons to create a pull request,

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

No branches or pull requests

2 participants