recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

call qsprintf() without warnings

Summary:
After T13217 import_repository_symbols.php was showing a lot of warnings, using %LQ fixes that.
I'm aware, that there are changes planned to the whole managing the symbols complex but until then less warnings are nice.

Test Plan: No more warnings when updating symbols

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D19962

authored by

Marc Lindenberg and committed by
epriestley 360ebce0 e8069dfe

+2 -2
+2 -2
scripts/symbols/import_repository_symbols.php
··· 110 110 $conn_w, 111 111 'INSERT INTO %T 112 112 (repositoryPHID, symbolContext, symbolName, symbolType, 113 - symbolLanguage, lineNumber, pathID) VALUES %Q', 113 + symbolLanguage, lineNumber, pathID) VALUES %LQ', 114 114 $symbol->getTableName(), 115 - implode(', ', $chunk)); 115 + $chunk); 116 116 } 117 117 } 118 118