Skip to content

Ddd#10

Open
gastontole wants to merge 9 commits intoDGh0st:masterfrom
gastontole:ddd
Open

Ddd#10
gastontole wants to merge 9 commits intoDGh0st:masterfrom
gastontole:ddd

Conversation

@gastontole
Copy link

No description provided.

Copy link
Owner

@DGh0st DGh0st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As is this is not going to be merged.

Looks like there are bunch of AI generated .md files which aren't really helpful for anything so should not be included. And the .log file is also not helpful should not be included.

The changes break lot of things that need to be fixed if you want this merged:

  • Removes armv7 support (32-bit devices) which is what this tweak was mainly focused towards.
  • Conversion to ARC is wrong. Should not be calling CFRelease on prefs. Its needs to be a CFBridgingRelease before casting to NSDictionary *.
  • Older devices don't have haptic feedback, it needs to use haptic feedback if supported otherwise should use the old approach of vibrations.
  • All the gesture methods are being removed which would basically break all the functionality, which needs to be reverted.
  • Preferences are not being compiled anymore which it still should.

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

Successfully merging this pull request may close these issues.

2 participants