TermiT / Flycut

Clean and simple clipboard manager for developers

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Possibly incorrect boolean value when calling SMLoginItemSetEnabled

MarkJerde opened this issue · comments

Hi TermiT,
I'm just getting around to reviewing some merged changes and noticed that in toggleLoadOnStartup: the same SMLoginItemSetEnabled call is made in both the enabled and disabled paths. (The line of code is identical.) I'm pretty sure it should be NO in the second case, but wanted your input before changing it. I'll include the change in a PR soon if you agree.

SMLoginItemSetEnabled((__bridge CFStringRef)kFlycutHelperId, YES);

Thanks and best regards!
Mark

Yes, definitely copy-paste issue :( Thanks for catching it.

This should be closed since it was fixed in #254.