Bytes panel is broken on CompiledMethods

[Ducasse]

[MarcusDenker]

The reason is that CompiledMethod inherits ByteArray>>#inspectBytes:, but that makes no sense for any of the CompiledCode subclasses.

[Ducasse]

I do not really understand but we should fix it or remove it.

[MarcusDenker]

We could override it. I have not found a way to do nothing, so mayeb we could show the bytes of the Bytecode:

inspectBytes: specBuilder
	<inspectorPresentationOrder: 30 title: 'Bytes'>

	^ self size > 1000 
		ifTrue: [
			(StSimpleInspectorBuilder on: specBuilder)
				key: 'error' value: 'too many bytes, size is ' , self size asString;
				table ]
		ifFalse: [
			(StSimpleInspectorBuilder on: specBuilder)
				key: 'hex string bytecode' value: (String streamContents: [ :out | 
					self bytecodes do: [ :each | each printOn: out base: 16 nDigits: 2 ] ]);
				key: 'hex lines bytecode' value: (String streamContents: [ :out |
					self bytecodes withIndexDo: [ :each :index |
						index = 1 ifFalse: [ 
							index - 1 \\ 8 = 0 ifTrue: [ out cr ] ifFalse: [ out space ] ].
						each printOn: out base: 16 nDigits: 2 ] ]);
				table]

Code here instead of PR as I can not commit, I added another issue for that: #16019

[hernanmd]

Merged in pharo-spec/NewTools#670