Page MenuHomePhabricator

Fix some straggling qsprintf() warnings in repository import
ClosedPublic

Authored by epriestley on Dec 12 2018, 1:25 PM.
Tags
None
Referenced Files
F15527259: D19869.id47449.diff
Tue, Apr 22, 2:15 AM
F15507155: D19869.id47447.diff
Tue, Apr 15, 4:52 PM
F15503287: D19869.id.diff
Mon, Apr 14, 8:40 AM
F15501101: D19869.diff
Sun, Apr 13, 8:45 PM
F15471565: D19869.id47447.diff
Sat, Apr 5, 5:57 AM
F15444594: D19869.diff
Thu, Mar 27, 10:14 AM
F15426274: D19869.diff
Mar 23 2025, 7:54 AM
F15407738: D19869.diff
Mar 18 2025, 7:14 PM
Subscribers
None

Details

Summary

Ref T13217. See https://discourse.phabricator-community.org/t/unsafe-raw-string-warnings-while-importing-git-commits/2191.

Hunt down and fix two more qsprintf() things.

I just converted the "performance optimization" into a normal, safe call since we're dealing with far less SVN stuff nowadays and the actual issue has been lost in the mists of time. If it resurfaces, we can take another look.

Test Plan

Imported some commits, no longer saw these warnings in the daemon logs.

Diff Detail

Repository
rP Phabricator
Branch
svn1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21300
Build 28979: Run Core Tests
Build 28978: arc lint + arc unit