- Pharo-dev - lists.pharo.org

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50331 Home: https://github.com/pharo-project/pharo-core

Sept. 18, 2015

[pharo-project/pharo-core] 9c0520: 50331

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 9c0520696d637781a4d225f3b83f862488b36f6b https://github.com/pharo-project/pharo-core/commit/9c0520696d637781a4d225f3… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-09-18 (Fri, 18 Sep 2015) Changed paths: R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50330.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50331.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50330.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50331.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M System-Changes.package/ChangeSet.class/class/defaults/promptForDefaultChangeSetDirectoryIfNecessary.st M System-Changes.package/ManifestSystemChanges.class/class/meta-data - dependency analyser/manuallyResolvedDependencies.st R System-Changes.package/extension/PositionableStream/instance/nextChunkText.st A Text-Core.package/extension/PositionableStream/instance/nextChunkText.st Log Message: ----------- 50331 16594 System-Changes should not depend on Text-Core https://pharo.fogbugz.com/f/cases/16594 http://files.pharo.org/image/50/50331.zip

Sept. 18, 2015

Re: [Pharo-dev] EncodedCharSets

by Nicolas Cellier

Yes, but first step consists in reading http://www.ipa.go.jp/files/000005751.pdf to exactly understand which feature you're going to loose, or better, how you're going to support it differently. Nicolas 2015-09-18 10:10 GMT+02:00 Christophe Demarey <Christophe.Demarey(a)inria.fr>: > Thanks all for your explanations. > It looks like there are still a lot of things to clean. > > Christophe > > Le 17 sept. 2015 Ã 19:26, Eliot Miranda a Ã©crit : > > Hi Christophe, > > On Thu, Sep 17, 2015 at 7:09 AM, Christophe Demarey < > Christophe.Demarey(a)inria.fr> wrote: > >> Hi again, >> >> Does anyone know the rationale behind this? >> >> declareEncodedCharSet: anEncodedCharSetOrLanguageEnvironmentClass >> atIndex: aNumber >> >> EncodedCharSets at: aNumber put: >> anEncodedCharSetOrLanguageEnvironmentClass >> >> "this method is used to modularize the old initialize method: >> EncodedCharSets at: 0+1 put: Unicode. >> EncodedCharSets at: 1+1 put: JISX0208. >> EncodedCharSets at: 2+1 put: GB2312. >> EncodedCharSets at: 3+1 put: KSX1001. >> EncodedCharSets at: 4+1 put: JISX0208. >> EncodedCharSets at: 5+1 put: JapaneseEnvironment. >> EncodedCharSets at: 6+1 put: SimplifiedChineseEnvironment. >> EncodedCharSets at: 7+1 put: KoreanEnvironment. >> EncodedCharSets at: 8+1 put: GB2312. >> EncodedCharSets at: 12+1 put: KSX1001. >> EncodedCharSets at: 13+1 put: GreekEnvironment. >> EncodedCharSets at: 14+1 put: Latin2Environment. >> EncodedCharSets at: 15+1 put: RussianEnvironment. >> EncodedCharSets at: 17+1 put: Latin9Environment. >> EncodedCharSets at: 256 put: Unicode. >> >> " >> > > what Henrik says is correct. Here's the relevant definition in Character: > > Character>>leadingChar > "Answer the value of the 8 highest bits which is used to identify the > language. > This is mostly used for east asian languages CJKV as a workaround against > unicode han-unification." > ^ self asInteger bitShift: -22 > > > i.e. the top 8 bytes of the leading character in a string is (was?) used > to index EncodedCharSets to determine what language the string is in. > > _,,,^..^,,,_ > best, Eliot > > >

Sept. 18, 2015

Re: [Pharo-dev] [review needed] speed up RBClassNotReferencedRule

by Marcus Denker

>>> >> >> What I did is to implement #isUsed like this in e.g the hierarchy of RBLintRule: >> >> isUsed >> "all my sublasses are used" >> ^self name ~= âRBLintRule' >> >> this way if RBLintRule is not used, it is seen by the Critique, but all subclasses >> are used by default. >> >> I implemented that in TestCase, too, and removed the explicit check for TestCase >> subclasses from the rule. > > This is a nice thing. In the same way we can deal with abstract classes. So if someone develops an abstract class with an intent to use it, he can specify that itâs ok to use the class. > True. > Also is there any work on pragmas for classes, at some point Iâve heard something about it but canât recall. > Yes, the question is where to put it syntactically and how to save it in monticelloâ¦ could be interesting to have. > I will try to hack a bit this weekend and see if I can add some kind of automation to marking a class as used. I.e. if you see a critic, you should be able to mark the class as being used with 1-2 clicks. If you are developing some king of facade, you should know that you will not reference it from your library. > ok. Marcus

