Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

For consistency with SafeMin(), SafeMax(), and SafeClamp(). And so that we avoid introducing a namespace.

BUG=webrtc:7459
Review-Url: https://codereview.webrtc.org/2802423002
Cr-Commit-Position: refs/heads/master@{#18756}
Committed: https://chromium.googlesource.com/external/webrtc/+/93ecc5dad05b6996ce42ead407ba0e769ce5b370

[kwiberg-webrtc, 2017-04-08 12:31:13]

The CQ bit was checked by kwiberg@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-08 12:31:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-08 14:32:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-08 14:32:03]

Dry run: Try jobs failed on following builders:
  linux_baremetal on master.tryserver.webrtc (JOB_TIMED_OUT, build hasn't
started yet, builder probably lacks capacity)
  mac_baremetal on master.tryserver.webrtc (JOB_TIMED_OUT, build hasn't started
yet, builder probably lacks capacity)
  win_baremetal on master.tryserver.webrtc (JOB_TIMED_OUT, build hasn't started
yet, builder probably lacks capacity)

[kwiberg-webrtc, 2017-04-09 11:54:32]

Description was changed from

==========
Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

Because for maximum safety, even sub-namespaces should be globally
unique, and because two-letter function names are too likely to
collide with stuff.

Also, for consistency with SafeMin(), SafeMax(), and SafeClamp().

BUG=webrtc:7459
==========

to

==========
Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

For consistency with SafeMin(), SafeMax(), and SafeClamp(). And so that we avoid
introducing a namespace.

BUG=webrtc:7459
==========

[kwiberg-webrtc, 2017-06-13 12:33:29]

The CQ bit was checked by kwiberg@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-13 12:33:32]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[kwiberg-webrtc, 2017-06-13 12:33:59]

Patchset #1 (id:1) has been deleted

[kwiberg-webrtc, 2017-06-13 12:54:27]

kwiberg@webrtc.org changed reviewers:
+ nisse@webrtc.org, ossu@webrtc.org

[nisse-webrtc, 2017-06-13 13:18:54]

lgtm

[commit-bot: I haz the power, 2017-06-13 13:21:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-13 13:21:50]

Dry run: This issue passed the CQ dry run.

[kwiberg-webrtc, 2017-06-26 02:25:19]

The CQ bit was checked by kwiberg@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-26 02:25:26]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-06-26 02:58:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-26 02:58:46]

Dry run: Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

[kwiberg-webrtc, 2017-06-26 03:05:27]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2017-06-26 03:05:37]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-06-26 03:29:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-26 03:29:28]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

[kwiberg-webrtc, 2017-06-26 08:13:54]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2017-06-26 08:14:00]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-06-26 08:31:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498464834643500,
"parent_rev": "001d00a79afb59ca0cd47419ef45827f14ed9b88", "commit_rev":
"93ecc5dad05b6996ce42ead407ba0e769ce5b370"}

[commit-bot: I haz the power, 2017-06-26 08:31:35]

Description was changed from

==========
Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

For consistency with SafeMin(), SafeMax(), and SafeClamp(). And so that we avoid
introducing a namespace.

BUG=webrtc:7459
==========

to

==========
Rename safe_cmp::{Eq,Ne,Lt,Le,Ge,Gt} to Safe{Eq,Ne,Lt,Le,Ge,Gt}

For consistency with SafeMin(), SafeMax(), and SafeClamp(). And so that we avoid
introducing a namespace.

BUG=webrtc:7459

Review-Url: https://codereview.webrtc.org/2802423002
Cr-Commit-Position: refs/heads/master@{#18756}
Committed:
https://chromium.googlesource.com/external/webrtc/+/93ecc5dad05b6996ce42ead40...
==========

[commit-bot: I haz the power, 2017-06-26 08:31:36]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/93ecc5dad05b6996ce42ead40...