File: abseil-time-subtraction.rst

package info (click to toggle)
llvm-toolchain-11 1%3A11.0.1-2
  • links: PTS, VCS
  • area: main
  • in suites: bullseye
  • size: 995,808 kB
  • sloc: cpp: 4,767,656; ansic: 760,916; asm: 477,436; python: 170,940; objc: 69,804; lisp: 29,914; sh: 23,855; f90: 18,173; pascal: 7,551; perl: 7,471; ml: 5,603; awk: 3,489; makefile: 2,573; xml: 915; cs: 573; fortran: 503; javascript: 452
file content (39 lines) | stat: -rw-r--r-- 1,286 bytes parent folder | download | duplicates (7)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
.. title:: clang-tidy - abseil-time-subtraction

abseil-time-subtraction
=======================

Finds and fixes ``absl::Time`` subtraction expressions to do subtraction
in the Time domain instead of the numeric domain.

There are two cases of Time subtraction in which deduce additional type
information:

- When the result is an ``absl::Duration`` and the first argument is an
  ``absl::Time``.
- When the second argument is a ``absl::Time``.

In the first case, we must know the result of the operation, since without that
the second operand could be either an ``absl::Time`` or an ``absl::Duration``.
In the second case, the first operand *must* be an ``absl::Time``, because
subtracting an ``absl::Time`` from an ``absl::Duration`` is not defined.

Examples:

.. code-block:: c++

  int x;
  absl::Time t;

  // Original - absl::Duration result and first operand is a absl::Time.
  absl::Duration d = absl::Seconds(absl::ToUnixSeconds(t) - x);

  // Suggestion - Perform subtraction in the Time domain instead.
  absl::Duration d = t - absl::FromUnixSeconds(x);


  // Original - Second operand is an absl::Time.
  int i = x - absl::ToUnixSeconds(t);

  // Suggestion - Perform subtraction in the Time domain instead.
  int i = absl::ToInt64Seconds(absl::FromUnixSeconds(x) - t);