Skip to content

Add the equivalent of concatMap - #20

Open
denizcoskun wants to merge 1 commit into
CombineCommunity:masterfrom
denizcoskun:concatMap
Open

Add the equivalent of concatMap#20
denizcoskun wants to merge 1 commit into
CombineCommunity:masterfrom
denizcoskun:concatMap

Conversation

@denizcoskun

Copy link
Copy Markdown

concatMap is a quite useful operator in the Rx world and it was a bit disappointing that Combine didn't have a built-in equivalent of it until I found that flatMap behaves like concatMap when used correctly.

@freak4pc

freak4pc commented Aug 8, 2020

Copy link
Copy Markdown
Member

I'm not sure that's entirely true, there was a huge discussion about this in one of the slack channels and some folks mentioned it has some differences. I'll try to find that info.

@denizcoskun

Copy link
Copy Markdown
Author

@freak4pc thanks, really appreciate that!

@denizcoskun

Copy link
Copy Markdown
Author

I'm not sure that's entirely true, there was a huge discussion about this in one of the slack channels and some folks mentioned it has some differences. I'll try to find that info.

here's where I got the information: https://forums.swift.org/t/combine-what-are-those-multicast-functions-for/26677/26

@freak4pc

Copy link
Copy Markdown
Member

I couldn't find anything against this claim, if you rebase this I'll merge it . Thanks!

@denizcoskun

denizcoskun commented Nov 29, 2020

Copy link
Copy Markdown
Author

I couldn't find anything against this claim, if you rebase this I'll merge it . Thanks!

@freak4pc thanks, it is rebased now.

@freak4pc

freak4pc commented Dec 1, 2020

Copy link
Copy Markdown
Member

Hey @denizcoskun -

There was an interesting research and discussion in CombineExt which seems to indicate this variation isn't quite similar to concatMap in all forms.

What do you think?

CombineCommunity/CombineExt#68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants