Make Type and SecondFormat enum classes since this is more readable and allows us to have less includes in header files.
Created attachment 406674 [details] Patch
Created attachment 406675 [details] Patch
Comment on attachment 406675 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=406675&action=review > Source/WebCore/platform/DateComponents.h:39 > +enum class SecondFormat { I suggest adding ": uint8_t". > Source/WebCore/platform/DateComponents.h:45 > +enum class DateComponentsType { I suggest adding ": uint8_t".
Created attachment 406683 [details] Patch
Comment on attachment 406675 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=406675&action=review >> Source/WebCore/platform/DateComponents.h:39 >> +enum class SecondFormat { > > I suggest adding ": uint8_t". Done. >> Source/WebCore/platform/DateComponents.h:45 >> +enum class DateComponentsType { > > I suggest adding ": uint8_t". Done.
<rdar://problem/67622319>
Committed r266114: <https://trac.webkit.org/changeset/266114> All reviewed patches have been landed. Closing bug and clearing flags on attachment 406683 [details].