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

bugfix Sweep #3

Merged
merged 42 commits into from Oct 7, 2015
Merged

bugfix Sweep #3

merged 42 commits into from Oct 7, 2015

Conversation

@marianopeck
Copy link

I would like to have GsDevKit#72 fixed, but you said another kind of better solution may be needed. The one proposed there DOES work for me.
GsDevKit#70 and GsDevKit#71 and enchacements and those are not easy to test. I have been using both in production since some time already and no problems.
#20 should be integrated I think. Problem was with uppercase symbols and with the need of extended parser. A simple failure test that gets fixed with those changes is as simple as self assert: '7.3470570000000004E+02' asNumber printString = '7.3470569999999998E+02'
GsDevKit#58 would be nice, but you said you should check in other Gemstones...

Those are all I've remember myself.

@dalehenrich
Copy link
Member Author

Take a look at GsDevKit#73 and consider for inclusion

@dalehenrich
Copy link
Member Author

@marianopeck ... excellent ... just what I was looking for!

@dalehenrich
Copy link
Member Author

add Issue GsDevKit#74: class migration problem

@jbrichau
Copy link
Member

I'm nominating GsDevKit#29 : include Grease 1.1.? into GLASS1

@IwanVosloo
Copy link

Pull request created for GsDevKit#75 at: GsDevKit#78

Iwan Vosloo and others added 12 commits September 15, 2015 18:31
…Parser were a concrete class, using a defaultParserClass (fixes GsDevKit#75)
Fix for issue GsDevKit#75 ... perfect the tests all passed this time as well ...
…e used as if NumberParser were a concrete class, using a defaultParserClass (fixes GsDevKit#75) from Ilwan ... update the monticello meta data ...
…eak.v32 and Squeak.v33 packages from Squeakv.3
…nitialization of DefaultParserClass class var in NumberParser ... just answer default class
…ak.v32 and Squeak.v3 packages from Squeak.v33
…low MCPackageLoader>>equalDefinition:to: to make a last second comparison of the new defintion against the actual state of the image when loading ... and we can catch the version history superclass difference and schedule the definition for loading
@dalehenrich dalehenrich mentioned this pull request Sep 16, 2015
…ck, ObsoleteMetaclass. These classes finally deleted in GemStone 3.3.
…ry class>>objectLogCleanupExampleScript and ObjectLogEntry class>>objectLogEntries:
…the default number parser), so that when ExtendedNumberParser is chosen, it gets used everywhere ... change the list of exponents to include cap versions
…mberParser to accomodate the extended ExtendecNumberParser tests ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants