Open Bug 1636049 Opened 5 years ago Updated 5 years ago

[meta] Application of modern C++ coding style in CacheAPI

Categories

(Core :: Storage: Cache API, enhancement)

Product:
Core
Component:
Storage: Cache API
Type:
enhancement

Tracking

()

People

(Reporter: ssengupta, Unassigned)

References

Details

(Keywords: meta)

The goal is to revise the C++ code in the CacheAPI component to apply modern C++ coding style where applicable. The revisions could include, but do not have to be limited to, the following:

  • Replacing unsafe/raw pointer usage where it does not pose considerable overhead.

  • Returning by return types instead of by out-parameters in methods, preferring mozilla::Result where data on success and error code otherwise desired.

  • Replacing magic/sentinel values to represent empty/unused objects with mozilla::maybe.

  • Replacing MOZ_CRASH with error return where the caller is equipped to handle it.

Depends on: 1636050
Depends on: 1627892
You need to log in before you can comment on or make changes to this bug.