logos icon indicating copy to clipboard operation
logos copied to clipboard

Check %property for attributes before assigning to array

Open L1ghtmann opened this issue 3 years ago • 0 comments

What does this implement/fix? Explain your changes.

  • Adds a check for $1 before its use

Does this close any currently open issues?

Should resolve #69

Any relevant logs, error output, etc?

Using the code provided in the issue, logos spits out Use of uninitialized value $1 in split at /home/lightmann/theos/bin/logos.pl line 604. and does not fail the build.

With this change, logos now spits out Tweak.x:8: error: %property declaration is missing attributes (e.g., nonatomic, retain, etc) and does fail the build.

Any other comments?

Clang will spit out an error if the %property is missing a type or name (though the error isn't particularly useful), so it doesn't seem like a check is necessary for them.

Where has this been tested?

Operating System:

Linux (WSL)

Platform:

Target Platform:

Toolchain Version:

SDK Version:

L1ghtmann avatar Jul 04 '22 00:07 L1ghtmann