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

[pharo-project/pharo-core] edf113: 50069

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: edf11372826bd59dde682e7e6f05ca21ebb1f8a9 https://github.com/pharo-project/pharo-core/commit/edf11372826bd59dde682e7e… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-27 (Wed, 27 May 2015) Changed paths: R Komitter.package/Komitter.class/class/world menu/initialize.st M Komitter.package/Komitter.class/class/world menu/komitterMenuOn_.st M Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testConditionWithArguments.st M Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testHookWithArguments.st M Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testHookWithVirtualMeta.st R Reflectivity.package/Breakpoint.class/instance/as yet unclassified/initialize.st A Reflectivity.package/Breakpoint.class/instance/initialization/initialize.st A Reflectivity.package/HookGenerator.class/class/instance creation/node_link_.st R Reflectivity.package/HookGenerator.class/instance/initialize/initForNode_.st A Reflectivity.package/HookGenerator.class/instance/initialize/node_link_.st R Reflectivity.package/ManifestReflectivity.class/instance/code-critics/ruleRBBadMessageRuleV1FalsePositive.st R Reflectivity.package/MetaLink.class/class/as yet unclassified/uninstallAll.st A Reflectivity.package/MetaLink.class/class/cleanup/uninstallAll.st M Reflectivity.package/MetaLink.class/instance/ast/hookOn_.st A Reflectivity.package/RFArgumentsReification.class/README.md A Reflectivity.package/RFArgumentsReification.class/class/plugin interface/key.st A Reflectivity.package/RFArgumentsReification.class/class/plugin interface/nodes.st A Reflectivity.package/RFArgumentsReification.class/definition.st A Reflectivity.package/RFArgumentsReification.class/instance/generate/genForRBProgramNode.st R Reflectivity.package/RFLinkReification.class/instance/as yet unclassified/genForRBProgramNode.st A Reflectivity.package/RFLinkReification.class/instance/generate/genForRBProgramNode.st M Reflectivity.package/RFNewValueReification.class/instance/generate/genForRBAssignmentNode.st R Reflectivity.package/RFNodeReification.class/instance/as yet unclassified/genForRBProgramNode.st A Reflectivity.package/RFNodeReification.class/instance/generate/genForRBProgramNode.st R Reflectivity.package/RFObjectReification.class/instance/code generation/genForRBProgramNode.st A Reflectivity.package/RFObjectReification.class/instance/generate/genForRBProgramNode.st R Reflectivity.package/RFReification.class/class/accessing/allKeys.st R Reflectivity.package/RFReification.class/class/accessing/for_.st R Reflectivity.package/RFReification.class/class/as yet unclassified/node_link_.st A Reflectivity.package/RFReification.class/class/instance creation/node_link_.st A Reflectivity.package/RFReification.class/class/plugin interface/allKeys.st A Reflectivity.package/RFReification.class/class/plugin interface/for_.st M Reflectivity.package/RFSemanticAnalyzer.class/instance/visiting/visitNode_.st A Reflectivity.package/RFSenderReification.class/README.md A Reflectivity.package/RFSenderReification.class/class/plugin interface/key.st A Reflectivity.package/RFSenderReification.class/class/plugin interface/nodes.st A Reflectivity.package/RFSenderReification.class/definition.st A Reflectivity.package/RFSenderReification.class/instance/generate/genForRBMessageNode.st A Reflectivity.package/RFSenderReification.class/instance/generate/genForRBMethodNode.st R Reflectivity.package/RFTArgumentsReification.class/README.md R Reflectivity.package/RFTArgumentsReification.class/class/plugin interface/key.st R Reflectivity.package/RFTArgumentsReification.class/class/plugin interface/nodes.st R Reflectivity.package/RFTArgumentsReification.class/definition.st R Reflectivity.package/RFTArgumentsReification.class/instance/generate/genForRBProgramNode.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50068.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50069.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50068.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50069.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M System-Caching.package/DoubleLinkedList.class/README.md M System-Caching.package/DoubleLinkedList.class/instance/accessing/first.st A System-Caching.package/DoubleLinkedList.class/instance/accessing/firstLink.st M System-Caching.package/DoubleLinkedList.class/instance/accessing/last.st A System-Caching.package/DoubleLinkedList.class/instance/accessing/lastLink.st M System-Caching.package/DoubleLinkedList.class/instance/adding/addAll_.st M System-Caching.package/DoubleLinkedList.class/instance/adding/addFirst_.st M System-Caching.package/DoubleLinkedList.class/instance/adding/addLast_.st M System-Caching.package/DoubleLinkedList.class/instance/adding/add_.st M System-Caching.package/DoubleLinkedList.class/instance/adding/add_afterLink_.st M System-Caching.package/DoubleLinkedList.class/instance/adding/add_beforeLink_.st M System-Caching.package/DoubleLinkedList.class/instance/converting/asArray.st M System-Caching.package/DoubleLinkedList.class/instance/enumerating/collect_.st M System-Caching.package/DoubleLinkedList.class/instance/enumerating/do_.st A System-Caching.package/DoubleLinkedList.class/instance/enumerating/linksDo_.st M System-Caching.package/DoubleLinkedList.class/instance/enumerating/reject_.st M System-Caching.package/DoubleLinkedList.class/instance/enumerating/reverseDo_.st A System-Caching.package/DoubleLinkedList.class/instance/enumerating/reverseLinksDo_.st M System-Caching.package/DoubleLinkedList.class/instance/enumerating/select_.st A System-Caching.package/DoubleLinkedList.class/instance/private/emptyCheck.st M System-Caching.package/DoubleLinkedList.class/instance/removing/removeAll.st M System-Caching.package/DoubleLinkedList.class/instance/removing/removeFirst.st M System-Caching.package/DoubleLinkedList.class/instance/removing/removeLast.st M System-Caching.package/DoubleLinkedList.class/instance/removing/removeLink_.st M System-Caching.package/DoubleLinkedList.class/instance/testing/includes_.st M System-Caching.package/DoubleLinkedList.class/instance/testing/isEmpty.st A System-CachingTests.package/DoubleLinkedListTests.class/instance/testing/testFirstAndLastLink.st A System-CachingTests.package/DoubleLinkedListTests.class/instance/testing/testLinksDo.st A System-CachingTests.package/DoubleLinkedListTests.class/instance/testing/testReverseLinksDo.st A Tool-Base.package/PharoShortcuts.class/instance/kaymaps - Tools/openKomitterShortcut.st A Tool-Base.package/ToolShortcutsCategory.class/instance/keymaps/openKomitter.st Log Message: ----------- 50069 15340 Extending DoubleLinkedList API https://pharo.fogbugz.com/f/cases/15340 15617 Shortcut($k command) doesnt open the Komitter https://pharo.fogbugz.com/f/cases/15617 15616 Improve Reifications https://pharo.fogbugz.com/f/cases/15616 http://files.pharo.org/image/50/50069.zip