Sept. 18, 2015

Re: [Pharo-dev] Bitmap VS WordArray in System-Hashing

by Max Leske

> On 18 Sep 2015, at 13:51, Christophe Demarey <christophe.demarey(a)inria.fr> wrote: > > Hi, > > With Max M., we try to kill the dependency between System-Hashing to Graphics - Display Objects. > The dependency is there because SHA1 and SHA256 use the Bitmap class to store raw data. > We came to replace the use of Bitmap by a WordArray. Tests are still working. > > Could someone tell us if this fix is right or if we miss something? > > Thanks, > Max and Christophe Looks good to me. Also: the âtotalsâ variable (which is the bitmap) is never used in SHA-256 (only written once and then assigned nil later). Cheers, Max

Sept. 18, 2015

Bitmap VS WordArray in System-Hashing

by Christophe Demarey

Hi, With Max M., we try to kill the dependency between System-Hashing to Graphics - Display Objects. The dependency is there because SHA1 and SHA256 use the Bitmap class to store raw data. We came to replace the use of Bitmap by a WordArray. Tests are still working. Could someone tell us if this fix is right or if we miss something? Thanks, Max and Christophe

Sept. 18, 2015

Re: [Pharo-dev] ANSICompatibility package

by Jan Vrany

> > I would rename it to SmalltalkCompatibility to ANSICompatibility > > because > > "Compatibility" is a too general term and could mean backward > > compatibility > > of Pharo versions or compatibility to other languages as well. > > That was me :) I was thinking more broadly that over time the > "repository" might hold different packages Compatibility-ANSI, > Compatibility-VW, Compatibility-Dolphin. But maybe that is > overreaching and its better to be more focussed. Let me know what you > think on this and restate what you think the repository name should > be. Ben, could you please give me a commit access? I intend to move Compatibility-SmalltalkX and Compatibility-ANSI (which I created and "maintain" for my own need). Maybe some others may find them useful or maybe even contribute. Cheers, Jan > > > Also see http://wiki.squeak.org/squeak/2384 and > > http://map.squeak.org/package/d7011055-4446-4b8e-b6da-e7e32d3389ec > > which could be a start. > > I saw those, but they interact with existing classes, so I thought it > easier to kickstart by loading in the tests first, which are all new > classes. I only created the repository since the tests were *hard* > to > find eventually at > http://www.spacmunke.org/svn/jst/trunk/src/test/ansi/ansi-compatibili > ty/ > So I thought it would be good make them more accessible as a > Monticello Package. > > cheers -ben > > P.S. I don't quite follow "to SmalltalkCompatibility to > ANSICompatibility" > do you mean "to SmalltalkCompatibility or ANSICompatibility" ? > > > > > > Gesendet: Dienstag, 08. September 2015 um 22:53 Uhr > > > Von: stepharo <stepharo(a)free.fr> > > > An: "Pharo Development List" <pharo-dev(a)lists.pharo.org> > > > Betreff: Re: [Pharo-dev] ANSICompatibility package > > > > > > We should have a SmalltalkCompatibility (not Ansi per se) package > > > were we can add some methods to help migration. > > > > > > Stef > > > > > > > > > Le 8/9/15 06:04, Ben Coman a Ã©crit : > > > > On Tue, Sep 8, 2015 at 9:21 AM, Jan Vrany < > > > > jan.vrany(a)fit.cvut.cz> wrote: > > > > > Hi, > > > > > > > > > > is there actually such a thing like ANSICompatibility > > > > > package? > > > > > (it has been mentioned here lately, but I cannot find it) > > > > > > > > > > Thanks, Jan > > > > > > > > > I made it up for the purpose of discussion to see what interest > > > > it generated. > > > > Should we start one ? > > > > cheers -ben > > > > > > > > > > > > > > > > > > > > >

Sept. 18, 2015

Pharo 5 and method extensions

by roberto.minelli＠usi.ch

Dear all, I just moved to Pharo 5 for development and discovered that it does not correctly show method extensions. I opened a case https://pharo.fogbugz.com/f/cases/16601/Pharo-5-does-not-show-method-extens… Cheers, R

Sept. 18, 2015

Re: [Pharo-dev] Configuration Madness

by roberto.minelli＠usi.ch

