Closed Bug 1300080 Opened 3 years ago Closed 3 years ago

[Static Analysis][Coverity] Need to store modelling file in mozilla repository

Categories

(Firefox Build System :: Source Code Analysis, defect)

defect
Not set

Tracking

(firefox51 fixed)

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: andi, Assigned: andi)

References

(Blocks 1 open bug)

Details

(Keywords: coverity)

Attachments

(1 file)

As we are also using Coveirty as a static analysis tool and because it has support for analysis modelling through a cpp file where hard to debug function can be re-written in order to diminish the number of false positive we would need to store this file on mercurial in order to have a history for the modifications of that file.

This will not be build in our code base.
As the original author of this file, I agree we should do it!
Comment on attachment 8787602 [details]
Bug 1300080 - add to mozilla repository the modeling file for coverity.

https://reviewboard.mozilla.org/r/76322/#review74440

Please move this to tools/coverity/model.cpp.  r=me with that!
Attachment #8787602 - Flags: review?(ehsan) → review+
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6ed0e5a8247a
add to mozilla repository the modeling file for coverity. r=Ehsan
https://hg.mozilla.org/mozilla-central/rev/6ed0e5a8247a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.