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
> [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

Reply via email to