[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.1that 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