|
From: Tsz S. C. <jas...@gm...> - 2018-10-29 22:31:41
|
Hi Chris, John, Colin, and all, The pull request is submitted for review: https://github.com/quickfix-j/quickfixj/pull/212 Cheers, Jason On Mon, Oct 29, 2018 at 3:04 PM Christoph John <chr...@ma...> wrote: > Hi Jason, > > yes, please do. > > Thanks, > Chris. > > > On 29/10/2018 15:55, Tsz Shun Chow wrote: > > Hi Chris, > > It is fully backward compatible and it is completely optional. In fact, > they are all new classes only. > > I could raise a pull request soon for this. > > Regards > Jason > > On Mon, 29 Oct 2018 at 14:52, Christoph John <chr...@ma...> > wrote: > >> Hi Jason, >> >> I think it would be sufficient to open a pull request only. >> Does your change break anything? >> >> Thanks in advance and best regards, >> Chris. >> >> On 29/10/2018 13:02, Tsz Shun Chow wrote: >> >> QuickFIX/J Documentation: http://www.quickfixj.org/documentation/ >> QuickFIX/J Support: http://www.quickfixj.org/support/ >> >> >> Hi all, >> >> I have recently developed an implementation of quickfix.Application that >> transforms the usage to be functional like. E.g. registering single method >> interface for operation like onLogon(), and it makes the usage with lambda >> possible. >> >> It also supports FIFO handling for multiple registered callbacks and fail >> fast exception propagation. Also optionally support the ApplicationExtended >> interface. >> >> Would anyone be interested in having this included in the standard >> quickfix library under the core module? >> >> If it’s of interest to people, I reckon I could submit a feature ticket >> in JIRA and submit a pull request after. >> >> Regards, >> Jason >> >> >> _______________________________________________ >> Quickfixj-users mailing lis...@li...://lists.sourceforge.net/lists/listinfo/quickfixj-users >> >> >> -- >> Christoph John >> Software Engineering >> T +49 241 557...@ma... >> >> MACD GmbHOppenhoffallee 103 >> 52066 Aachen, Germany <https://maps.google.com/?q=Oppenhoffallee+103%0D%0A52066+Aachen,+Germany&entry=gmail&source=g>www.macd.com >> >> Amtsgericht Aachen: HRB 8151 >> Ust.-Id: DE 813021663 >> Geschäftsführer: George Macdonald >> >> > -- > Christoph John > Software Engineering > T +49 241 557...@ma... > > MACD GmbH > Oppenhoffallee 103 > 52066 Aachen, Germanywww.macd.com > > Amtsgericht Aachen: HRB 8151 > Ust.-Id: DE 813021663 > Geschäftsführer: George Macdonald > > |