Help, SE 17.0.1 Stopped inserting { } for c++

[lwb78750]

This may be a stupid question, but...

In SE 16 if I had the following syntax:
    if(foo)
       bar = 10;

I could just enter (I think it was) crtrl-shift-{ in front of the "bar = 10;" and the result would be this:
    if(foo)
    {
        bar = 10;
    }

Now, that doesn't seem to work anymore (or I can't figure out how to get it to work.)

Does anyone know how to get this to work in 17.0.1?

Thanks!

[patrick]

Is this C/C++?  If so, just hitting "{" there should do what you want, assuming the "{" key hasn't been rebound to something other than c-begin.  Let me know if that doesn't work for you.

[lwb78750]

Yes, it's not ctrl-shift-{, its just {, but that doesn't work either.
I installed 17.0.1 and I get this:

    if(foo)
    {}
    bar = 20;

thanks,

[davehohl]

I've noticed this problem also, although I am not sure if it happened when I installed 17.0.0 or 17.0.1.

[patrick]

Strange.  There is a fix that will be in the first 17.0.1 hotfix that takes care of the problem where both of the braces get put on the same line.  But the only way I can reproduce your problem is line below the if isn't indented past the 'if' column  ie,

Code: [Select]

if (foo)
bar=20;

When "bar=20" is indented, it works for me in 17.0.1.  Maybe I'm missing something in the configuration that triggers it, I'll take another look through the code.

[davehohl]

I tried to reproduce the problem, but it seems to be working for me now, as long as I am writing "normal" code. However, I discovered a case where the problem does occur -- when the 'if' is within a block that is not part of a function. The following sample code illustrates 3 scenarios, 2 of which work and 1 which does not. Perhaps this can help you track down the source of the issue.


int u;

// *** frestanding if (not inside any function or block) works
    if(1)
        ++u;
// becomes
    if(1)
    {
        ++u;
    }

// *** if within a freestanding block does not work
{
    if(1)
        ++u;
}
// becomes
{
    if(1)
{}        ++u;
}

// *** if within a function works
void x(void)
{
    if(1)
        ++u;
}
// becomes
void x(void)
{
    if(1)
    {
        ++u;
    }
}

[patrick]

Ok, I can reproduce that, in both 16 and 17.  I'll take a look at it.

[lwb78750]

This is the results I get.  Is there some sort of config option3I have set/not set?

// *** frestanding if (not inside any function or block) works
    if(1)
        ++u;
// becomes
    if(1)
        {}++u;
// *** if within a freestanding block does not work
{
    if(1)
       ++u;
}
// becomes
{
    if(1)
        {}++u;
}

// *** if within a function works
void x(void)
{
    if(1)
        ++u;
}
// becomes
void x(void)
{
    if(1)
        {}++u;
}
   
   
   

[patrick]

Actually, there is a quick-brace setting, but I can't find it in the UI for C/C++.  Having it turned off would give the results you're seeing.

While I try to figure out what happened to the UI for it, you can try this:  Save off the attached macro file, quickbrace.e, open it up in slickedit, and then bring up the command prompt, and run "load".  That should make sure quick-brace is enabled in your configuration.  See if it works after that.

[Phil Barila]

Actually, there is a quick-brace setting, but I can't find it in the UI for C/C++.

This would kind of suggest that SE is approaching undiscoverable.

[patrick]

The setting isn't in the UI for C/C++, it got left out for C++ and Objective-C in v17.  I'll put that back in, it will be in a hotfix sometime soon.

[ExtremeXS]

I am also getting this issue after switching from 17.0.0.8 to 17.0.1.

Do you think the missing config option is being reset after importing the config across versions and needs resetting?

Cheers

[patrick]

I'm not sure yet.  I'm doing installs now to do a 17.0.0.8 -> 17.0.1 upgrade now to see if I can catch it happening.

[ExtremeXS]

I also seemed to loose a 'Tick' on the 'Documents/Indent with Tabs' setting when I tried switched my current project to 17.0.1.  It *may* be another example of a C/C++ setting loss.

Cheers