Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] memory browser reviews - cdt-dev Digest, Vol 141, Issue 6

Hi Andrew,

That is really great to hear. Thanks for stepping forward. 

We are happy to help to keep things moving and get you onto being CDT committers. 

For these 3 bugs my understanding is that all the reviews have been done we are just waiting on Norman to have some time to address them. If this is not the case please let me know. 

Regards,
Tracy





On Sat, Apr 15, 2017 at 2:07 PM, McLachlan, Andrew <Andrew.McLachlan@xxxxxxxxxx> wrote:
Hi Jonah,

Sounds like that is not going to be an option. We'd love to become (new) CDT committers, so if anyone can help move these code reviews along, that would be appreciated. We'd love to start contributing more to CDT.

Thanks in advance,

Andrew

-----Original Message-----
From: Jonah Graham [mailto:jonah@xxxxxxxxxxxxxxxx]
Sent: Wednesday, April 05, 2017 9:53 AM
To: McLachlan, Andrew <Andrew.McLachlan@xxxxxxxxxx>
Cc: Yee, Norman <Norman.Yee@xxxxxxxxxx>; cdt-dev@xxxxxxxxxxx
Subject: Re: memory browser reviews - cdt-dev Digest, Vol 141, Issue 6

In Bug 338545, Alvaro has referred to internal code at Ericsson.

Alvaro, would you be able to open source the code mentioned [1]?
Perhaps if it was open sourced Andrew and Norman could adapt it for general purpose and contribute that back to CDT?

Either way we can continue the discussion on the bug.

