HomePhabricator

Fixed runtime error on older gcc/g++ (<4.9) where std::reg_ex is non-standards…
79943feda175Unpublished

Unpublished Commit · Learn More

Repository Importing: This repository is still importing.

Description

Fixed runtime error on older gcc/g++ (<4.9) where std::reg_ex is non-standards compliant and produced a regex compile fail. This new simpler regex does not trip up even broken libstdc++ std::regex implementations.

Summary: Fixes phabricator issue T56

Test Plan: make check on Ubuntu 14.04 / 14.05 (Travis equivalent)

Reviewers: #bitcoin_abc, deadalnix, freetrader, sickpig, kyuupichan, awemany, O1 Bitcoin ABC

Reviewed By: #bitcoin_abc, deadalnix, O1 Bitcoin ABC

Differential Revision: https://reviews.bitcoinabc.org/D290

Details

Provenance
CCulianuAuthored on Jul 2 2017, 10:16
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Reviewer
Restricted Project
Differential Revision
D290: Fixed runtime error on older gcc/g++ (<4.9) where std::reg_ex is non-standards compliant and produced a regex compile fail. This new simpler regex does not trip up even broken libstdc++ std::regex implementations.
Parents
rSTAGINGe0ec90e662e6: Align UAHF spec with upstream Bitcoin-UAHF
Branches
Unknown
Tags
Unknown