Project

General

Profile

Actions

Bug #11006

closed

filetostring.cmake generates file literals that exceed the minimum maximum string literal length

Added by Roel Standaert about 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
High
Assignee:
Roel Standaert
Target version:
Start date:
10/13/2022
Due date:
% Done:

100%

Estimated time:

Description

The C++ standard sets the minimum maximum string literal length to 65536 characters: https://timsong-cpp.github.io/cppwp/n4140/implimits#2.15

Some compilers actually have this maximum, like the VS2017 compiler. Other compilers, like clang, will produce a -Woverlength-strings warning.

In order to comply with the standard, we could change our filetostring code to e.g. output an array of hex characters instead of using a string literal.


Related issues 2 (0 open2 closed)

Related to Improvements #8629: Change JavaScript minifierClosedRoel Standaert06/03/2021

Actions
Related to Bug #10941: [MSVC] Wt failed to build with error C2026 on MSVCClosedRoel Standaert09/14/2022

Actions
Actions #1

Updated by Roel Standaert about 2 years ago

Actions #2

Updated by Roel Standaert about 2 years ago

  • Related to Bug #10941: [MSVC] Wt failed to build with error C2026 on MSVC added
Actions #3

Updated by Roel Standaert about 2 years ago

  • Status changed from New to InProgress
Actions #4

Updated by Roel Standaert about 2 years ago

  • Status changed from InProgress to Review
  • Assignee changed from Roel Standaert to Korneel Dumon
Actions #5

Updated by Roel Standaert about 2 years ago

  • Priority changed from Normal to High

This issue is blocking the 4.9.0 release and must be resolved for release, hence bumping priority up to high.

Actions #6

Updated by Korneel Dumon about 2 years ago

  • Assignee changed from Korneel Dumon to Roel Standaert
Actions #7

Updated by Roel Standaert about 2 years ago

  • Status changed from Review to Implemented @Emweb
  • % Done changed from 0 to 100
Actions #8

Updated by Roel Standaert about 2 years ago

  • Status changed from Implemented @Emweb to Resolved
Actions #9

Updated by Roel Standaert about 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF