Author Topic: The 16 November 2013 build (9455) is out.  (Read 84375 times)

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 16 November 2013 build (9455) is out.
« Reply #30 on: November 22, 2013, 06:00:20 am »
I have tried this and apart from the old problem [...]
Did the setting resolve your old problem?

[...] a new issue appeared; it displays the same available keyword twice and still insists to concatenate the word right after colon.
My apologies, but I do not understand your description.  Could you try to explain it again?

ToApolytoXaos

  • Guest
Re: The 16 November 2013 build (9455) is out.
« Reply #31 on: November 22, 2013, 07:24:09 am »
Did the setting resolve your old problem?
Nope, I'm afraid I haven't.

My apologies, but I do not understand your description.  Could you try to explain it again?
I did what you have suggested (with Fill up characters) and now it duplicates the goto label twice in auto-completion popped up window while the old issue remains the same.

Offline killerbot

  • Administrator
  • Lives here!
  • *****
  • Posts: 5195
Re: The 16 November 2013 build (9455) is out.
« Reply #32 on: November 22, 2013, 07:42:55 am »
"-Wno-unused-local-typedefs" , yes, ensure for sure that one can remain, cause this one is a warning exploder for wx, boost , ...

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 5247
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
Re: The 16 November 2013 build (9455) is out.
« Reply #33 on: November 22, 2013, 04:05:02 pm »
I see this error: (see screen shot below)


It happens when I open a simple wx2.8.12 project(from wx wizard), or press the build button.



Have any CB project in the past used two scripts in the Compiler other options before?

I think two does NOT work; I deleted either one and the error went away on the next project re-build.

Tim S.

Thank Tim, indeed, I see that the compiler options has such code:
Code: [Select]
-pipe
-mthreads
[[if (PLATFORM == PLATFORM_MSW && (GetCompilerFactory().GetCompilerVersionString(_T("gcc")) >= _T("4.0.0"))) print(_T("-Wno-attributes"));]]
[[if (GetCompilerFactory().GetCompilerVersionString(_T("gcc")) >= _T("4.8.0")) print(_T("-Wno-unused-local-typedefs"));]]
-Winvalid-pch
-include wx_pch.h

@Alpha:
I don't know the reason we need "-Wno-attributes" options.
I do know that "-Wno-unused-local-typedefs" was used to remove build warnings.


If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 16 November 2013 build (9455) is out.
« Reply #34 on: November 22, 2013, 09:56:23 pm »
-Wno-attributes came from rev 4596:
Code: [Select]
http://cb.biplab.in/websvn/log.php?repname=codeblocks&path=%2F&isdir=1&peg=9462&sr=4596&er=1&max=1&search=
I have removed -Wno-attributes from the wizard in trunk now, as -Wno-unused-local-typedefs looks to be currently a much more useful command.

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: The 16 November 2013 build (9455) is out.
« Reply #35 on: November 22, 2013, 10:06:42 pm »
-Wno-attributes came from rev 4596:
Code: [Select]
http://cb.biplab.in/websvn/log.php?repname=codeblocks&path=%2F&isdir=1&peg=9462&sr=4596&er=1&max=1&search=
I have removed -Wno-attributes from the wizard in trunk now, as -Wno-unused-local-typedefs looks to be currently a much more useful command.

Why don't we have check boxes for these -W flags in compiler settings?

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9508
Re: The 16 November 2013 build (9455) is out.
« Reply #36 on: November 22, 2013, 10:44:24 pm »
Why don't we have check boxes for these -W flags in compiler settings?
Seems nobody configured it in the related XML compiler settings.
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: http://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: http://wiki.codeblocks.org/index.php?title=FAQ

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 16 November 2013 build (9455) is out.
« Reply #37 on: November 25, 2013, 04:21:45 am »
Why don't we have check boxes for these -W flags in compiler settings?
Seems nobody configured it in the related XML compiler settings.
During the conversion to XML, I pretty much tried to do a straight copy of what existed.  My personal opinion is that there are already too many flags, and that uncommon/special case flags are better suited to be manually entered.
Do note that the GUI does support Right click->New flag, so if you want your local version to have more flags, it is as easy as that.

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9508
Re: The 16 November 2013 build (9455) is out.
« Reply #38 on: November 25, 2013, 07:30:03 am »
Do note that the GUI does support Right click->New flag, so if you want your local version to have more flags, it is as easy as that.
Sure thing, but as (at least) -Wno-unused-local-typedefs is relevant to wxWidgets it should be natively supported.
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: http://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: http://wiki.codeblocks.org/index.php?title=FAQ

ToApolytoXaos

  • Guest
Re: The 16 November 2013 build (9455) is out.
« Reply #39 on: November 25, 2013, 02:30:00 pm »
I went to Create a new project, right-click on GTK+ project, then Edit this script. The file opens in new tab window; I switch from Code::Blocks to Firefox using Alt+TAB, I come back to Code::Blocks and I get



Is this normal, or I hit a bug?

P.S.: Special thanks goes to @oBFusCATed for his reply to this request; if I have found a bug, he should get the credit for this :D

http://forums.codeblocks.org/index.php/topic,18620.msg127477.html#msg127477

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 12129
    • Travis build status
Re: The 16 November 2013 build (9455) is out.
« Reply #40 on: November 25, 2013, 02:47:44 pm »
Is this normal, or I hit a bug?
Yes, I think it is normal, but I don't know if this is a good behaviour.
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

ToApolytoXaos

  • Guest
Re: The 16 November 2013 build (9455) is out.
« Reply #41 on: November 28, 2013, 02:45:34 pm »
Currently compiled svn9468 on Windows 7 machine (at work) and auto-completion does not work properly.

As soon as I insert (*prd->st_info)->, instead of seeing the desired variables, I see whatever prd
has in it.

See the entire sample below:

Code: C
  1. /*
  2.  * Placed in Public Domain where applicable permitted by law.
  3.  * USE IT AT YOUR OWN RISK!
  4.  */
  5. #include <stdio.h>
  6. #include <stdlib.h>
  7.  
  8. #define nil ((void *)0)
  9.  
  10. typedef struct {
  11.     int postal_code;
  12.     const char *address;
  13. } street_info_t;
  14.  
  15. typedef struct {
  16.     street_info_t **st_info;
  17.     const char *company_name;
  18. } generic_t;
  19.  
  20. street_info_t * str_constructor(void) {
  21.     street_info_t *tmp = (street_info_t *) malloc(sizeof(street_info_t));
  22.     if (tmp == nil) {
  23.         return nil;
  24.     }
  25.     return tmp;
  26. }
  27.  
  28. void str_destructor(street_info_t **self) {
  29.     if (*self != nil) {
  30.         free(*self);
  31.         *self = nil;
  32.     }
  33.     return;
  34. }
  35.  
  36. generic_t * constructor(void) {
  37.     generic_t *tmp = (generic_t *) malloc(sizeof(generic_t));
  38.     if (tmp == nil) {
  39.         return nil;
  40.     }
  41.     return tmp;
  42. }
  43.  
  44. void destructor(generic_t **self) {
  45.     if (*self != nil) {
  46.         free(*self);
  47.         *self = nil;
  48.     }
  49.     return;
  50. }
  51.  
  52. int main(void)
  53. {
  54.     generic_t *prd = constructor();
  55.     street_info_t *street = str_constructor();
  56.  
  57.     prd->st_info = &street;
  58.     prd->company_name = "Disney Land";
  59.     (*prd->st_info)->address = "Mikey Mouse avenue, Neverland";
  60.     (*prd->st_info)->postal_code = 1010;
  61.  
  62.     printf("Company Name: %s\n", prd->company_name);
  63.     printf("Address: %s\n", (*prd->st_info)->address);
  64.     printf("Postal code: %d\n", (*prd->st_info)->postal_code);
  65.  
  66.     str_destructor(&street);
  67.     prd->st_info = nil;
  68.  
  69.     destructor(&prd);
  70.  
  71.     return 0;
  72. }
  73.  

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 12129
    • Travis build status
Re: The 16 November 2013 build (9455) is out.
« Reply #42 on: November 28, 2013, 02:47:05 pm »
Is this a regression in this build?
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 5247
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
Re: The 16 November 2013 build (9455) is out.
« Reply #43 on: November 28, 2013, 03:13:28 pm »
@ToApolytoXaos, If I remember correctly, we don't handle the parentheses correctly in CC. (It need some kind of operator precedence parsing of the statement, but currently we don't have this feature)
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.

ToApolytoXaos

  • Guest
Re: The 16 November 2013 build (9455) is out.
« Reply #44 on: November 28, 2013, 03:15:02 pm »
Ah OK, no worries. So it's not a bug, but a lack of feature :) nice. Also, should not be logical to offer only C keywords if you use a project as C, unless it's used in C++ as extern C? Most of the times I get full list of unnecessary keyword tags.
« Last Edit: November 28, 2013, 03:16:54 pm by ToApolytoXaos »