May 27, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50068 Home: https://github.com/pharo-project/pharo-core

May 27, 2015

[pharo-project/pharo-core] c9dbbd: 50068

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: c9dbbdb8f54d76633304cbe15a0b0b5864a9c05f https://github.com/pharo-project/pharo-core/commit/c9dbbdb8f54d76633304cbe1… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-27 (Wed, 27 May 2015) Changed paths: A Balloon.package/extension/InfiniteForm/instance/fillRectangle_on_.st R Collections-Abstract.package/Collection.class/instance/adapting/adaptToString_andSend_.st R Collections-Arithmetic.package/extension/String/instance/%2A.st R Collections-Arithmetic.package/extension/String/instance/%2F%2F.st R Collections-Arithmetic.package/extension/String/instance/%2F.st R Collections-Arithmetic.package/extension/String/instance/%5C%5C.st R Collections-Arithmetic.package/extension/String/instance/+.st R Collections-Arithmetic.package/extension/String/instance/-.st R Collections-Strings.package/String.class/instance/converting/adaptToCollection_andSend_.st R Collections-Strings.package/String.class/instance/converting/adaptToNumber_andSend_.st R Collections-Strings.package/String.class/instance/converting/adaptToPoint_andSend_.st R Collections-Strings.package/String.class/instance/converting/adaptToString_andSend_.st R Kernel.package/Number.class/instance/converting/adaptToString_andSend_.st R Kernel.package/Point.class/instance/converting/adaptToString_andSend_.st M Morphic-Core.package/WorldState.class/class/world menu items/mostUsedToolsOn_.st M Morphic-Core.package/WorldState.class/class/world menu items/quitItemsOn_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/README.md A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/definition.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/accessing/maxLength.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/accessing/maxLength_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/accessing/minExtent.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/changed/keyboardFocusChange_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/event handling/keyStroke_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/event handling/mouseMove_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/events/localHandleKeystroke_.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/find-replace/sharesFindReplace.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/geometry/minimumExtent.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/layout/minHeight.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/layout/minWidth.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/private/editorClass.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/private/paragraph.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/private/useFindReplaceSelection.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/private/useSecondarySelection.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/private/useSelectionBar.st A Morphic-Widgets-Pluggable.package/TextMorphForFieldView.class/instance/testing/isEditEvent_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/README.md R Polymorph-Widgets.package/TextMorphForFieldView.class/definition.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/accessing/maxLength.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/accessing/maxLength_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/accessing/minExtent.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/changed/keyboardFocusChange_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/event handling/keyStroke_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/event handling/mouseMove_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/events/localHandleKeystroke_.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/find-replace/sharesFindReplace.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/geometry/minimumExtent.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/layout/minHeight.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/layout/minWidth.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/private/editorClass.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/private/paragraph.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/private/useFindReplaceSelection.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/private/useSecondarySelection.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/private/useSelectionBar.st R Polymorph-Widgets.package/TextMorphForFieldView.class/instance/testing/isEditEvent_.st R Polymorph-Widgets.package/extension/FillStyle/instance/fillRectangle_on_.st R Polymorph-Widgets.package/extension/InfiniteForm/instance/fillRectangle_on_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50067.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50068.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50067.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50068.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R Spec-Inspector.package/extension/Object/instance/basicInspect.st R Spec-Inspector.package/extension/Object/instance/explore.st R Spec-Inspector.package/extension/Object/instance/inspect.st R Spec-Inspector.package/extension/Object/instance/inspectOnce.st R Spec-Inspector.package/extension/Object/instance/inspectWithLabel_.st R Spec-Inspector.package/extension/Object/instance/inspector.st R Spec-Inspector.package/extension/Object/instance/inspectorClass.st R Spec-Tools.package/extension/Object/instance/displaySubObjectAt_.st R Spec-Tools.package/extension/Object/instance/subObjectsToInspect.st R Spec-Tools.package/extension/SequenceableCollection/instance/displaySubObjectAt_.st R Spec-Tools.package/extension/SequenceableCollection/instance/subObjectsToInspect.st M System-Settings.package/SettingBrowser.class/class/world menu/menuCommandOn_.st A Tool-Base.package/extension/Object/instance/basicInspect.st A Tool-Base.package/extension/Object/instance/explore.st A Tool-Base.package/extension/Object/instance/inspect.st A Tool-Base.package/extension/Object/instance/inspectOnce.st A Tool-Base.package/extension/Object/instance/inspectWithLabel_.st A Tool-Base.package/extension/Object/instance/inspector.st A Tool-Base.package/extension/Object/instance/inspectorClass.st M Tool-Transcript.package/extension/ThreadSafeTranscript/class/menuCommandOn_.st Log Message: ----------- 50068 13829 TextMorphForEditorView should be packaged in Moprhic-Widgets-Pluggable https://pharo.fogbugz.com/f/cases/13829 2906 Analyze #adaptToNumber:andSend: https://pharo.fogbugz.com/f/cases/2906 14040 FillStyle>>fillRectangle: aRectangle on: aCanvas should be packaged in Balloon https://pharo.fogbugz.com/f/cases/14040 15613 Display most use tools shortcuts on World menu. https://pharo.fogbugz.com/f/cases/15613 15135 Object extensions related to explore/inspect should be repackaged into their own independent package https://pharo.fogbugz.com/f/cases/15135 http://files.pharo.org/image/50/50068.zip

