Issue 2635 - methods recategorization

>
> On Jul 6, 2010, at 10:20 PM, Pavel Krivanek wrote:
>
>> Hi Stef,
>>
>> of course, I will do that. For faster processing I filtered the
>> commands that only change caps in the package names.
>
> good idea
>
>> Added as a
>> comment 10 to this issue. It needs only a patch from comment 5 that
>> adds a method that enables categories merging.
>>
>> !Categorizer methodsFor: 'accessing' stamp: 'pavel.krivanek 7/4/2010 11:54'!
>> moveCategory: sourceCategory to: destinationCategory
>>
>>       self classifyAll: (self listAtCategoryNamed: sourceCategory) under:
>> destinationCategory.
>>       self removeCategory: sourceCategory.
>>       ! !
>
> Ok I will have a look.
>
>>
>> MethodReference organization renameCategory: '*fixunderscores' toBe:
>> '*FixUnderscores'.
>> MethodReference organization renameCategory: '*monticello' toBe: '*Monticello'.
>> MethodReference organization renameCategory: '*packageinfo-base' toBe:
>> '*PackageInfo-Base'.
>> FileStream organization renameCategory: '*compression' toBe: '*Compression'.
>> FileStream organization renameCategory: '*network-uri' toBe: '*Network-URI'.
>> FileStream organization renameCategory: '*network-url' toBe: '*Network-URL'.
>> FileStream organization renameCategory: '*tools' toBe: '*Tools'.
>> FileStream class organization renameCategory: '*network-uri' toBe:
>> '*Network-URI'.
>> PositionableStream organization renameCategory: '*compression' toBe:
>> '*Compression'.
>> TranscriptStream organization renameCategory: '*morphic' toBe: '*Morphic'.
>> Boolean class organization renameCategory: '*system-settings-browser'
>> toBe: '*System-Settings-Browser'.
>> SmartRefStream organization renameCategory: '*compression' toBe: '*compression'.
>> StandardFileStream class organization renameCategory: '*network-uri'
>> toBe: '*Network-URI'.
>> Magnitude organization renameCategory: '*kernel-extensions-streaming'
>> toBe: '*Kernel-Extensions-Streaming'.
>> MacFileDirectory class organization renameCategory: '*network-uri'
>> toBe: '*Network-URI'.
>> Object organization renameCategory: '*system-settings-browser' toBe:
>> '*System-Settings-Browser'.
>> Object organization renameCategory: '*system-support' toBe: '*System-Support'.
>> Object organization renameCategory: '*tools-browser' toBe: '*Tools-Browser'.
>> Object organization renameCategory: '*tools-explorer' toBe: '*Tools-Explorer'.
>> Behavior organization moveCategory: '*system-support' to: '*System-Support'.
>> Integer organization moveCategory: '*files' to: '*Files'.
>> Integer organization moveCategory: '*system-hashing-core' to:
>> '*System-Hashing-Core'.
>> Integer organization moveCategory: '*tools-explorer' to: '*Tools-Explorer'.
>> OrderedCollection organization moveCategory: '*tools-inspector' to:
>> '*Tools-Inspector'.
>> SystemOrganizer organization moveCategory: '*gofer-core-accessing' to:
>> '*Gofer-Core-Accessing'.
>> Trait organization moveCategory: '*monticello' to: '*Monticello'.
>> TraitBehavior organization moveCategory: '*tools-browser' to: '*Tools-Browser'.
>> String organization moveCategory: '*compression' to: '*Compression'.
>> String organization moveCategory: '*files' to: '*Files'.
>> String organization moveCategory: '*monticello' to: '*Monticello'.
>> String organization moveCategory: '*morphic' to: '*Morphic'.
>> String organization moveCategory: '*network-mime' to: '*Network-MIME'.
>> String organization moveCategory: '*network-protocols' to: '*Network-Protocols'.
>> String organization moveCategory: '*network-uri' to: '*Network-URI'.
>> String organization moveCategory: '*network-url' to: '*Network-Url'.
>> String organization moveCategory: '*st80' to: '*ST80'.
>> String organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> Text organization moveCategory: '*morphic-converting' to: '*Morphic-Converting'.
>> Announcer organization moveCategory: '*announcements-view' to:
>> '*Announcements-View'.
>> Announcement organization moveCategory: '*announcements-view' to:
>> '*Announcements-View'.
>> ClassDescription organization moveCategory: '*system-support' to:
>> '*System-Support'.
>> WeakSet organization moveCategory: '*tools-inspector' to: '*Tools-Inspector'.
>> Class organization moveCategory: '*monticello' to: '*Monticello'.
>> FloatArray organization moveCategory: '*tools-inspector' to: '*Tools-Inspector'.
>> Collection organization moveCategory: '*tools-explorer' to: '*Tools-Explorer'.
>> Stream organization moveCategory: '*monticello' to: '*Monticello'.
>> BitBlt organization moveCategory: '*FreeType-addition' to: '*FreeType-Addition'.
>> ReadWriteStream organization moveCategory: '*compression' to: '*Compression'.
>> RunArray class organization moveCategory: '*system-changes' to:
>> '*System-Changes'.
>> Color organization moveCategory: '*morphic' to: '*Morphic'.
>> Color class organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> AcornFileDirectory class organization moveCategory: '*network-uri' to:
>> '*Network-URI'.
>> ImageSegment organization moveCategory: '*tests-system' to: '*Tests-System'.
>> ChangeRecord organization moveCategory: '*monticello' to: '*Monticello'.
>> Form organization moveCategory: '*morphic' to: '*Morphic'.
>> Number class organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> DosFileDirectory class organization moveCategory: '*network-uri' to:
>> '*Network-URI'.
>> Dictionary organization moveCategory: '*compiler' to: '*Compiler'.
>> Dictionary organization moveCategory: '*tools-inspector' to: '*Tools-Inspector'.
>> Association organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> SequenceableCollection organization moveCategory: '*kernel-extensions'
>> to: '*Kernel-Extensions'.
>> SequenceableCollection organization moveCategory:
>> '*morphic-newcurves-cubic support' to: '*Morphic-Newcurves-Cubic
>> support'.
>> SequenceableCollection organization moveCategory: '*traits' to: '*Traits'.
>> Array organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> CrLfFileStream organization moveCategory: '*monticello' to: '*Monticello'.
>> Point class organization moveCategory: '*system-settings-browser' to:
>> '*System-Settings-Browser'.
>> SystemNavigation organization moveCategory: '*multilingual-editor' to:
>> '*Multilingual-Editor'.
>> SystemNavigation organization moveCategory: '*tools-browser' to:
>> '*Tools-Browser'.
>> FileDirectory organization moveCategory: '*network-uri' to: '*network-URI'.
>> FileDirectory class organization moveCategory: '*network-uri' to:
>> '*Network-URI'.
>> Set organization moveCategory: '*tools-inspector' to: '*Tools-Inspector'.
>> StringHolder organization moveCategory: '*tools' to: '*Tools'.
>> StringHolder organization moveCategory: '*tools-traits' to: '*Tools-Traits'.
>> ByteArray organization moveCategory: '*compression' to: '*Compression'.
>> ByteArray organization moveCategory: '*system-hashing-core' to:
>> '*System-Hashing-Core'.
>>
>> -- Pavel
>>
>>
>>
>> On Tue, Jul 6, 2010 at 9:15 PM, Stéphane Ducasse
>> <[hidden email]> wrote:
>>> pavel
>>>
>>> could you chop it and send it piece by piece to the list so that everybody can get a feel about it
>>> and discuss if necessary?
>>> I'm sorry but I like that we act a group and not a bunch of cool individual and building a mind-share is important
>>> for me.
>>>
>>> Stef
>>> On Jul 6, 2010, at 4:43 PM, Pavel Krivanek wrote:
>>>
>>>> Hi,
>>>>
>>>> I finished the list of changes for the issue
>>>> http://code.google.com/p/pharo/issues/detail?id=2635.
>>>>
>>>> This issue is the next step after already integrated packages
>>>> reorganization and the goal is to have compact package base a the
>>>> small headless PharoKernel image.
>>>>
>>>> Most changes are simple naming unification and some classes I skipped
>>>> because they need deeper refactorings.
>>>>
>>>> I will now check the list once more.
>>>>
>>>> -- Pavel
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

