September 2015 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] About the Unicode class

by Henrik Johansen

> On 17 Sep 2015, at 2:58 , Christophe Demarey <christophe.demarey(a)inria.fr> wrote: > > Hi, > > Does anyone know why there are so many class variables in Unicode and what they are used for? > Most of them have no sender. > > EncodedCharSet subclass: #Unicode > instanceVariableNames: '' > classVariableNames: 'Cc Cf Cn Co Cs DecimalProperty GeneralCategory Ll Lm Lo Lt Lu Mc Me Mn Nd Nl No Pc Pd Pe Pf Pi Po Ps Sc Sk Sm So ToCasefold ToLower ToUpper Zl Zp Zs' > category: 'Multilingual-Encodings' > > Thanks, > Christophe A partial / incomplete / broken implementation of querying character properties, I'd guess. See for instance: http://en.wikipedia.org/wiki/Unicode_character_property http://www.fileformat.info/info/unicode/category/index.htm <http://www.fileformat.info/info/unicode/category/index.htm> Cheers, Henry

Sept. 17, 2015

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

by Paul DeBruicker

Marcus Denker-4 wrote > Hi, > > Review needed for > > > https://pharo.fogbugz.com/f/cases/16074/Is-RBClassNotReferencedRule-200x-sl… > > Speedup can be checked with: > > rule:=RBClassNotReferencedRule new. > environment:=RBBrowserEnvironment new forPackageNames: #('Fuel'). > [RBSmalllintChecker runRule: rule onEnvironment: > environment] timeToRun > > > Pharo5: "0:00:00:13.526" > Pharo5 with fix: "0:00:00:00.761" > Pharo3: 0:00:00:00.547 > > Checking all classes for usage: > > [Smalltalk allClassesAndTraits select: [ : each | each isUsed not]] > timeToRun > > still takes 1:40 to find 600 unused classes. > > This shows that adding a cache for âreferenced classesâ per environment > would speed this > up further. But it should be general, not just for RB or even RB Code > Critique. > > The nice things is that code environments only change when you add classes > or methods, > so adding a very general cache should be quite possible. > > Another thing we should add is a way for classes to define âI am usedâ. > E.g this > now finds all Configurations, all the Manifests and all cases like the RB > Rules themselves. > All these should know that they are not useless even though there are no > references. > > Marcus Thanks for working on this. I have a use for it this week. For methods RB allows one to add a pragma e.g.: <lint: 'Methods implemented but not sent' rationale: 'this code plays a role in examining the system' author: 'pad'> and described here: http://www.lukas-renggli.ch/blog/ignoring-lint-rules. Maybe we can hijack that mechanism with a class side method e.g. rbNotReferenced <lint> Do your modifications test sends of subclassesDo: and friends? OR #(#MyClass1 #MyClass2 #MyClass3) collect:[:ea | (Smalltalk at: ea) doSomething]. The reason I ask is I'm planning to do a bunch of cleanup of a package and it would be nice to not worry about those cases getting missed -- View this message in context: http://forum.world.st/review-needed-speed-up-RBClassNotReferencedRule-tp484… Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

Sept. 17, 2015

[Reflectivity] Links on Class Variables

by Marcus Denker

Hi, We can put meta links on AST Nodes. But with Variables being represented by Objects in Pharo, it is easy to allow those to be annotated, too. With the changes from https://pharo.fogbugz.com/f/cases/16596/before-for-Class-Variable <https://pharo.fogbugz.com/f/cases/16596/before-for-Class-Variable> it is now possible to put #before links on class variables: testBeforeClassVariable | classVar link | classVar := ReflectivityExamples classVariableNamed: #ClassVar. link := MetaLink new metaObject: self; selector: #tagExec. classVar link: link. self assert: classVar hasMetalinkBefore. self assert: (ReflectivityExamples >> #exampleClassVarRead) class = ReflectiveMethod. self assert: tag isNil. self assert: ReflectivityExamples new exampleClassVarRead isNil. self assert: tag = 'yes'. self assert: (ReflectivityExamples >> #exampleClassVarRead) class = CompiledMethod. self deny: (ReflectivityExamples >> #exampleClassVarRead) isQuick. link uninstall. ReflectivityExamples recompile: #exampleClassVarRead TODO: #after, #instead, then reifications (e.g. value), then tests for Globals, then the same for instance vars (slots) and tempsâ¦ step by step. Marcus

Sept. 17, 2015

Re: [Pharo-dev] EncodedCharSets

by Henrik Johansen

