Homec4science

Fix various TempFile issues

Authored by epriestley <git@epriestley.com> on Jul 18 2012, 01:15.

Description

Fix various TempFile issues

Summary:
This class is a mess, I think it hasn't been touched since it came out of FB.

  • $dir parameter to constructor does nothing.
  • preserve() does nothing.
  • Fix documentation.
  • Use more Filesystem.

Test Plan: Used setPreserveFile() in a future diff.

Reviewers: btrahan, vrana

Reviewed By: btrahan

CC: aran

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

Details

Committed
epriestley <git@epriestley.com>Jul 18 2012, 01:15
Pushed
aubortMar 17 2017, 12:03
Parents
rPHU5b211411be61: Add ldap_sprintf() for LDAP escaping
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHU5363f990f834: Fix various TempFile issues (authored by epriestley <git@epriestley.com>).Jul 18 2012, 01:15