Re: [Flex-devel] Pull Request #16

[Will E. <wes...@gm...>]

Thanks for your interest. I've got this in my queue to review.

> I tweaked the build system along the way to ensure built sources could be
> cleaned on demand and to give hints to make's dependency tracking.

How does this effect the automake/autoconf/libtool stuff? In what ways were the features you list missing?

> 
> It's a small change, but I'll build on it.

In general, the best practice is to segregate these sorts of changes. So, the c++ stuff and the build system should be two separate pull requests so that they can be handled independently--given that maybe one needs to be handled before the other, especially if one of your change sets assumes the existence of the other.

Thanks!

--Will


On Saturday,  7 November 2015, 11:43 am -0500, Mighty Jo <mig...@gm...> wrote:

> All,
> 
> I have C++-idiomatic stream handing working in a branch even with
> master@HEAD.  I kept the old interface in addition to the new one - the
> old-style calls just delegate to the new ones.  Had to do something
> unwieldy with the constructors for yyFlexLexer since delegating
> constructors didn't become part of the standard until C++11 and I didn't
> want to add that as a dependency.
> 
> I tweaked the build system along the way to ensure built sources could be
> cleaned on demand and to give hints to make's dependency tracking.
> 
> It's a small change, but I'll build on it.
a> 
> -Joe

> ------------------------------------------------------------------------------

> _______________________________________________
> Flex-devel mailing list
> Fle...@li...
> https://lists.sourceforge.net/lists/listinfo/flex-devel


-- 
Will Estes
Flex Project Maintainer
wes...@gm...
https://github.com/westes/flex