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

implement #basicNextPutAll: on pseudo streams #736

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 3 comments
Closed

implement #basicNextPutAll: on pseudo streams #736

GoogleCodeExporter opened this issue Mar 25, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

Our encoders are supposed to be polymorphic with streams. In important place 
where this is used is for encodings. There is no clear published protocol that 
is expected from streams but the following messages are sent #isBinary 
#basicNextPut: #basicNextPutAll:. We implement some of them but not all in all 
the places. This causes compatibility problems with Squeak but also opens the 
possibility for latent bugs.

Original issue reported on code.google.com by philippe...@gmail.com on 13 Aug 2012 at 7:31

@GoogleCodeExporter
Copy link
Author

Original comment by philippe...@gmail.com on 13 Aug 2012 at 7:31

  • Changed state: Started
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-Pharo-Core-pmm.22
Author: pmm
Time: 14 August 2012, 8:07:29 pm
UUID: 44ff7f76-043c-4660-9b8d-ae5a3d976398
Ancestors: Seaside-Pharo-Core-pmm.21

- Issue 736:    implement #basicNextPutAll: on pseudo streams
- http://code.google.com/p/seaside/issues/detail?id=736

Name: Seaside-Tests-Core-pmm.259
Author: pmm
Time: 14 August 2012, 8:07:59 pm
UUID: 9625f7a8-7ef5-4bc2-8739-f0a8ae112c84
Ancestors: Seaside-Tests-Core-pmm.258

- Issue 736:    implement #basicNextPutAll: on pseudo streams
- http://code.google.com/p/seaside/issues/detail?id=736

Name: Seaside-Tests-Pharo-Core-pmm.29
Author: pmm
Time: 14 August 2012, 8:08:26 pm
UUID: d4f667be-b8e8-46fe-9063-1950515c0d50
Ancestors: Seaside-Tests-Pharo-Core-pmm.28

- Issue 736:    implement #basicNextPutAll: on pseudo streams
- http://code.google.com/p/seaside/issues/detail?id=736

Original comment by philippe...@gmail.com on 14 Aug 2012 at 6:08

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Original comment by philippe...@gmail.com on 14 Aug 2012 at 6:08

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

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

No branches or pull requests

1 participant