Closed
Bug 1899524
Opened 4 months ago
Closed 3 months ago
Use nsACString types for the crash annotations' generated code
Categories
(Toolkit :: Crash Reporting, task)
Toolkit
Crash Reporting
Tracking
()
RESOLVED
FIXED
129 Branch
Tracking | Status | |
---|---|---|
firefox129 | --- | fixed |
People
(Reporter: gsvelto, Assigned: gsvelto)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
The code generated from CrashAnnotations.h.in
used bare const char*
when dealing with string because it was included in the crash reporter client where XUL strings could not be used. With the old crash reporter client gone we can now use nsACString instead which simplifies things and opens up using this code with non-NUL-terminated strings.
Assignee | ||
Comment 1•4 months ago
|
||
Updated•4 months ago
|
Assignee: nobody → gsvelto
Attachment #9404526 -
Attachment description: WIP: Bug 1899524 - Use XUL strings in the code generated to deal with crash annotations → Bug 1899524 - Use XUL strings in the code generated to deal with crash annotations r=afranchuk
Status: NEW → ASSIGNED
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/52dc5cfa5439
Use XUL strings in the code generated to deal with crash annotations r=afranchuk
Comment 3•3 months ago
|
||
Backed out for causing build bustages with rust_minidump_writer_linux
- Backout link
- Push with failures
- Failure Log
- Failure line: error[E0658]: use of unstable library feature 'offset_of'
error: could not compilerust_minidump_writer_linux
(lib) due to 4 previous errors
gmake[4]: *** [/builds/worker/checkouts/gecko/config/makefiles/rust.mk:498: force-cargo-library-build] Error 101
Flags: needinfo?(gsvelto)
Assignee | ||
Updated•3 months ago
|
Flags: needinfo?(gsvelto)
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f8a80c4ad32
Use XUL strings in the code generated to deal with crash annotations r=afranchuk
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
status-firefox129:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•