Thanks a lot Dale, this works indeed. P.s. Sorry for the delay in the answer but my mailbox went crazy! R > On 16 Sep 2015, at 17:43, Dale Henrichs <dale.henrichs(a)gemtalksystems.com> wrote: > > > > On 9/16/15 8:07 AM, Max Leske wrote: >>> On 16 Sep 2015, at 16:39, roberto.minelli(a)usi.ch wrote: >>> >>> Dear all, >>> >>> I have a question about the ConfigurationOf my project, and I really hope you can help. >>> >>> In my project I have a package called âDevFlow-Extensionsâ that is a âplaceholderâ for code that is dependent on the Pharo version. >>> >>> On the ConfigurationOf, according to the Pharo version I do something like >>> >>> spec for: #'pharo3.x' >>> do: [ >>> >>> spec blessing: #baseline. >>> spec repository: 'http://smalltalkhub.com/mc/RobertoMinelli/DevFlow/main'. >>> >>> spec package: 'DevFlow-Extensions' with: 'DevFlow-Extensions30'. >>> ]. >>> >>> spec for: #'pharo4.x' >>> do: [ >>> >>> spec blessing: #baseline. >>> spec repository: 'http://smalltalkhub.com/mc/RobertoMinelli/DevFlow/main'. >>> >>> spec package: 'DevFlow-Extensions' with: 'DevFlow-Extensions40'. >>> ]. >>> >>> In this way, in Pharo 3.0 Iâd load DevFlow-Extensions30 and in Pharo 4.0 Iâd load DevFlow-Extensions40. Super cool. >>> I am using a single package (âDevFlow-Extensionsâ) to specify of groups, for example: >>> >>> spec group: 'User' with: #( >>> 'DevFlow-Meta' >>> ... >>> 'DevFlow-Announcement' >>> 'DevFlow-Extensions'). >>> >>> Now the problem is that for Pharo 5 I have to load both 'DevFlow-Extensions40â and 'DevFlow-Extensions50â. >>> I read the chapter on Metacello, but didnât find a solution. I tried some combinations of #includes: and #requires: but with no luck. >>> >>> Any help is appreciated, thank you very much! >>> >>> Roberto >> Shouldnât this work? >> >> spec package: 'DevFlow-Extensions' with: #('DevFlow-Extensions40â âDevFlow-Extensions50). >> >> >> > Good idea, but it's not quite correct, I think this will work: > > spec for: #'pharo5.x' > do: [ > spec package: 'DevFlow-Extensions40' . > spec package: 'DevFlow-Extensions50' . > spec group: 'DevFlow-Extensions' with: #('DevFlow-Extensions40â âDevFlow-Extensions50) ]. > > > Dale >

Sept. 18, 2015

Re: [Pharo-dev] EncodedCharSets

by Christophe Demarey

Thanks all for your explanations. It looks like there are still a lot of things to clean. Christophe Le 17 sept. 2015 Ã 19:26, Eliot Miranda a Ã©crit : > Hi Christophe, > > On Thu, Sep 17, 2015 at 7:09 AM, Christophe Demarey <Christophe.Demarey(a)inria.fr> wrote: > Hi again, > > Does anyone know the rationale behind this? > > declareEncodedCharSet: anEncodedCharSetOrLanguageEnvironmentClass atIndex: aNumber > > EncodedCharSets at: aNumber put: anEncodedCharSetOrLanguageEnvironmentClass > > "this method is used to modularize the old initialize method: > EncodedCharSets at: 0+1 put: Unicode. > EncodedCharSets at: 1+1 put: JISX0208. > EncodedCharSets at: 2+1 put: GB2312. > EncodedCharSets at: 3+1 put: KSX1001. > EncodedCharSets at: 4+1 put: JISX0208. > EncodedCharSets at: 5+1 put: JapaneseEnvironment. > EncodedCharSets at: 6+1 put: SimplifiedChineseEnvironment. > EncodedCharSets at: 7+1 put: KoreanEnvironment. > EncodedCharSets at: 8+1 put: GB2312. > EncodedCharSets at: 12+1 put: KSX1001. > EncodedCharSets at: 13+1 put: GreekEnvironment. > EncodedCharSets at: 14+1 put: Latin2Environment. > EncodedCharSets at: 15+1 put: RussianEnvironment. > EncodedCharSets at: 17+1 put: Latin9Environment. > EncodedCharSets at: 256 put: Unicode. > > " > > what Henrik says is correct. Here's the relevant definition in Character: > > Character>>leadingChar > "Answer the value of the 8 highest bits which is used to identify the language. > This is mostly used for east asian languages CJKV as a workaround against unicode han-unification." > ^ self asInteger bitShift: -22 > > > i.e. the top 8 bytes of the leading character in a string is (was?) used to index EncodedCharSets to determine what language the string is in. > > _,,,^..^,,,_ > best, Eliot

Sept. 18, 2015