> Hi,
>
> I finished the list of changes for the issue
> http://code.google.com/p/pharo/issues/detail?id=2635.
>
> This issue is the next step after already integrated packages
> reorganization and the goal is to have compact package base a the
> small headless PharoKernel image.
>
> Most changes are simple naming unification and some classes I skipped
> because they need deeper refactorings.
>
> I will now check the list once more.
>
> -- Pavel
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

> Hi Pavel
>
> where should I start?
> Because the changes you sent by emails are not in the same order than on the list?
>
> I was planning to start with the mail where you just uppercase the category?
>
> Stef
>
> On Jul 6, 2010, at 4:43 PM, Pavel Krivanek wrote:
>
>> Hi,
>>
>> I finished the list of changes for the issue
>> http://code.google.com/p/pharo/issues/detail?id=2635.
>>
>> This issue is the next step after already integrated packages
>> reorganization and the goal is to have compact package base a the
>> small headless PharoKernel image.
>>
>> Most changes are simple naming unification and some classes I skipped
>> because they need deeper refactorings.
>>
>> I will now check the list once more.
>>
>> -- Pavel
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

> Hi Stef,
>
> start with patch in comment 5 (reason in comment 2) and then with
> comment 10 up to 19 (code/description)
>
> Cheers,
> -- Pavel
>
> On Wed, Jul 7, 2010 at 7:05 PM, Stéphane Ducasse
> <[hidden email]> wrote:
>> Hi Pavel
>>
>> where should I start?
>> Because the changes you sent by emails are not in the same order than on the list?
>>
>> I was planning to start with the mail where you just uppercase the category?
>>
>> Stef
>>
>> On Jul 6, 2010, at 4:43 PM, Pavel Krivanek wrote:
>>
>>> Hi,
>>>
>>> I finished the list of changes for the issue
>>> http://code.google.com/p/pharo/issues/detail?id=2635.
>>>
>>> This issue is the next step after already integrated packages
>>> reorganization and the goal is to have compact package base a the
>>> small headless PharoKernel image.
>>>
>>> Most changes are simple naming unification and some classes I skipped
>>> because they need deeper refactorings.
>>>
>>> I will now check the list once more.
>>>
>>> -- Pavel
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