[1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=338545#c14


~~~
Jonah Graham
Kichwa Coders Ltd.
www.kichwacoders.com


On 4 April 2017 at 18:45, McLachlan, Andrew <Andrew.McLachlan@xxxxxxxxxx> wrote:
>
>  Hi Jonah,
>
>  Thanks for reviewing our patches and providing feedback. We've provided updates for the bug fixes:
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=338545
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=338589
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=339289
>
https://bugs.eclipse.org/bugs/show_bug.cgi?id=338545 seems to be the only outstanding issue and I'm wondering what the best way is to push this one forward.
>
>  >> https://wiki.eclipse.org/CDT/calls/Apr2017 : existing committers need to help newcomers and review patches and answer questions.
>
>  Any advice and help that you can offer us to speed up our patch acceptance etc. would be greatly appreciated. We're more than willing to contribute.
>
>  Thanks in advance,
>
>  Andrew (McLachlan)
>  Analog Devices
>
> -----Original Message-----
> From: Jonah Graham [mailto:jonah@xxxxxxxxxxxxxxxx]
> Sent: Tuesday, November 08, 2016 10:55 PM
> To: McLachlan, Andrew; Norman Yee
> Subject: Re: FW: cdt-dev Digest, Vol 141, Issue 6
>
> Hi Andrew and Norm,
>
> I am happy to review them. Thanks for bringing them up to date. I have
> started looking at them, but as we are one week out from RC1 of CDT
> 9.2 and I have a couple of critical things to get in (Bug 494246 is my current focus), it may wait a touch longer. Please ping me on or off list if you want an update of when I can get to them.
>
> Jonah
> ~~~
> Jonah Graham
> Kichwa Coders Ltd.
> www.kichwacoders.com
>
>
> On 4 November 2016 at 16:26, McLachlan, Andrew <Andrew.McLachlan@xxxxxxxxxx> wrote:
>>
>> Hi Jonah,
>>
>> Thanks for assisting Norm with contributing these memory browser
>> fixes. After speaking to you guys at EclipseCon Europe 2016, we
>> thought we'd try out the process :)
>>
>> Andrew (McLachlan)
>>
>> -----Original Message-----
>> From: cdt-dev-bounces@xxxxxxxxxxx
>> [mailto:cdt-dev-bounces@eclipse.org]
>> On Behalf Of cdt-dev-request@xxxxxxxxxxx
>> Sent: 04 November 2016 16:00
>> To: cdt-dev@xxxxxxxxxxx
>> Subject: cdt-dev Digest, Vol 141, Issue 6
>>
>> Send cdt-dev mailing list submissions to
>>         cdt-dev@xxxxxxxxxxx
>>
>> To subscribe or unsubscribe via the World Wide Web, visit
>>         https://dev.eclipse.org/mailman/listinfo/cdt-dev
>> or, via email, send a message with subject or body 'help' to
>>         cdt-dev-request@xxxxxxxxxxx
>>
>> You can reach the person managing the list at
>>         cdt-dev-owner@xxxxxxxxxxx
>>
>> When replying, please edit your Subject line so it is more specific than "Re: Contents of cdt-dev digest..."
>>
>>
>> Today's Topics:
>>
>>    1. Re: Gdb Full Console request (Marc Khouzam)
>>    2. User Questions (John Eblen)
>>    3. Re: User Questions (Doug Schaefer)
>>    4. Re: Memory browser fixes (Jonah Graham)
>>
>>
>> ---------------------------------------------------------------------
>> -
>>
>> Message: 1
>> Date: Thu, 3 Nov 2016 16:52:32 +0000
>> From: Marc Khouzam <marc.khouzam@xxxxxxxxxxxx>
>> To: CDT General developers list. <cdt-dev@xxxxxxxxxxx>
>> Subject: Re: [cdt-dev] Gdb Full Console request
>> Message-ID: <8cbe4479-77fc-4f25-a25e-a913ff32baad@xxxxxxxxxxxx>
>> Content-Type: text/plain; charset="us-ascii"
>>
>> Hi Stephen,
>>
>> There are currently no plans to have the equivalent of a ConsolePageParticipant, but I would like there to be one :). We are focusing on making the new console experience smooth for users in this last stretch before the release, and are not left with much time to improve things for extenders.
>>
>> My guess is that it wouldn't be too hard to support, by copying the current support from the platform console for such a ConsolePageParticipant.  Your best bet is to submit a patch for this.
>>
>> As for adding an ID to the context menu, of course we should do that. I'll do my best to find time to do it. Again, the fastest way to get this done is to submit a patch that you know will satisfy your requirement.
>>
>> Thanks for your interest
>>
>> Marc
>>
>> ________________________________
>> From: "Flynn, Stephen" <Stephen.Flynn@xxxxxxxx>
>> Sent: Nov 2, 2016 3:40 PM
>> To: CDT General developers list.
>> Subject: [cdt-dev] Gdb Full Console request
>>
>> Are there plans to allow page participants with the new console?  If not, could doing so be considered?
>>
>> My ask here is based on a requirement to add pattern matching.
>>
>> Also, will the context menu be registered with an Id?  We have a couple custom menu items that I would like to add.
>>
>> Thanks,
>>
>> Stephen Flynn
>> Software Engineer 2
>> 508-249-9848 / (821) 49848
>>
>>
>> _______________________________________________
>> cdt-dev mailing list
>> cdt-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or
>> unsubscribe from this list, visit
>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>> -------------- next part -------------- An HTML attachment was
>> scrubbed...
>> URL:
>> <https://dev.eclipse.org/mailman/private/cdt-dev/attachments/20161103
>> /
>> 16e93269/attachment.html>
>>
>> ------------------------------
>>
>> Message: 2
>> Date: Fri, 4 Nov 2016 11:31:08 -0400
>> From: John Eblen <jeblen@xxxxxxx>
>> To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
>> Subject: [cdt-dev] User Questions
>> Message-ID:
>>
>> <CANxFD_mfAUC5=y_b0dAcDA03wkSgtkBuxfqB0bMVGeOhj27RZw@xxxxxxxxxxxxxx>
>> Content-Type: text/plain; charset="utf-8"
>>
>> Hi
>>
>>
>> Occasionally, people will ask questions on ptp-user that are specific to CDT.
>>
>> Where should we redirect them, since there is no cdt-user mailing list?
>>
>>
>> Thanks
>> John Eblen
>> -------------- next part -------------- An HTML attachment was
>> scrubbed...
>> URL:
>> <https://dev.eclipse.org/mailman/private/cdt-dev/attachments/20161104
>> /
>> 08aee73d/attachment.html>
>>
>> ------------------------------
>>
>> Message: 3
>> Date: Fri, 4 Nov 2016 15:40:43 +0000
>> From: Doug Schaefer <dschaefer@xxxxxxxxxxxxxx>
>> To: CDT General developers list. <cdt-dev@xxxxxxxxxxx>
>> Subject: Re: [cdt-dev] User Questions
>> Message-ID: <D4422410.2182D%dschaefer@blackberry.com>
>> Content-Type: text/plain; charset="windows-1252"
>>
>> There is a pretty active forum for CDT. That?s where a lot of users hang out.
>>
>> https://www.eclipse.org/forums/index.php/f/80/
>>
>> Doug.
>>
>> From:
>> <cdt-dev-bounces@xxxxxxxxxxx<mailto:cdt-dev-bounces@eclipse.org>> on
>> behalf of John Eblen <jeblen@xxxxxxx<mailto:jeblen@acm.org>>
>> Reply-To: "CDT General developers list."
>> <cdt-dev@xxxxxxxxxxx<mailto:cdt-dev@xxxxxxxxxxx>>
>> Date: Friday, November 4, 2016 at 11:31 AM
>> To: "CDT General developers list."
>> <cdt-dev@xxxxxxxxxxx<mailto:cdt-dev@xxxxxxxxxxx>>
>> Subject: [cdt-dev] User Questions
>>
>> Hi
>>
>>
>> Occasionally, people will ask questions on ptp-user that are specific to CDT.
>>
>> Where should we redirect them, since there is no cdt-user mailing list?
>>
>>
>> Thanks
>> John Eblen
>> -------------- next part -------------- An HTML attachment was
>> scrubbed...
>> URL:
>> <https://dev.eclipse.org/mailman/private/cdt-dev/attachments/20161104
>> /
>> 11de4c49/attachment.html>
>>
>> ------------------------------
>>
>> Message: 4
>> Date: Fri, 4 Nov 2016 15:42:59 +0000
>> From: Jonah Graham <jonah@xxxxxxxxxxxxxxxx>
>> To: Norman Yee <normankyee@xxxxxxxxx>,  "CDT General developers list."
>>         <cdt-dev@xxxxxxxxxxx>
>> Subject: Re: [cdt-dev] Memory browser fixes
>> Message-ID:
>>
>> <CAPmGMvjmwkgt11GeUOLUn9GgRk_uMMoHQR7KUekdiwBiKBPkvg@mail.gmail.com>
>> Content-Type: text/plain; charset=UTF-8
>>
>> Hi Norman,
>>
>> Thanks for coming back to the list on these items. I would be pleased to review them. However, they are currently very out of date (they were last updated crica 2011). The patches need be be created as Gerrits against the current CDT master:
>> https://wiki.eclipse.org/CDT/contributing
>>
>> Once you have made the gerrits, please add me to the Reviewers list so I can have a look.
>>
>> Thanks,
>> Jonah
>>
>> ~~~
>> Jonah Graham
>> Kichwa Coders Ltd.
>> www.kichwacoders.com
>>
>>
>> On 3 November 2016 at 14:41, Norman Yee <normankyee@xxxxxxxxx> wrote:
>>> Hi all,
>>>
>>> A few years ago, I posted some fixes to the memory browser:
>>>
>>> view doesn't update contents of memory tabs when debug context
>>> changes
>>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=338545
>>>
>>> view doesn't persist list of memory tabs in launch configuration
>>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=338589
>>>
>>> need a way to change the rendering type of the active tab
>>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=339289
>>>
>>> I was wondering if someone could review the fixes and commit them if
>>> they're acceptable?
>>>
>>> Thanks,
>>> Norman Yee
>>>
>>>
>>> _______________________________________________
>>> cdt-dev mailing list
>>> cdt-dev@xxxxxxxxxxx
>>> To change your delivery options, retrieve your password, or
>>> unsubscribe from this list, visit
>>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>>
>>
>> ------------------------------
>>
>> _______________________________________________
>> cdt-dev mailing list
>> cdt-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or
>> unsubscribe from this list, visit
>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>>
>> End of cdt-dev Digest, Vol 141, Issue 6
>> ***************************************
>>
>

_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev


Back to the top