May 27, 2015

Re: [Pharo-dev] Gofer upgrade error

by Marcus Denker

> On 26 May 2015, at 17:30, Ben Coman <btc(a)openInWorld.com> wrote: > > Thanks Leo. Indeed I see the commands you tried are suggested in those > references. > > On Tue, May 26, 2015 at 3:26 AM, Leo <lpaniceres(a)gmail.com> wrote: >> Ben, >> the documentation for "Gofer upgrade" is >> http://file-pharo.inria.fr/doc/4.0/class/Gofer.html > > Doing this is 50064 I gives... 'MNU receiver of "default" is nil' in... > GoferUnload>>unregisterPackageInfo: aWorkingCopy > PackageOrganizer default > unregisterPackage: aWorkingCopy packageInfo The problem is that it loads a very old version of Goferâ¦ one that e.g. uses the old filesystem, Packageinfo instead of RPackage and things like that. We should just remove the update method and solve update systemwide better. Marcus

May 27, 2015

Re: [Pharo-dev] Is there an official JSON parser/writer?

by Levente Uzonyi

On Tue, 26 May 2015, Esteban A. Maringolo wrote: > 2015-05-26 16:15 GMT-03:00 Stephan Eggermont <stephan(a)stack.nl>: >> On 26/05/15 18:40, Esteban A. Maringolo wrote: >>> >>> I'm modifying the PostgresV2 package to support JSON datatype but >>> instead of using the JSON package from PharoExtras I'm trying to use >>> NeoJSON. > >> Is there a specific reason not to move to PostgresV3? > > I'd love to do so. But where is it? It is not in SmalltalkHub certainly. Google is your friend in this case. It's on squeaksource.com, but I also saw a Pharo port on Github. > It would also require to adapt/implement GLORP's adaptor for PostgresV3. > Or is it a plug-in replacement? It has slightly different API than the V2 version. I've never used Glorp, so I can't tell if it works or not, but I assume that it does not. Levente > > Please provide me with pointers, now is when :) > > > Esteban A. Maringolo > >