> code:
>
> MethodReference organization moveCategory: '*Kernel-Traits' to:
> '*Traits-Kernel'.
> LRUCache organization moveCategory: '*Polymorph-Widgets-override' to:
> 'accessing'.
> Project class organization moveCategory: 'utilities' to: '*Morphic'.
> Character organization moveCategory: '*Multilingual' to:
> '*Multilingual-Encodings'.
> TranscriptStream organization moveCategory: 'toolbuilder' to:
> '*ToolBuilder-Kernel'.
> TranscriptStream class organization moveCategory: 'toolbuilder' to:
> '*ToolBuilder-Kernel'.
>
> description:
>
> MethodReference organization moveCategory: '*Kernel-Traits' to:
> '*Traits-Kernel'.
> - swap package name parts to a more logical ownership. It contains
> only a method actualClass that has no other kernel implementation.
> Both packages will be in the kernel image.

> code:
>
> SystemNavigation organization renameCategory: '*multilingual-editor'
> toBe: '*Multilingual-Encodings'.
> Project class organization classify: #spawnNewProcessIfThisIsUI:
> under: '*Morphic'.
> Project class organization removeCategory:'squeaklet on server'.
> ImmX11 organization classify: #keyboardFocusForAMorph: under: '*Morphic'.
> FileStream class organization classify:
> #fileReaderServicesForFile:suffix: under: '*Tools-FileList'.
> FileStream class organization classify: #serviceFileIn under: '*Tools-FileList'.
> FileStream class organization classify: #serviceRemoveLineFeeds under:
> '*Tools-FileList'.
> FileStream class organization classify: #services under: '*Tools-FileList'.
> Character organization classify: #macRomanToUnicode under:
> '*Multilingual-TextConversion'.
> Character organization classify: #unicodeToMacRoman under:
> '*Multilingual-TextConversion'.
> TranscriptStream organization classify: #codePaneMenu:shifted: under:
> '*Tools-Base'.
> TranscriptStream class organization classify: #openMorphicTranscript
> under: '*Morphic'.
>
> description:
>
> SystemNavigation organization renameCategory: '*multilingual-editor'
> toBe: '*Multilingual-Encodings'.
> - move the single method allSelect: to a base Multilingual package

