unity-missing-references-finder icon indicating copy to clipboard operation
unity-missing-references-finder copied to clipboard

While, should be do while.

Open jwallTopps opened this issue 3 years ago • 1 comments

https://github.com/edcasillas/unity-missing-references-finder/blob/183e49b96580cfcd348ec0437d6fa8f7c4e9b280/Editor/MissingReferencesFinder.cs#L274

Hey, thank you very much for this example. It's super helpful for a project I'm working on. To help out a little bit, I wanted to point out a tiny issue in your code.

Unfortunately this bit here will skip the first property, as you are immediately calling sp.NextVisible(true) right after getting the first property. You might want to change this to do{ ... }while(sp.NextVisible(true)); so it will process the first iterator before advancing.

jwallTopps avatar Apr 13 '22 13:04 jwallTopps

In fact, even if the code skips the first property, it does not affect the final result, since the first property are always seems to be "Base" (of type Generic), which is of no interest.

Ommand avatar Aug 24 '22 08:08 Ommand