Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] performApply does not save at BinaryParsTab class 
Hi ilke,

Thanks for filing the bug report. I have reproduced it with the extra
information provided. Lets continue the discussion in the bug you
raised, Bug 515876.

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


On 27 April 2017 at 07:46, ilke Muhtaroglu <ilke.muhtaroglu@xxxxxxxxx> wrote:
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=515876
>
> İ issued a bug report. Because I can reproduce this at cdt 8.6 in win  XP
> and win 7
>
>
>
>
> On Wed, Apr 26, 2017 at 4:23 PM, <cdt-dev-request@xxxxxxxxxxx> wrote:
>>
>> 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: CDT Website (Mirko Stocker)
>>    2. performApply does not save at BinaryParsTab class
>>       (ilke Muhtaroglu)
>>    3. Re: performApply does not save at BinaryParsTab class
>>       (Jonah Graham)
>>
>>
>> ----------------------------------------------------------------------
>>
>> Message: 1
>> Date: Tue, 25 Apr 2017 20:51:41 +0200
>> From: Mirko Stocker <me@xxxxxxxx>
>> To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
>> Subject: Re: [cdt-dev] CDT Website
>> Message-ID: <1590157.DVxB2pk1pP@francis>
>> Content-Type: text/plain; charset="us-ascii"
>>
>> Hi all,
>>
>> > If you like markdown and Git, Jekyll might be for you too. Highly
>> > recommended!
>>
>> Agreed, it's also what we use for Cevelop: http://cevelop.com/ and the
>> code
>> https://github.com/Cevelop/cevelop.github.io
>>
>> You can also do things like blog posts (our news) fairly easily, for
>> comments,
>> we just use Disqus.
>>
>> Cheers
>>
>> Mirko
>>
>> --
>> Mirko Stocker | mirko@stocker.email
>> Work: http://ifs.hsr.ch | http://infoq.com
>> Personal: http://misto.ch | http://twitter.com/m_st
>>
>>
>> ------------------------------
>>
>> Message: 2
>> Date: Wed, 26 Apr 2017 15:12:26 +0300
>> From: ilke Muhtaroglu <ilke.muhtaroglu@xxxxxxxxx>
>> To: cdt-dev@xxxxxxxxxxx
>> Subject: [cdt-dev] performApply does not save at BinaryParsTab class
>> Message-ID:
>>
>> <CAAAoW5-=yVJu-6Dir6Dg_vFQhU463Eup1RxY0RkqK1GR-91+0A@xxxxxxxxxxxxxx>
>> Content-Type: text/plain; charset="utf-8"
>>
>> I browse and select new values at C build -  Settings -  Binary Parsers
>> via
>> Binary Parser Options as a command.
>>
>> But these selected values are can not be saved.
>>
>> At the BinaryParsTab there is a comment. So these BinaryParserPages are
>> can
>> not be saved at the moment as well ...
>>
>>
>> /**
>> * @noextend This class is not intended to be subclassed by clients.
>> */
>> public class BinaryParsTab extends AbstractCPropertyTab {
>> /* Settings from binary parser pages are NOT saved by prior CDT version.
>> * So existing binary parsers _always_ use default values.
>> * Moreover, obsolete interface is used while attempting to save.
>> *
>> * We have to affect both parsers and pages
>> * to teach them to save data really.
>> *
>> * It will be done in next versions. Currently pages are disabled.
>> */
>> -------------- next part --------------
>> An HTML attachment was scrubbed...
>> URL:
>> <https://dev.eclipse.org/mailman/private/cdt-dev/attachments/20170426/fd749ec9/attachment.html>
>>
>> ------------------------------
>>
>> Message: 3
>> Date: Wed, 26 Apr 2017 14:22:33 +0100
>> From: Jonah Graham <jonah@xxxxxxxxxxxxxxxx>
>> To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
>> Subject: Re: [cdt-dev] performApply does not save at BinaryParsTab
>>         class
>> Message-ID:
>>
>> <CAPmGMvgg9iq7McXLGWjLEiu29SLd0-MFTMaaZQnwLCXdPRXL_Q@xxxxxxxxxxxxxx>
>> Content-Type: text/plain; charset="utf-8"
>>
>>
>> I am not 100% sure what the comment refers to. My *guess* is that it is
>> referring to the UI that existed before the "newui". If so that comment
>> means: Settings from binary parser pages are not saved in CDT versions
>> before CDT 4.0, so parsers settings saved in CDT versions <= 4.0 always
>> use
>> default values... Note the pre CDT 4.0 version of the page
>> is org.eclipse.cdt.ui.dialogs.BinaryParserBlock, the "newui" is the one
>> you
>> reference, note the newui in the package
>> name: org.eclipse.cdt.ui.newui.BinaryParsTab
>>
>> Therefore, if you could explain what specifically is failing, and if you
>> have a reproducible test case put it in a bug report
>> https://bugs.eclipse.org/bugs/enter_bug.cgi?product=CDT. If you also
>> provide a fix I would be happy to review it.
>>
>> Jonah
>>
>>
>> ~~~
>> Jonah Graham
>> Kichwa Coders Ltd.
>> www.kichwacoders.com
>>
>> On 26 April 2017 at 13:12, ilke Muhtaroglu <ilke.muhtaroglu@xxxxxxxxx>
>> wrote:
>>
>> > I browse and select new values at C build -  Settings -  Binary Parsers
>> > via Binary Parser Options as a command.
>> >
>> > But these selected values are can not be saved.
>> >
>> > At the BinaryParsTab there is a comment. So these BinaryParserPages are
>> > can not be saved at the moment as well ...
>> >
>> >
>> > /**
>> > * @noextend This class is not intended to be subclassed by clients.
>> > */
>> > public class BinaryParsTab extends AbstractCPropertyTab {
>> > /* Settings from binary parser pages are NOT saved by prior CDT version.
>> > * So existing binary parsers _always_ use default values.
>> > * Moreover, obsolete interface is used while attempting to save.
>> > *
>> > * We have to affect both parsers and pages
>> > * to teach them to save data really.
>> > *
>> > * It will be done in next versions. Currently pages are disabled.
>> > */
>> >
>> > _______________________________________________
>> > 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/20170426/cb230664/attachment.html>
>>
>> ------------------------------
>>
>> _______________________________________________
>> 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 146, Issue 23
>> ****************************************
>
>
>
> _______________________________________________
> 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