May 27, 2015

Inconsistent number comparison?

by Sean P. DeNigris

Why does "1 = 1.0" but "273.15 ~= (5463/20)"? You must send #asFloat to the fraction to answer true... ----- Cheers, Sean -- View this message in context: http://forum.world.st/Inconsistent-number-comparison-tp4828795.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

May 27, 2015

Re: [Pharo-dev] translateBy: vs translatedBy:

by Ben Coman

> > On Tue, May 26, 2015 at 6:14 PM, Ben Coman <btc(a)openinworld.com> wrote: >> >> On Tue, May 26, 2015 at 9:38 PM, Peter UhnÃ¡k <i.uhnak(a)gmail.com> wrote: >> >> I vote for renaming to Rectangle>>translatedBy:, because it returns a >> >> new >> >> translated rectangle instance. >> > >> > >> > -1 >> > There is no benefit in renaming a single method, apart from creating >> > more >> > confusion. >> >> I wouldn't say "no benefit". Consistent conventions are beneficial. >> It could be useful to bring these into line with the general "-ed" >> convention of returning a new object. I don't see how this would >> create confusion, though it might create work. Whether its worth the >> work is a slightly different question. >> >> > If anything, the whole Rectangle API would have to be changed, as there >> > is >> > plethora of similar methods (expandBy:, extendBy:, merge:, etc.) >> >> For a better feel of the work involved... >> >> Rectangle methods select: [ :m | m sourceCode includesSubstring: '^ >> Rectangle ' ]. >> Rectangle>>#intersect: >> Rectangle>>#quickMerge: >> Rectangle>>#encompass: >> Rectangle>>#intersect:ifNone: >> >> Rectangle methods select: [ :m | m sourceCode includesSubstring: >> '^Rectangle ' ]. >> Rectangle>>#intersect: >> Rectangle>>#translateBy: >> Rectangle>>#scaleBy: >> Rectangle>>#merge: >> Rectangle>>#compressTo: >> Rectangle>>#expandTo: >> Rectangle>>#roundTo: >> Rectangle>>#truncateTo: >> >> The following already follow the convention... >> Rectangle>>#compressed >> Rectangle>>#expanded >> Rectangle>>#rounded >> Rectangle>>#truncated >> >> > Pretty much all Rectangle messages are immutable and create new >> > Rectangle, >> > thus I feel it is unnecessary to state it explicitly. >> > >> > In VW there is plenty of methods (e.g. left:) that mutates it, thus >> > unlike >> > Pharo a clear distinction is required. >> >> Still, it is reasonable to consider the impact of inconsistent >> conventions on the rest of the system outside of Rectangle. >> >> cheers -ben >> > On Wed, May 27, 2015 at 1:16 AM, Peter UhnÃ¡k <i.uhnak(a)gmail.com> wrote: > That's not all, many methods return new rectangles via Point's API, namely > Point>>{#corner: #extent: #rectangle:} > > I don't know if I can perform static analysis of the code (are there tools > for that in Pharo?) to see which methods exactly return Rectangle > > rectMethods := #(#corner: #extent: #rectangle:). > (Point methods select: [ :m | rectMethods includes: m selector ] > thenCollect: [ :m | m senders ]) flattened select: [ :s | s className = > #Rectangle ]. "an Array( > Rectangle>>#areasOutside: > Rectangle>>#withSideOrCorner:setToPoint:minExtent:limit: > Rectangle>>#withBottom: > Rectangle>>#bottom: > Rectangle>>#areasOverlapingOutside: > Rectangle>>#floor > Rectangle>>#allAreasOutsideList:startingAt:do: > Rectangle>>#scaledAndCenteredIn: > Rectangle>>#squishedWithin: > Rectangle>>#ceiling > Rectangle>>#withHeight: > Rectangle>>#rectanglesAt:height: > Rectangle>>#scaleFrom:to: > Rectangle>>#withLeft: > Rectangle>>#left: > Rectangle>>#withTop: > Rectangle>>#right: > Rectangle>>#top: > Rectangle>>#innerCorners > Rectangle>>#interpolateTo:at: > Rectangle>>#withWidth: > Rectangle>>#quickMergePoint: > Rectangle>>#withRight: > Rectangle>>#rotateBy:centerAt: > Rectangle>>#withSideOrCorner:setToPoint:minExtent:limit: > Rectangle>>#flipBy:centerAt:) > " > > This list is maybe a bit excessive (sometimes it returns self, sometimes new > rectangle -- e.g. floor). > Good Point ;) However the "-ed" convention doesn't make sense for many of those... #areasdOutside: , #areasedOutside , #areaOutsided: ? ... and #areasOutside is obviously a different rectangle than the original. The scope might be restricted to only those where the usual convention makes the names misleading, for example: #merge cheers -ben