>
> On Jul 6, 2010, at 10:51 PM, Pavel Krivanek wrote:
>
>> code:
>>
>> MethodReference organization moveCategory: '*Kernel-Traits' to:
>> '*Traits-Kernel'.
>> LRUCache organization moveCategory: '*Polymorph-Widgets-override' to:
>> 'accessing'.
>> Project class organization moveCategory: 'utilities' to: '*Morphic'.
>> Character organization moveCategory: '*Multilingual' to:
>> '*Multilingual-Encodings'.
>> TranscriptStream organization moveCategory: 'toolbuilder' to:
>> '*ToolBuilder-Kernel'.
>> TranscriptStream class organization moveCategory: 'toolbuilder' to:
>> '*ToolBuilder-Kernel'.
>>
>> description:
>>
>> MethodReference organization moveCategory: '*Kernel-Traits' to:
>> '*Traits-Kernel'.
>> - swap package name parts to a more logical ownership. It contains
>> only a method actualClass that has no other kernel implementation.
>> Both packages will be in the kernel image.
>
> What about just folding actualClass in MethodReference?
> I do not understand the point of the KernelTraits or Traits-Kernel.
>
> May be MCDefinition>>actualClass should also be folded into MCDefinition and not Polymorph-Tools-Diff
>
>
>
>> LRUCache organization moveCategory: '*Polymorph-Widgets-override' to:
>> 'accessing'.
>> - only one method. Declared as override but has no old version in the
>> history and has general functionality
> ok
>
>> Project class organization moveCategory: 'utilities' to: '*Morphic'.
>> - all methods have no real senders or are called only from Morphic
> In fact these methods should be moved in another place so that we can remove Project.
> For now this is ok to move them to Morphic
>
>
>> Character organization moveCategory: '*Multilingual' to:
>> '*Multilingual-Encodings'.
>> - only fix MC package name
> Ok even if I was wondering why encodings is not folded into Collections-Strings..
>
>
>> TranscriptStream organization moveCategory: 'toolbuilder' to:
>> '*ToolBuilder-Kernel'.
>> - single method buildWith:, used only for the Toolbuilder
>>
>> TranscriptStream class organization moveCategory: 'toolbuilder' to:
>> '*ToolBuilder-Kernel'.
>> - the same
>
> with this logic it means that we could remove toolbuilder or plug it on demand.
> Why not.
>
> I will integrate already that.
>
>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

>
> On Jul 6, 2010, at 11:24 PM, Pavel Krivanek wrote:
>
>> code:
>>
>> SystemNavigation organization renameCategory: '*multilingual-editor'
>> toBe: '*Multilingual-Encodings'.
>> Project class organization classify: #spawnNewProcessIfThisIsUI:
>> under: '*Morphic'.
>> Project class organization removeCategory:'squeaklet on server'.
>> ImmX11 organization classify: #keyboardFocusForAMorph: under: '*Morphic'.
>> FileStream class organization classify:
>> #fileReaderServicesForFile:suffix: under: '*Tools-FileList'.
>> FileStream class organization classify: #serviceFileIn under: '*Tools-FileList'.
>> FileStream class organization classify: #serviceRemoveLineFeeds under:
>> '*Tools-FileList'.
>> FileStream class organization classify: #services under: '*Tools-FileList'.
>> Character organization classify: #macRomanToUnicode under:
>> '*Multilingual-TextConversion'.
>> Character organization classify: #unicodeToMacRoman under:
>> '*Multilingual-TextConversion'.
>> TranscriptStream organization classify: #codePaneMenu:shifted: under:
>> '*Tools-Base'.
>> TranscriptStream class organization classify: #openMorphicTranscript
>> under: '*Morphic'.
>>
>> description:
>>
>> SystemNavigation organization renameCategory: '*multilingual-editor'
>> toBe: '*Multilingual-Encodings'.
>> - move the single method allSelect: to a base Multilingual package
> I would fold it into the class. Especially since Dictionary unreferencedKeys
> ok?
>

>
>> Project class organization classify: #spawnNewProcessIfThisIsUI:
>> under: '*Morphic'.
>> Project class organization removeCategory:'squeaklet on server'.
>> - UI specific and sent only from Morphic dependent methods
> ok
>
>
>> ImmX11 organization classify: #keyboardFocusForAMorph: under: '*Morphic'.
>> - used only for Morphic
> ok
>
>> FileStream class organization classify:
>> #fileReaderServicesForFile:suffix: under: '*Tools-FileList'.
>> FileStream class organization classify: #serviceFileIn under: '*Tools-FileList'.
>> FileStream class organization classify: #serviceRemoveLineFeeds under:
>> '*Tools-FileList'.
>> FileStream class organization classify: #services under: '*Tools-FileList'.
>> - FileList files associations. However a deeper refactoring is welcome.
> this is ok for now.
>
>
>> Character organization classify: #macRomanToUnicode under:
>> '*Multilingual-TextConversion'.
>> Character organization classify: #unicodeToMacRoman under:
>> '*Multilingual-TextConversion'.
>> - both packages in the PharoKernel. Move to a more fitting package.
> ok sounds good.
>
>> TranscriptStream organization classify: #codePaneMenu:shifted: under:
>> '*Tools-Base'.
>
> codePaneMenu: aMenu shifted: shifted looks like a hack. I will open a ticket.
> all the class in fact! It sucks
>
>> TranscriptStream class organization classify: #openMorphicTranscript
>> under: '*Morphic'.
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>