[TIMOB-25205] Remove platform specific code
GitHub Issue | n/a |
---|---|
Type | Sub-task |
Priority | Low |
Status | Closed |
Resolution | Won't Do |
Resolution Date | 2020-01-28T11:54:10.000+0000 |
Affected Version/s | n/a |
Fix Version/s | n/a |
Components | Windows |
Labels | n/a |
Reporter | Ewan Harris |
Assignee | Unknown |
Created | 2017-08-29T10:22:18.000+0000 |
Updated | 2020-01-28T11:54:10.000+0000 |
I think this does not have to be so high priority and is not mandatory (blocker) for TIMOB-25202 because we can leave the code as it is and nothing should go wrong then because C/C++ macros just ignore the old codes. I agree that we should remove the old codes eventually in the future release.
Just in case I missed something, [~eharris] when you say "remove platform specific code" it means removing "Windows Phone 8.1/Windows 8.1 specific code", right?
[~kiguchi] Yeah you're correct, I think I had typed
Windows Phone 8.1/Windows 8.1
that much that I gave up here and just used platform specific :) I agree that it's not really a super high priority item, more of a cleanup task to help keep the code clean. I think it would be good to group together (by namespace or some other grouping) in case we ever need to revert.Windows was removed