Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sort and SortField Added #148

Merged
merged 15 commits into from May 21, 2018

Conversation

alinakazi
Copy link
Member

Commented :
import mx.resources.IResourceManager;
import mx.resources.ResourceManager;

Copied ISort.as FlexSDK->Royale
Sort,SortField,ISortField Copied FlexSDK->Royale
Added SortError.as FlexSDK->Royale
Update SortError.as
Update Sort.as
Update SortField
Update MXRoyaleClasses.as
@aharui aharui requested a review from pentapache April 7, 2018 15:28
@aharui
Copy link
Contributor

aharui commented Apr 7, 2018

Alina,

Please comment out or delete any APIs and code that are not being used by your application. We have enough other code paths to migrate that we don't want to take on things we don't need to.

@alinakazi
Copy link
Member Author

Sort and SortField are API's used by my Application . Sort implements ISort and SortField implements ISortField. SortError used in SortField. It was easy to create emulation for ISort , ISortField & SortError. What if my Application is using a method api1.fn_functionName():api2
api1 is the API that is Used by my Application and api1.fn_functionName():api2 is the method that is used by my Application and its return type is api2. So how can I comment out or delete api2.
It is not Clear to me. Please Explain. What if api2 in implements ,extends,constructor, function-parameter, function-return-type etc.

@aharui
Copy link
Contributor

aharui commented Apr 9, 2018

According to your API Report, some APIs in Sort and SortField like "unique", "reverse" and others are not being used by your app. Those APIs should be commented out for now so there is less code that we have to get working.

Sort and SortField are being used by your application so it is fine to have these emulated, but we should not bring over APIs in those classes that are not being used.

@carlosrovira carlosrovira added this to To Do in Emulation Components via automation Apr 9, 2018
Copy link
Contributor

@aharui aharui left a comment

Choose a reason for hiding this comment

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

This is better, but I expected to see changes to SortField as well, and also ISort and ISortField. I think Sort will need to continue to implement ISort.

I would like to understand better your thought process as you made these changes. Maybe you haven't fully grasped what interfaces are and how they work, which is understandable if you are new to languages that support Interfaces. You can email me in private if you don't want to discuss that publicly. I'm wondering if a better feel for these concepts will help you make better decisions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants