Midori Web Browser

Merge lp://qastaging/~ricotz/midori/format-security into lp://qastaging/midori

  1. format-security
  2. Merge into trunk
Proposed by Rico Tzschichholz
Status: Merged
Approved by: Cris Dywan
Approved revision: 7026
Merged at revision: 7026
Proposed branch: lp://qastaging/~ricotz/midori/format-security
Merge into: lp://qastaging/midori
Diff against target: 23 lines (+2/-0)
2 files modified
extensions/external-download-manager.vala (+1/-0)
extensions/transfers.vala (+1/-0)
To merge this branch: bzr merge lp://qastaging/~ricotz/midori/format-security
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Cody Garver (community) Approve
Review via email: mp+268570@code.qastaging.launchpad.net

Commit message

Fix warnings for -Wformat-security

Description of the change

Fix warnings for -Wformat-security

To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

https://fedoraproject.org/wiki/Format-Security-FAQ#How_do_I_fix_these_errors.3F

review: Approve
Revision history for this message
Cris Dywan (kalikiana) wrote :

We should look into getting these warnings as errors on our builds, or see if Vala has any way to warn about it.

The change itself is obviously correct.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: