Opened 7 years ago

Closed 3 years ago

Last modified 3 years ago

#4446 closed enhancement (fixed)

Get rid of Dehydra scripts

Reported by: Itms Owned by: Itms
Priority: Nice to Have Milestone: Alpha 25
Component: Build & Packages Keywords:
Cc: Philip Taylor Patch:

Description (last modified by Stan)

We currently have one Dehydra script under ps/trunk/build/dehydra which can check our printf-like functions. It is supposed to complement GCC's own analysis.

However, Dehydra is completely left abandoned (https://developer.mozilla.org/en-US/docs/Archive/Mozilla/Dehydra) and we don't really use it anyways.

We should get rid of this and include the custom script in the static code analysis we are setting up. Basically, are cppcheck and clang's scan-build both unable to detect the issue? If yes, try to extend somehow one of them to detect it. Else just drop the dehydra code.

Change History (3)

comment:1 by Stan, 7 years ago

Also one shouldn't forget to delete it from the wiki wiki:CodeQuality

Somehow related to the static analysis #3958, #4447

Last edited 7 years ago by Stan (previous) (diff)

comment:2 by Vladislav Belov, 3 years ago

Resolution: fixed
Status: newclosed

In 25523:

Removes dehydra build scripts following rPrP25521. Fixes #4446

Refs rP7175.

comment:3 by Stan, 3 years ago

Description: modified (diff)
Milestone: BacklogAlpha 25
Note: See TracTickets for help on using tickets.