On Wed, Jul 7, 2010 at 10:34 PM, Stéphane Ducasse
<[email protected]> wrote:
>
> 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?
>

yes :-)

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

_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to