500875 - Find literal strings/arrays/data that should be const static

Status: RESOLVED FIXED

(Developer Infrastructure :: Source Code Analysis, defect)

Description

[Stuart Parmenter]

Make things readonly in the library for the win.

Comment 1

[Migael Moolman]

(In reply to comment #0)
> Make things readonly in the library for the win.

Can you please give some more detailed information about this bug. Id like to have a look at this to learn about the whole process of contributing to the project.

Comment 2

[vipul]

plz reply with more description on bug

Comment 3

[Ashwini.H.M]

Iam a student of mysore india iam a stu from sjce mysore,india and would like to take up dis bug.

Comment 4

[Ashwini.H.M]

Iam a student of mysore India. Iam a student from SJCE mysore,India and i would like to take up dis bug.

Comment 5

[Stuart Parmenter]

The idea here is to look and find data in the application that should be marked readonly in the binary so that it can be mapped in directly, rather than creating copies on the heap

Comment 6

[Ashwini.H.M]

sir i will work on the bug500875 now im plugin to gcc

Comment 7

[Ashwini.H.M]

im doing the task plugin to gcc some errors so i wil do it soon

Comment 8

[Ashwini.H.M]

i want some more ideas abt this bug

Comment 9

[Ashwini.H.M]

Attachment: Find literal strings/arrays that should be declared as both const and static and warns

This JS warns if literal strings/arrays are initialized to variables that are not declared as both const and static.

Comment 10

[Ashwini.H.M]

Attachment: Find literal strings/arrays that should be declared as both const and static and warns

This JS warns if literal strings/arrays are initialised to variables that are not both const and static.

Comment 11

[David Humphrey (:humph)]

Comment on attachment 444315 [details]
Find literal strings/arrays that should be declared as both const and static and warns

Apologies for losing track of this.  If you're still working on it, it needs another reviewer.

Comment 12

[Wim Lewis]

related to bug 74803?

Comment 13

[Sylvestre Ledru [:Sylvestre]]

We have a bunch of checkers in particular readability-non-const-parameter at review phase:
https://searchfox.org/mozilla-central/source/tools/clang-tidy/config.yaml#128