profile
viewpoint

taocpp/PEGTL 1153

Parsing Expression Grammar Template Library

taocpp/operators 150

A highly efficient, move-aware operators library

taocpp/sequences 85

Variadic templates and std::integer_sequence support library

d-frey/lsort 7

Sort almost-sorted files in-place

d-frey/todo 7

Compile-time expiring TODOs

d-frey/pool 3

Shared Object Pool

flonatel/statsd-cpp 1

Implementation of the statsd in C++

issue commenttaocpp/PEGTL

`contrib/integer.hpp` should include `<numeric_limits>`

No problem, thanks for reporting, even if it turned out to be a non-issue :-)

le-migou

comment created time in 7 hours

issue closedtaocpp/PEGTL

`contrib/integer.hpp` should include `<numeric_limits>`

#include <tao/pegtl/contrib/integer.hpp>
int main () {}

Gives

In file included from a.cpp:1:
../../../external/taocpp/pegtl/include/tao/pegtl/contrib/integer.hpp:63:60: error: ‘numeric_limits’ is not a member of ‘std’
   63 |       template< typename Integer, Integer Maximum = ( std::numeric_limits< Integer >::max )() >

closed time in 8 hours

le-migou

issue commenttaocpp/PEGTL

`contrib/integer.hpp` should include `<numeric_limits>`

Sorry, my bad I was really too far behind !

le-migou

comment created time in 8 hours

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

@Bjoe Thanks a lot, greatly appreciated!

d-frey

comment created time in 9 hours

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

@d-frey I created a proposal. See PR #233

d-frey

comment created time in 10 hours

PR opened taocpp/PEGTL

Fix Android CI builds

This is a proposal to fix failing android CI builds. See issue #216

It uses a docker where the android NDK r22 beta 1 is installed to build PEGTL.

See also travis build: https://travis-ci.org/github/Bjoe/PEGTL/jobs/745332637

It needs maybe some "cleanup" regarding to the output.

Maybe some output of the current NDK/SDK version would be useful.

+16 -26

0 comment

1 changed file

pr created time in 10 hours

issue commenttaocpp/PEGTL

`contrib/integer.hpp` should include `<numeric_limits>`

Sorry, I meant <limits> of course.

le-migou

comment created time in 10 hours

issue openedtaocpp/PEGTL

`contrib/integer.hpp` should include `<numeric_limits>`

#include <tao/pegtl/contrib/integer.hpp>
int main () {}

Gives

In file included from a.cpp:1:
../../../external/taocpp/pegtl/include/tao/pegtl/contrib/integer.hpp:63:60: error: ‘numeric_limits’ is not a member of ‘std’
   63 |       template< typename Integer, Integer Maximum = ( std::numeric_limits< Integer >::max )() >

created time in 10 hours

PR closed taocpp/PEGTL

Updated contrib/json so that it is not greedy.

contrib/json no longer consumes the whitespace after the json value.

+13 -7

5 comments

2 changed files

kelvinhammond

pr closed time in a day

pull request commenttaocpp/PEGTL

Updated contrib/json so that it is not greedy.

Your changes fixed my issue. Thank you.

kelvinhammond

comment created time in a day

push eventtaocpp/config

Dr. Colin Hirsch

commit sha e7982cb0b9404fedc12843cd85a6b75ee9e78abe

Allow all JAXN in extensions.

view details

push time in 2 days

push eventtaocpp/config

Dr. Colin Hirsch

commit sha e5a016178091c423d88c41f395ec36b5bf70c687

Small fixes.

view details

push time in 2 days

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

@d-frey Thank you for condolence ... it's three months ago so we are getting back to normal...

I found some more information about the NDK r22 release. So there exists a NDK "22.0.6917172-beta1" version. There are also a changlog for this version.

I used this version locally to build PEGTL for android and it compiles ! :-)

I try now to add this beta version in travis .. see https://travis-ci.org/github/Bjoe/PEGTL/builds/744958133

It will help you to test if PEGTL is compiling for android. It won't help your "customer" as NDK r22 is not yet released.

To introduce -llibstdc++fs in the NDK, I do not recommend this. If you really will have "now" a solution for your "customer", then I purpose to use boost::filesystem. This works 100% on android. I already use boost::filesystem on android. But this means, your project is depended on a third party lib, not sure if you like it.