May 26, 2015

Re: [Pharo-dev] Is there an official JSON parser/writer?

by Esteban A. Maringolo

2015-05-26 16:42 GMT-03:00 Sven Van Caekenberghe <sven(a)stfx.eu>: >> On 26 May 2015, at 19:24, Esteban A. Maringolo <emaringolo(a)gmail.com> wrote: >> It could be dead simple as specifying the class if the different JSON >> libraries shared a common to/from API selectors, but they don't. > Yes, it would be very good if there was such a thing as a common API for any reader/writer of any format (JSON, CSV, XML, STON, FUEL, ...). > > For a reader that is pretty easy, I think. Any reader object should understand > > #readFrom: <stream> (binary or character ReadStream as appropriate) +1 > And maybe also > > #readFromString: <string> It could be implemented using #readFrom: > #readFromBytes: <bytearray> This should also specify encoding, I think. > For the writer, I am not so sure. Maybe > #nextPut: <object> toStream: <stream> (binary or character ReadStream as appropriate) > And maybe also > #toString: <object> In this particular case it is all what I'd need, #fromString:/toString: > #toBytes: <object> > Note that the reader and writer objects could be different. This is also a requirement, particularly if you want to use NeoJSON/CSV > But like I said, I am not so sure about the API. We could try to define a Trait for this. I think it's more an Interface/protocol (contract) rather than a trait. But maybe because I don't know how a trait could be implemented to solve this. Regards! Esteban A. Maringolo

