Skip to content
This repository was archived by the owner on May 15, 2024. It is now read-only.

Optimmization in TTS.Android#1450

Merged
mattleibow merged 2 commits intoxamarin:mainfrom
Mrnikbobjeff:feature/PR1030Subset
Oct 22, 2020
Merged

Optimmization in TTS.Android#1450
mattleibow merged 2 commits intoxamarin:mainfrom
Mrnikbobjeff:feature/PR1030Subset

Conversation

@Mrnikbobjeff
Copy link
Copy Markdown
Contributor

Description of Change

Since #1030 was not merged I decided to split it up in to smaller parts which can easily be merged without requiring much review. See #1030 for pr details. This pr contains simply the enum check optimization (dont call object.equals on enums as they dont provide a enum specific override)

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard
  • Updated documentation (see walkthrough)

@mattleibow mattleibow merged commit 5913f57 into xamarin:main Oct 22, 2020
@mattleibow mattleibow added this to the 1.6.0 milestone Oct 22, 2020
@mattleibow
Copy link
Copy Markdown
Member

Thanks for this PR.

@Mrnikbobjeff Mrnikbobjeff deleted the feature/PR1030Subset branch October 23, 2020 08:22
dimonovdd added a commit to dimonovdd/Essentials that referenced this pull request Oct 24, 2020
* main:
  [Enhancement] Share Multiple Files (xamarin#1445)
  [FilePicker] Add support for selecting PDF files (Issue 1404) (xamarin#1412)
  Subset of 1030 pr, easy to review (xamarin#1450)
  Add docs for ContinueUserActivity
  Add ContinueUserActivity impl for Universal App Links
dimonovdd added a commit to dimonovdd/Essentials that referenced this pull request Oct 24, 2020
* main:
  [Enhancement] Share Multiple Files (xamarin#1445)
  [FilePicker] Add support for selecting PDF files (Issue 1404) (xamarin#1412)
  Subset of 1030 pr, easy to review (xamarin#1450)
  Add docs for ContinueUserActivity
  Add ContinueUserActivity impl for Universal App Links
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants