|222 ↗||(On Diff #126044)|
Can you initialize Exynos to some value here?
|233–234 ↗||(On Diff #126044)|
if (I.consume_front("CPU part")) I.ltrim("\t :").getAsInteger(16, Part);
As an example of why I prefer consume_front, it fixes what appears to be above in the preceding if statement (about CPU variant), where you've hardcoded 8 but actually meant 11.
|236 ↗||(On Diff #126044)|
Can you move the declaration of Exynos down here and just say:
unsigned Exynos = (Variant << 12) + Part;
That also simultaneously addresses my previous comment about initialization.