Page MenuHomePhabricator

Fix some format strings
ClosedPublic

Authored by joshuaspence on May 25 2015, 11:25 AM.
Tags
None
Referenced Files
F12837053: D12996.diff
Thu, Mar 28, 4:30 PM
Unknown Object (File)
Fri, Mar 22, 12:01 PM
Unknown Object (File)
Tue, Mar 5, 9:02 PM
Unknown Object (File)
Feb 8 2024, 4:29 PM
Unknown Object (File)
Feb 1 2024, 8:55 PM
Unknown Object (File)
Dec 24 2023, 7:06 PM
Unknown Object (File)
Dec 21 2023, 2:20 AM
Unknown Object (File)
Dec 15 2023, 1:42 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP1b12249b2ca3: Fix some format strings
Summary

These format strings use %d instead of %s.

Test Plan

Eyeball it.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6250
Build 6272: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Fix some format strings.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

What's up with those test failures?

src/applications/xhprof/controller/PhabricatorXHProfSampleListController.php
59–60 ↗(On Diff #31335)

not 100% sure this works without pht()?

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementProbeWorkflow.php
72 ↗(On Diff #31335)

This one proably isn't right, since it's sprintf() [not pht()].

This revision is now accepted and ready to land.May 25 2015, 11:27 AM
joshuaspence added inline comments.
src/applications/xhprof/controller/PhabricatorXHProfSampleListController.php
59–60 ↗(On Diff #31335)

I figured it would be cast to a string due to the string concatenation... although PhutilNumber doesn't have a __toString method so I don't know how this all works.

joshuaspence marked an inline comment as done.
joshuaspence edited edge metadata.

Smaller changeset

This revision was automatically updated to reflect the committed changes.