Opened 17 months ago

Last modified 16 months ago

#6378 reviewing change

[emscripten] Make DependentString constexpr

Reported by: sergz Assignee: rjeschke
Priority: P3 Milestone:
Module: Core Keywords:
Cc: hfiguiere Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29722755/

Description

Background

We do use a certain number of string literals as a source for immutable DependentString and sometimes we have to introduce some fundamental helper functions like LENGTH_OF what can be avoided by constexpr variables and classes.

What to change

Change String and DependentString in accordance with the requirements to constexpr variables, functions and constructors and adjust the related variables in the code.
Consider removing the function (or macro what ever will be there later) LENGTH_OF mentioned above if it's not used anymore.

Change History (3)

comment:1 Changed 17 months ago by sergz

  • Priority changed from Unknown to P3

comment:2 Changed 16 months ago by rjeschke

  • Owner set to rjeschke

comment:3 Changed 16 months ago by rjeschke

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.