This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] __support: Implement strtol_l() / strtoul_l() consistently
AbandonedPublic

Authored by Mordante on Jun 1 2023, 1:24 AM.

Details

Reviewers
philnik
brad
Group Reviewers
Restricted Project
Summary
  • Move fallback functions for strtol_l() and strtoul_l() to __strtonum_fallback.h.
  • Add strtoul_l() for older Android.
  • Windows provides _strtol_l and _strtoul_l.

Diff Detail

Event Timeline

brad created this revision.Jun 1 2023, 1:24 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 1:24 AM
brad requested review of this revision.Jun 1 2023, 1:24 AM
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptJun 1 2023, 1:24 AM
philnik requested changes to this revision.Jun 1 2023, 8:01 AM
philnik added a subscriber: philnik.

It looks like we don't actually use strtol_l or strtoul_l anywhere, so we might as well just remove these functions.

This revision now requires changes to proceed.Jun 1 2023, 8:01 AM
Mordante commandeered this revision.Sep 4 2023, 9:45 AM
Mordante added a reviewer: brad.
Mordante added a subscriber: Mordante.

Since this patch is inactive and we're moving to GitHub PR's closing this patch.

Mordante abandoned this revision.Sep 4 2023, 9:45 AM