May 26, 2015

Re: [Pharo-dev] Is there an official JSON parser/writer?

by Norbert Hartl

> Am 26.05.2015 um 22:13 schrieb Sven Van Caekenberghe <sven(a)stfx.eu>: > > >> On 26 May 2015, at 21:54, Norbert Hartl <norbert(a)hartl.name> wrote: >> >>> >>> Am 26.05.2015 um 21:42 schrieb Sven Van Caekenberghe <sven(a)stfx.eu>: >>> >>> >>>> On 26 May 2015, at 19:24, Esteban A. Maringolo <emaringolo(a)gmail.com> wrote: >>>> >>>> It could be dead simple as specifying the class if the different JSON >>>> libraries shared a common to/from API selectors, but they don't. >>> >>> Yes, it would be very good if there was such a thing as a common API for any reader/writer of any format (JSON, CSV, XML, STON, FUEL, ...). >>> >>> For a reader that is pretty easy, I think. Any reader object should understand >>> >>> #readFrom: <stream> (binary or character ReadStream as appropriate) >>> >>> And maybe also >>> >>> #readFromString: <string> >>> #readFromBytes: <bytearray> >>> >>> For the writer, I am not so sure. Maybe >>> >>> #nextPut: <object> toStream: <stream> (binary or character ReadStream as appropriate) >>> >>> And maybe also >>> >>> #toString: <object> >>> #toBytes: <object> >>> >>> Note that the reader and writer objects could be different. >>> >>> But like I said, I am not so sure about the API. We could try to define a Trait for this. >>> >> I'd like to have #readFrom:as: . Some of the mappers instantiate common classes that should be overwritable. Of course this does not apply for e.g. fuel. >> >> Norbert > > Hmm, that would become quickly much more complex than I intended it to be. More complex configuration should be done up front when you configure the reader or writer object, I think. > > The idea is to find a usage common API. > Agreed somehow :) Norbert

May 26, 2015

Re: [Pharo-dev] Is there an official JSON parser/writer?

by Sven Van Caekenberghe

> On 26 May 2015, at 21:54, Norbert Hartl <norbert(a)hartl.name> wrote: > >> >> Am 26.05.2015 um 21:42 schrieb Sven Van Caekenberghe <sven(a)stfx.eu>: >> >> >>> On 26 May 2015, at 19:24, Esteban A. Maringolo <emaringolo(a)gmail.com> wrote: >>> >>> It could be dead simple as specifying the class if the different JSON >>> libraries shared a common to/from API selectors, but they don't. >> >> Yes, it would be very good if there was such a thing as a common API for any reader/writer of any format (JSON, CSV, XML, STON, FUEL, ...). >> >> For a reader that is pretty easy, I think. Any reader object should understand >> >> #readFrom: <stream> (binary or character ReadStream as appropriate) >> >> And maybe also >> >> #readFromString: <string> >> #readFromBytes: <bytearray> >> >> For the writer, I am not so sure. Maybe >> >> #nextPut: <object> toStream: <stream> (binary or character ReadStream as appropriate) >> >> And maybe also >> >> #toString: <object> >> #toBytes: <object> >> >> Note that the reader and writer objects could be different. >> >> But like I said, I am not so sure about the API. We could try to define a Trait for this. >> > I'd like to have #readFrom:as: . Some of the mappers instantiate common classes that should be overwritable. Of course this does not apply for e.g. fuel. > > Norbert Hmm, that would become quickly much more complex than I intended it to be. More complex configuration should be done up front when you configure the reader or writer object, I think. The idea is to find a usage common API.

May 26, 2015