Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PersistentVector, transients, polishing #7

Open
wants to merge 295 commits into
base: master
Choose a base branch
from

Conversation

tomaskulich
Copy link

Not sure, if you want to merge this.. we did several modifications to the codebase, which you may not like

  • we removed some tests
  • we also removed some Haskell/Scala stuff (such as Option; we rather provide orElse named parameter)
  • given that m is PersistentMap and m[k] is not defined, m[k] throws instead of just returning null. We know, that this is different from Dart standards but we hate the 'null policy' more than we love keeping conventions.

Roman Hudec and others added 30 commits August 29, 2014 14:43
…transientMap

Conflicts:
	lib/src/map_impl.dart
Conflicts (deleted):
	benchmark/map_bench_wordcount.dart
	benchmark/src/benchmark.dart
	benchmark/src/simple_map_1.dart
	benchmark/src/simple_map_2.dart
@tosh
Copy link

tosh commented Feb 21, 2015

@tomaskulich ping

@tomaskulich
Copy link
Author

sorry guys, wasn't following this discussion for some time.

@polux : I'll ask the contributors to sign the CLA; I'll let you know, when this happens. Also, I created new PR with the newest changes; you can close this one.

@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@polux
Copy link
Owner

polux commented Feb 23, 2015

@tomaskulich, awesome! Thanks!

@polux polux mentioned this pull request Feb 23, 2015
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@tomaskulich tomaskulich force-pushed the master branch 3 times, most recently from 391facb to 1447048 Compare May 20, 2015 13:28
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.

9 participants