If recommended following: Use the NDK r22 "Beta" in travis, so you know that PEGTL will compile on android (I'm already on it). Add a hint for the android users that your project can compile/use on android when NDK r22 is released (or if they like use the beta NDK r22). I think it will not take a long time if NDK r22 is released.

d-frey

comment created time in 3 days

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

@ColinH @d-frey As @wravery mentioned. std::filesystem is supported at NDK r22. At the moment NDK r21 is released. So r22 is a future release.

See also NDK issue: android/ndk/issues/1272

NDK r21 is not yet fully support C++17 features.

d-frey

comment created time in 3 days

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

@d-frey Sorry for the late response... some terrible thinks happens on Aug. ... anyways

I'm surprise why the the android toolchain is using libstdc++ ... it is not anymore updated by the NDK. PEGTL should use libc++ instead. Give me some day, I will create a PR to fix it.

d-frey

comment created time in 4 days

pull request commenttaocpp/PEGTL

Auto generate src/test/pegtl/file_äöü𝄞_data.txt

I was also really surprised @ColinH This file system is a (stupid) proprietary NFS that is used on my office network. It is very bad and probably very expensive... I have no control upon it, but obviously since work stations are Linux boxes, we have recent gcc or clang compilers running on them.

delpinux

comment created time in 4 days

pull request commenttaocpp/PEGTL

Auto generate src/test/pegtl/file_äöü𝄞_data.txt

Are there any systems that have a C++17 compiler but don't support Unicode filenames?

delpinux

comment created time in 4 days

issue closedtaocpp/PEGTL

How to parse this minimalist program?

I'm learning PEGs by trying out a basic pascal-ish language. I've got the following grammar:

struct Word: pad<sor<plus<identifier>>, space> { };
struct BasicReal: rep_min_max<1, 1, star<digit>, string<'.'>,plus<digit>> { };
struct Exponent: sor<string<'e'>, string<'E'>> { };
struct OptNegPos: opt<sor<string<'+'>, string<'-'>>> {};
struct RealDigits: rep_min_max<3, 3, digit> { };
struct ScientificForm: rep_min_max<1, 1, sor<rep_min_max<1, 1, Exponent, OptNegPos, RealDigits>>> { };
struct Real: pad<sor<BasicReal, rep_min_max<1, 1, BasicReal, ScientificForm>>, space> { };
struct Special: pad<sor<string<','>, string<'('>, string<')'>, string<'['>, string<']'>, string<';'>, string<'<'>, string<'>'>, string<'*'>, string<'/'>, string<'+'>, string<'-'>, string<'='>>, space> { };
struct Integer: pad<plus<digit>, space>{ };
struct HexInteger: pad<rep_min_max<1, 1, string<'0', 'x'>, plus<xdigit>>, space> { };
struct Grammar: must<opt<sor<bof, bol>>, until<plus<sor<Word, Real, Special, Integer, HexInteger>>>> { };

All my code does right now is tokenize the input, but I keep getting a parse error when I try to evaluate the following code:

program abc
integer a,b,c
real x
begin
    b=6;
    a=b+-4;
    x=-5.63e002;
end
end

It always breaks on the final line, either the "end" or, if I add an extra newline, on that newline. Is there something I'm missing?

closed time in 4 days

ethindp

pull request commenttaocpp/PEGTL

Add table of equivalent forms to Rule Reference

For now we've kept the combinator table, albeit on the "getting started" page, but have removed the other tables since they would need to be greatly extended in order to be really useful, which then creates its own set of issues.

eyepatchParrot

comment created time in 4 days

issue commenttaocpp/PEGTL

How to parse this minimalist program?

Yes, I have. I don't have any further questions at this time. I probably will sometime later though. :)

On 11/19/20, Dr. Colin Hirsch notifications@github.com wrote:

How are things going, have you managed to make some progress? Any further questions or issues that you might want to discuss?

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/taocpp/PEGTL/issues/227#issuecomment-730576767

-- Signed, Ethin D. Probst

ethindp

comment created time in 4 days

issue commenttaocpp/PEGTL

Compiling large grammars

@drtconway The grammar analysis now also supports the ICU rules.

drtconway

comment created time in 4 days

push eventtaocpp/PEGTL

Dr. Colin Hirsch

commit sha 1a28eceb47e794a46164259a6c8f6bc08c5e60c7

Support analyze with ICU rules.

view details

push time in 4 days

issue commenttaocpp/PEGTL

How to parse this minimalist program?

How are things going, have you managed to make some progress? Any further questions or issues that you might want to discuss?

ethindp

comment created time in 4 days

issue commenttaocpp/PEGTL

Compiling large grammars

@kelvinhammond It sounds like what we internally call "switching style" might be useful for you, i.e. changing the States and/or Actions for different parts of the grammar, see https://github.com/taocpp/PEGTL/blob/master/doc/Actions-and-States.md#changing-actions and below.

drtconway

comment created time in 4 days

issue commenttaocpp/PEGTL

Compiling large grammars

@drtconway I'll look into adding grammar analysis integration for the ICU rules.

drtconway

comment created time in 4 days

issue commenttaocpp/PEGTL

Fix or nix Android CI builds

Do you have experience with the NDK? We would greatly appreciate a PR that would fix things again.

d-frey

comment created time in 4 days

pull request commenttaocpp/PEGTL

Updated contrib/json so that it is not greedy.

Coverage Status

Coverage remained the same at 92.432% when pulling cc9742b992df740f16d46d2a2238740afc966b77 on catalogm:json_early into a6716edb39b822dc2ecf7d7b4769cf1611db7dcc on taocpp:master.

kelvinhammond

comment created time in 6 days

pull request commenttaocpp/PEGTL

Updated contrib/json so that it is not greedy.

Tests pass, master fails because of build errors in the json examples. I'm not sure how to fix it. @d-frey can you take a look at the example::control in the json_build.cpp, etc examples?

kelvinhammond

comment created time in 6 days

pull request commenttaocpp/PEGTL

Updated contrib/json so that it is not greedy.

I needed this so that I could parse an expression that takes json value or an identifier followed by one or more whitespaces and then a word but the json value was consuming all of the whitespaces.

kelvinhammond

comment created time in 6 days

PR opened taocpp/PEGTL

Updated contrib/json so that it is not greedy.

contrib/json no longer consumes the whitespace after the json value.

+13 -7

0 comment

2 changed files

pr created time in 6 days

more