Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(465)

Issue 1567123006: Use ::std for is_empty, is_class, add_xxx_reference, remove decay. (Closed)

Created:
4 years, 11 months ago by bungeman-skia
Modified:
4 years, 11 months ago
Reviewers:
mtklein
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Use ::std for is_empty, is_class, add_xxx_reference, remove decay. SkTLogic.h contains implemenetations of is_empty, is_class, decay, decay_t, add_lvalue_reference, add_rvalue_reference, and add_rvalue_reference_t. The declaration of is_class is only used by is_empty. The current implementation of is_empty will not work correctly with 'final'. The current implementation of add_lvalue_reference does not support clarifications in c++17. No Skia code is currently using decay, decay_t, add_rvalue_reference, and add_rvalue_reference_t. In addition, there are no known issues with any of these in ::std with any supported compiler. The implementations of is_function and is_convertible are commented to note why they are still present. Committed: https://skia.googlesource.com/skia/+/0e101667d604e4902eb24e0c0939d4f5b12c96ec

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -43 lines) Patch
M include/private/SkTLogic.h View 3 chunks +5 lines, -39 lines 0 comments Download
M include/private/SkUniquePtr.h View 2 chunks +2 lines, -2 lines 0 comments Download
M src/core/SkRecord.h View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 14 (8 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1567123006/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1567123006/1
4 years, 11 months ago (2016-01-08 18:34:56 UTC) #5
bungeman-skia
I think this is pretty much the most we can remove for now until we ...
4 years, 11 months ago (2016-01-08 18:41:25 UTC) #7
mtklein
lgtm
4 years, 11 months ago (2016-01-08 18:51:25 UTC) #8
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 11 months ago (2016-01-08 18:57:39 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1567123006/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1567123006/1
4 years, 11 months ago (2016-01-08 19:04:33 UTC) #12
commit-bot: I haz the power
4 years, 11 months ago (2016-01-08 19:05:12 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/0e101667d604e4902eb24e0c0939d4f5b12c96ec

Powered by Google App Engine
This is Rietveld 408576698