> On 17 Sep 2015, at 4:09 , 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. > > " If you mean the name EncodedCharSets, and the fact there are Encodings in that list, it is a leftover from when WideStrings code points could be in character sets other than Unicode. That made little sense since each WideChar takes 32 bits anyways (other than a possibly simpler codePoint -> glyph index translation described below), so most/all places where this happened have been removed and we now assume WideString code points to be equal to unicode code points. The goal of the old code was to give parameters to the old StrikeFont string display primitive, which is limited to using a table of 256 glyphs for any string it wants to render. The Scanners job was to introduce stops when it encounters a change that makes the string it's scanning unable to be displayed by a single call to the primitive. The presence of a leadingChar induced(/s) a stop in the Scanner, which explains the mix of encodings and Languages (whose codePoints are in Unicode) in the EncodedCharSets table. This stop let a properly constructed StrikeFontSet swap in a glyph table suitable for displaying the leading char, using a custom codePoint -> glyph index conversion. TLDR; It's a relic of a more complex past used in a mechanism that let StrikeFonts display other than macroman characters. Cheers, Henry P.S. Funnily enough, the mechanism is somewhat similar to what would be needed to swap in fallback fonts for code points not covered by the default font efficiently (although the stop would be on missing glyph, rather than leading char), instead of the current, doomed-to-fail approach of using a FallbackFont in each font.

Sept. 17, 2015

Re: [Pharo-dev] Contribution scheme confusing ...

by Torsten Bergmann

Hi Marcus, Could work - but is a fragile rule. We should document it somewhere. What about providing this info directly in the image. Alexandre has already added a help topic on how to contribute (but yet only for the slice process). I changed it so we can have the differences of config managed ones described as well and we can list the configs there. See the attached changeset that you can file in into the latest Pharo5.0 Latest update: #50329 File in and check "How to contribute to Pharo base code" topic in Help -> Help Browser I guess this would be more clear to us and the users and we can simply point to the help info is someone asks. And look up ourself when we forgot. Thx T. > Gesendet: Donnerstag, 17. September 2015 um 15:40 Uhr > Von: "Marcus Denker" <marcus.denker(a)inria.fr> > An: "Pharo Development List" <pharo-dev(a)lists.pharo.org> > Betreff: Re: [Pharo-dev] Contribution scheme confusing ... > > >> > >> Also: > >> I did a simple fix on NautilusGroupAutoBuilder package - there is > >> a ConfigurationOfNautilusGroupAutoBuilder. But is it managed internally > >> or externally now? > >> > > Even I can not tell you. > > > > I think the rule we should adopt: if there is a dedicated repository defined in the image for a package, people should commit > there. If there is non (just the inbox), then this should be used (with the Slice mechanism). > > The repositories in the image should be already up to date, I think. > > Marcus > > >

Sept. 17, 2015

EncodedCharSets

by Christophe Demarey

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. "

Sept. 17, 2015

Re: [Pharo-dev] Contribution scheme confusing ...

by Tudor Girba

Exactly! This is the cheapest way to document the process. We could even try to automatically remove the inbox if there is another repository associated with a package. What do you think? Cheers, Doru On Thu, Sep 17, 2015 at 3:40 PM, Marcus Denker <marcus.denker(a)inria.fr> wrote: > >> > >> Also: > >> I did a simple fix on NautilusGroupAutoBuilder package - there is > >> a ConfigurationOfNautilusGroupAutoBuilder. But is it managed internally > >> or externally now? > >> > > Even I can not tell you. > > > > I think the rule we should adopt: if there is a dedicated repository > defined in the image for a package, people should commit > there. If there is non (just the inbox), then this should be used (with > the Slice mechanism). > > The repositories in the image should be already up to date, I think. > > Marcus > > > -- www.tudorgirba.com "Every thing has its own flow"

Sept. 17, 2015

Re: [Pharo-dev] Contribution scheme confusing ...

by Marcus Denker

>> >> Also: >> I did a simple fix on NautilusGroupAutoBuilder package - there is >> a ConfigurationOfNautilusGroupAutoBuilder. But is it managed internally >> or externally now? >> > Even I can not tell you. > I think the rule we should adopt: if there is a dedicated repository defined in the image for a package, people should commit there. If there is non (just the inbox), then this should be used (with the Slice mechanism). The repositories in the image should be already up to date, I think. Marcus

Sept. 17, 2015

About the Unicode class

by Christophe Demarey

Hi, Does anyone know why there are so many class variables in Unicode and what they are used for? Most of them have no sender. EncodedCharSet subclass: #Unicode instanceVariableNames: '' classVariableNames: 'Cc Cf Cn Co Cs DecimalProperty GeneralCategory Ll Lm Lo Lt Lu Mc Me Mn Nd Nl No Pc Pd Pe Pf Pi Po Ps Sc Sk Sm So ToCasefold ToLower ToUpper Zl Zp Zs' category: 'Multilingual-Encodings' Thanks, Christophe

Sept. 17, 2015

Re: [Pharo-dev] Rubric/Event handling issue?

by Stephan Eggermont

On 17-09-15 13:59, Ben Coman wrote: > On Tue, Sep 15, 2015 at 11:49 PM, Stephan Eggermont <stephan(a)stack.nl> wrote: >> There is an interesting bad interaction between my drag-and-drop code and >> Rubric. It kills all Rubric editors at the same time, so it looks >> to me like a problem with the Rubric background thread interacting with >> mouseMove or mouseEnter/Leave. >> >> https://vimeo.com/139353992 > > It looks like there was meant to be sound, but I didn't hear any > (could just be me) > cheers -ben Yes, there is sound, and it is 720p video. I'm probably constructing the rubric editor incorrectly, and it would be good if that would only interfere with the ones I created... Stephan

Sept. 17, 2015