[Pharo-dev] [pharo-project/pharo-core]

2015-05-20 Thread GitHub
Branch: refs/tags/50053 Home: https://github.com/pharo-project/pharo-core

[Pharo-dev] [pharo-project/pharo-core] 2e7d6b: 50053

2015-05-20 Thread GitHub
Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 2e7d6b253c3729c723cc121b5248973a1372e252 https://github.com/pharo-project/pharo-core/commit/2e7d6b253c3729c723cc121b5248973a1372e252 Author: Jenkins Build Server bo...@pharo-project.org Date:

[Pharo-dev] Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

2015-05-20 Thread Sven Van Caekenberghe
Anyone care to review issue 15528 ? Add TTranscript and DailyNonInteractiveTranscript https://pharo.fogbugz.com/f/cases/15528/Add-TTranscript-and-DailyNonInteractiveTranscript This is pretty straightforward, I added a comment about the warning. Thx, Sven

[Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

2015-05-20 Thread Marcus Denker
Hi, For 50054 I have enabled AST based syntax highlighting for testing. The idea is to have it enabled to find out what is missing, if there are bugs. As soon as we find issues, we can turn it off again. If it works fine for 2 weeks, we can start to clean up Shout and move the Shout parser

Re: [Pharo-dev] Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

2015-05-20 Thread Marcus Denker
DONE. On 20 May 2015, at 09:24, Sven Van Caekenberghe s...@stfx.eu wrote: Anyone care to review issue 15528 ? Add TTranscript and DailyNonInteractiveTranscript https://pharo.fogbugz.com/f/cases/15528/Add-TTranscript-and-DailyNonInteractiveTranscript This is pretty straightforward, I

Re: [Pharo-dev] Critics Browser should diff formatted original

2015-05-20 Thread Sean P. DeNigris
Sean P. DeNigris wrote How about we diff the original source /re-formatted/ against the transformed source? Issue 15574: Critics: Ignore whitespace changes when diffing transformations https://pharo.fogbugz.com/default.asp?15574 - Cheers, Sean -- View this message in context:

Re: [Pharo-dev] [pharo-project/pharo-core] 2e7d6b: 50053

2015-05-20 Thread Sean P. DeNigris
GitHub wrote 15523 Code Cruft Rule Only Matches One-Liners https://pharo.fogbugz.com/f/cases/15523 FYI since there were situations where the lint rule should have matched but did not, new violations may appear which are actually not new but just being reported for the first time. -

Re: [Pharo-dev] Anyone care to review issue 15528, Add TTranscript and DailyNonInteractiveTranscript ?

2015-05-20 Thread Sven Van Caekenberghe
Thx! On 20 May 2015, at 11:44, Marcus Denker marcus.den...@inria.fr wrote: DONE. On 20 May 2015, at 09:24, Sven Van Caekenberghe s...@stfx.eu wrote: Anyone care to review issue 15528 ? Add TTranscript and DailyNonInteractiveTranscript

Re: [Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

2015-05-20 Thread Marcus Denker
On 20 May 2015, at 09:35, Marcus Denker marcus.den...@inria.fr wrote: Hi, For 50054 I have enabled AST based syntax highlighting for testing. The idea is to have it enabled to find out what is missing, if there are bugs. As soon as we find issues, we can turn it off again. If it

Re: [Pharo-dev] New Collections for Pharo

2015-05-20 Thread Sergio Fedi
Nice work! I frequently needed the MultiValueDictionary and had it implemented several times in the past. ​

[Pharo-dev] [pharo-project/pharo-core] 1694da: 50054

2015-05-20 Thread GitHub
Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 1694da0d0359039e191b1d685fbaee530c230c77 https://github.com/pharo-project/pharo-core/commit/1694da0d0359039e191b1d685fbaee530c230c77 Author: Jenkins Build Server bo...@pharo-project.org Date:

[Pharo-dev] [pharo-project/pharo-core]

2015-05-20 Thread GitHub
Branch: refs/tags/50054 Home: https://github.com/pharo-project/pharo-core

[Pharo-dev] [pharo-project/pharo-core] cf919f: 50055

2015-05-20 Thread GitHub
Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: cf919fc630785f19768ebcbb5b41ef80c1f3526b https://github.com/pharo-project/pharo-core/commit/cf919fc630785f19768ebcbb5b41ef80c1f3526b Author: Jenkins Build Server bo...@pharo-project.org Date:

[Pharo-dev] FileSystem not deleting symlinks

2015-05-20 Thread Damien Cassou
The FileSystem library uses DeleteVisitor and PostorderGuide (depth-first) to delete a directory and all its children. All its children? No. Symlinks are protected from deletion: DeleteVisitorvisit: aReference PostorderGuide show: aReference to: self selecting: [ :entry | entry

[Pharo-dev] [pharo-project/pharo-core]

2015-05-20 Thread GitHub
Branch: refs/tags/50055 Home: https://github.com/pharo-project/pharo-core

Re: [Pharo-dev] [Pharo-users] MongoTalk: new place on smalltalkhub

2015-05-20 Thread Norbert Hartl
Am 20.05.2015 um 18:06 schrieb Henrik Johansen henrik.s.johan...@veloxit.no: On 19 May 2015, at 4:52 , Norbert Hartl norb...@hartl.name mailto:norb...@hartl.name wrote: Henry, Am 19.05.2015 um 13:06 schrieb Henrik Johansen henrik.s.johan...@veloxit.no

Re: [Pharo-dev] [Pharo-users] MongoTalk: new place on smalltalkhub

2015-05-20 Thread Henrik Johansen
On 19 May 2015, at 4:52 , Norbert Hartl norb...@hartl.name wrote: Henry, Am 19.05.2015 um 13:06 schrieb Henrik Johansen henrik.s.johan...@veloxit.no mailto:henrik.s.johan...@veloxit.no: Hi! I made some (imho) improvements to the OID implementation, if someone with write access

Re: [Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

2015-05-20 Thread Sean P. DeNigris
Marcus Denker-4 wrote AST based syntax highlighting This is raaally cool :) - Cheers, Sean -- View this message in context: http://forum.world.st/Pharo-5-For-testing-enabled-AST-based-syntax-higlighting-tp4827558p4827682.html Sent from the Pharo Smalltalk Developers mailing list

Re: [Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread Sergio Fedi
Worked for me too.​

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Chris Cunningham
On Wed, May 20, 2015 at 12:55 PM, Aliaksei Syrel alex.sy...@gmail.com wrote: Check for the nil value should be banned in my opinion. There are well-known techniques to not have to use it (e.g., Null-object pattern). Absolutely agree! :) My only fear is that it could encourage the use of

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Aliaksei Syrel
On Wed, May 20, 2015 at 10:14 PM, Chris Cunningham cunningham...@gmail.com wrote: would generate a new OrderedCollection every time it is called!. Even more overkill. Last line from previous email: Blocks are of course better, because they are lazy and don't get executed if value is not nil

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Aliaksei Syrel
Yes, let's not introduce it :) Not a smalltalk way! Cheers, Alex On Wed, May 20, 2015 at 10:27 PM, Aliaksei Syrel alex.sy...@gmail.com wrote: On Wed, May 20, 2015 at 10:14 PM, Chris Cunningham cunningham...@gmail.com wrote: would generate a new OrderedCollection every time it is called!.

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Aliaksei Syrel
Check for the nil value should be banned in my opinion. There are well-known techniques to not have to use it (e.g., Null-object pattern). Absolutely agree! :) My only fear is that it could encourage the use of nil instead of proper Null Objects Sometimes it's impossible to always have

[Pharo-dev] MongoTalk UpdateOperation and options. How to?

2015-05-20 Thread p...@highoctane.be
In MongoTalk, there is UpdateOperation. But I can't find how one can do an update with options. http://docs.mongodb.org/manual/reference/method/db.collection.update/ I see Mongoupdate: origDictionary with: newDictionary collection: aString (UpdateOperation collection: aString

Re: [Pharo-dev] FileSystem not deleting symlinks

2015-05-20 Thread Sean P. DeNigris
Damien Pollet wrote Perhaps a risk of deleting the symlink target instead of the link itself? IIRC before the VM changes to distinguish symlinks, that was exactly the dangerous situation. If you had a symlink to your home directory in a folder which you deleted from Pharo... bye bye :/ -

Re: [Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

2015-05-20 Thread Sergio Fedi
So cool.​

Re: [Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread Max Leske
On 20 May 2015, at 20:46, stepharo steph...@free.fr wrote: Hi one student of saint-louis tried to download pharo from the pharo web site and on windows the archive seems broken. Could someone try and let me know? Works perfectly on Win 8 (Virtualbox). The only thing I can think of is

Re: [Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread Nicolai Hess
2015-05-20 20:46 GMT+02:00 stepharo steph...@free.fr: Hi one student of saint-louis tried to download pharo from the pharo web site and on windows the archive seems broken. Could someone try and let me know? works for me: http://files.pharo.org/platform/Pharo4.0-win.zip Tx. Stef

[Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Aliaksei Syrel
Hi, Null coalesce operator - returns the first not nil operand. http://en.wikipedia.org/wiki/Null_coalescing_operator Adding one simple method to an Object: ?? anObject ^ self and to UndefinedObject: ?? anObject ^ anObject could allow us to write: default1 := nil. default2 := nil.

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Sven Van Caekenberghe
Yes, #ifNil: is almost the same (except the #value): |default1 default2 default3 default4 | default1 := nil. default2 := nil. default3 := 'foo'. default4 := 'don''t care'. default1 ifNil: [ default2 ifNil: [ default3 ifNil: [ default4 ] ] ] = 'foo' On 20 May 2015, at 21:38, Damien Pollet

[Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread stepharo
Hi one student of saint-louis tried to download pharo from the pharo web site and on windows the archive seems broken. Could someone try and let me know? Tx. Stef

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Sergio Fedi
In my experience those situations end up been handled by collections, detect:ifNone: which even handles the case when all defaults are nil. But maybe that idiom is very common in another situations.

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Alexandre Bergel
Hi! I perfectly understand the purpose of this method ??” Although I do not think this will seriously impact the quality of the code being produced, I still believe it will not help making code of a better quality. Check for the nil value should be banned in my opinion. There are well-known

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Damien Pollet
…ifNone: is a nice idiom, but not always available (in which case, we should think about introducing it) ?? is basically an infix but eager version of #ifNil:, IMHO a pragmatic thing to have; could be quite handy. My only fear is that it could encourage the use of nil instead of proper Null

Re: [Pharo-dev] FileSystem not deleting symlinks

2015-05-20 Thread Damien Pollet
Perhaps a risk of deleting the symlink target instead of the link itself? On 20 May 2015 at 15:46, Damien Cassou damien.cas...@inria.fr wrote: The FileSystem library uses DeleteVisitor and PostorderGuide (depth-first) to delete a directory and all its children. All its children? No. Symlinks

Re: [Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread stepharo
tx you all! S Le 20/5/15 19:54, Max Leske a écrit : On 20 May 2015, at 20:46, stepharo steph...@free.fr wrote: Hi one student of saint-louis tried to download pharo from the pharo web site and on windows the archive seems broken. Could someone try and let me know? Works perfectly on Win 8

Re: [Pharo-dev] New Collections for Pharo

2015-05-20 Thread Hernán Morales Durand
Hello Alexander, For the MultiValueDictionary IMHO you can need to use other Collections beside OrderedCollection, so you can replace #asOrderedCollection with #asSortedCollection, etc adding an inst. var. storing the converting selector could result in a more flexible Collection. I know at

Re: [Pharo-dev] New Collections for Pharo

2015-05-20 Thread Sergio Fedi
I agree with Hernan. In the MultiValueDictionary sometimes it's necessary to parametrize the collection the value has. On Wed, May 20, 2015 at 8:30 PM, Hernán Morales Durand hernan.mora...@gmail.com wrote: Hello Alexander, For the MultiValueDictionary IMHO you can need to use other

Re: [Pharo-dev] Critics Browser should diff formatted original

2015-05-20 Thread Sean P. DeNigris
Sean P. DeNigris wrote Issue 15574: Critics: Ignore whitespace changes when diffing transformations https://pharo.fogbugz.com/default.asp?15574 Thanks to Yuriy, now the confusing diff on the left will look like the one on the right

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Eliot Miranda
Hi Aliaksei, On Wed, May 20, 2015 at 12:55 PM, Aliaksei Syrel alex.sy...@gmail.com wrote: Check for the nil value should be banned in my opinion. There are well-known techniques to not have to use it (e.g., Null-object pattern). Absolutely agree! :) My only fear is that it could encourage