No problem.
Stef
On Jun 10, 2008, at 9:51 PM, Oscar Nierstrasz wrote:
Very strange.
I updated earlier today and did not get Integer>>hex. Now it is
there.
Of course your implementation is better since it reuses existing
functionality.
Gremlins are definitely at work here ;-)
Sorry for the confusion, though I really do not understand why my
image stayed out of sync and now it is ok.
- on
On Jun 10, 2008, at 9:35 PM, Oscar Nierstrasz wrote:
Wow.
I took pharo0.1-10032-7075dev08.06.1 and updated it. There is no
Integer>>hex there.
Damien, can you help? It looks like the pharo-dev image is somehow
out of sync.
I will try both again now.
- on
On Jun 10, 2008, at 9:21 PM, Stéphane Ducasse wrote:
Hi oscar
I took the pharo image Pharo0.1-10028-2008-05-24 available on the
pharo web site
and did an update.
and there is
Integer>>hex
^ self printStringBase: 16
I checked the version and it is from 25 of may
Stef
Hi Stef,
I loaded a fresh pharo-dev image and updated everything. Then
the testHex tests failed, so I fixed them.
Can you please tell me which implementors of hex you find in your
image?
I only find Character>>hex and Float>>hex. I had to implement
Integer>>hex myself.
- on
On Jun 10, 2008, at 6:38 PM, Stéphane Ducasse wrote:
testHex
self assert: $a hex '61' = '61'.
self assert: Character space hex = '20'
is green on my image
as well as this one
testHex
self assert: 0 hex = '0'.
self assert: 12 hex = 'C'.
self assert: 1234 hex = '4D2'.
They are red in yours?
Stef
On Jun 10, 2008, at 4:00 PM, Oscar Nierstrasz wrote:
Well, I thought we would start to fix the existing broken tests
before adding new broken tests.
I would also be very happy to see some comments when someone
adds a new broken test, so I know what the plan is.
Now I added an implementation of Integer>>hex, for example, but
I am not so sure why we need this.
Ain't too many senders. An easier fix would have been to remove
the broken test.
Cheers,
- on
On Jun 10, 2008, at 15:53, Bill Schwab wrote:
Is your concern that the new tests are broken (which seems
normal to me
that they would start out that way), or are you objecting to
their being
included in the image?
I can see arguments either way on the latter. If code in the
image is
wrong and a clearly correct but currently failing test
highlights it,
I'd rather be alerted to it. You are also correct that having
them
included draws attention that could easily encourage
duplication of
effort.
Comments?
Bill
Wilhelm K. Schwab, Ph.D.
University of Florida
Department of Anesthesiology
PO Box 100254
Gainesville, FL 32610-0254
Email: [EMAIL PROTECTED]
Tel: (352) 846-1285
FAX: (352) 392-7029
[EMAIL PROTECTED] 06/10/08 8:31 AM >>>
I looked at the wiki page
http://code.google.com/p/pharo/wiki/CurrentlyWorkingOn
but it does not say much.
I would like to work through some tests, but I cannot find an
overview.
The tests CharacterTest>>testHex and IntegerTest>>testHex are
broken
but seem to be very new. Stef, are you working on these? You
seem to
have written the tests ...
- on
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-
project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
