On Wed, Jul 5, 2017 at 10:04 AM, Stephen R. van den Berg <[email protected]> wrote: > Peter Bortas wrote: >>This is not a release candidate. There are some changes without >>CHANGES and I'm not quite comfortable with the namespace the >>Debug.Peek functionality is living in. > > Any suggestions with regard to where Debug.Peek should live?
Not yet, _my_ primary concern is that it doesn't do anything I would expect with that name. A Peek better come with a Poke. Due to the LysKOM-exporter being busted again* I also don't think you've seen Nilsson's comment: srb>>I'd appreciate feedback on the implementation and usability of Debug.Peek srb>>(in Pike 8.1). nilsson>Isn't this more of an application than a module? I.e. shouldn't this nilsson>rather be in the bin directory or in Tools.Standalone? * I've agreed to take it over, but the final move hasn't been done yet. > I'll comb through the pgsql fixes I committed, some of those were cosmetic > though, not worthy of a CHANGES entry. I didn't see any checkins from you missing in CHANGES. It was something originating from Roxen. Regards, -- Peter Bortas
