Quantcast

performApply does not save at BinaryParsTab class

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

performApply does not save at BinaryParsTab class

ilke Muhtaroglu
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
[hidden email]
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: performApply does not save at BinaryParsTab class

Jonah Graham
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 <[hidden email]> 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
[hidden email]
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
[hidden email]
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: performApply does not save at BinaryParsTab class

ilke Muhtaroglu
In reply to this post by ilke Muhtaroglu

İ 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, <[hidden email]> wrote:
Send cdt-dev mailing list submissions to
        [hidden email]

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
        [hidden email]

You can reach the person managing the list at
        [hidden email]

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 <[hidden email]>
To: "CDT General developers list." <[hidden email]>
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 | [hidden 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 <[hidden email]>
To: [hidden email]
Subject: [cdt-dev] performApply does not save at BinaryParsTab class
Message-ID:
        <CAAAoW5-=[hidden email]>
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 <[hidden email]>
To: "CDT General developers list." <[hidden email]>
Subject: Re: [cdt-dev] performApply does not save at BinaryParsTab
        class
Message-ID:
        <[hidden email]>
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 <[hidden email]>
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
> [hidden email]
> 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
[hidden email]
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
[hidden email]
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: performApply does not save at BinaryParsTab class

Jonah Graham
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 <[hidden email]> 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, <[hidden email]> wrote:
>>
>> Send cdt-dev mailing list submissions to
>>         [hidden email]
>>
>> 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
>>         [hidden email]
>>
>> You can reach the person managing the list at
>>         [hidden email]
>>
>> 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 <[hidden email]>
>> To: "CDT General developers list." <[hidden email]>
>> 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 | [hidden 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 <[hidden email]>
>> To: [hidden email]
>> Subject: [cdt-dev] performApply does not save at BinaryParsTab class
>> Message-ID:
>>
>> <CAAAoW5-=[hidden email]>
>> 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 <[hidden email]>
>> To: "CDT General developers list." <[hidden email]>
>> Subject: Re: [cdt-dev] performApply does not save at BinaryParsTab
>>         class
>> Message-ID:
>>
>> <[hidden email]>
>> 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 <[hidden email]>
>> 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
>> > [hidden email]
>> > 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
>> [hidden email]
>> 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
> [hidden email]
> 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
[hidden email]
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev
Loading...