CCS/SIMPLELINK-CC2640R2-SDK: Code indentation really not okay?

Markel Robregado

Part Number: SIMPLELINK-CC2640R2-SDK

Tool/software: Code Composer Studio

Hi,

At work we follow coding guidelines and I would need to correct all code indentation at all code files, which is really tedious. See, below sample code snippet at Simple Peripheral SDK v1.50. Is there any settings at CCS that when I import the CCS project all code indentation is proper?

static void SimpleBLEPeripheral_processAppMsg(sbpEvt_t *pMsg)
{
  switch (pMsg->hdr.event)
  {
    case SBP_STATE_CHANGE_EVT:
      {
        SimpleBLEPeripheral_processStateChangeEvt((gaprole_States_t)pMsg->
                                                hdr.state);
      }
      break;

    case SBP_CHAR_CHANGE_EVT:
      {
        SimpleBLEPeripheral_processCharValueChangeEvt(pMsg->hdr.state);
      }
      break;

    // Pairing event
    case SBP_PAIRING_STATE_EVT:
      {
        SimpleBLEPeripheral_processPairState(pMsg->hdr.state, *pMsg->pData);

        ICall_free(pMsg->pData);
        break;
      }

    // Passcode event
    case SBP_PASSCODE_NEEDED_EVT:
      {
        SimpleBLEPeripheral_processPasscode(*pMsg->pData);

        ICall_free(pMsg->pData);
        break;
      }

-kel

over 5 years ago

0 TER over 5 years ago

TI__Guru**** 317180 points

In did a google search for "code indent CCS" and found these posts that could get you started:
e2e.ti.com/.../243604
e2e.ti.com/.../585415
e2e.ti.com/.../341445
e2e.ti.com/.../575521

0 Markel Robregado over 5 years ago in reply to TER

Guru 85695 points

Hi TER,

Thanks for sharing links that will help me fix the code indentation at all code files.

Here is another example there are spaces at the side of ( void ), while other C functions are normal, meaning no spaces in function arguments.

static void SimpleBLEPeripheral_init( void );
static void SimpleBLEPeripheral_taskFxn(UArg a0, UArg a1);

static uint8_t SimpleBLEPeripheral_processStackMsg(ICall_Hdr *pMsg);
static uint8_t SimpleBLEPeripheral_processGATTMsg(gattMsgEvent_t *pMsg);
static void SimpleBLEPeripheral_processAppMsg(sbpEvt_t *pMsg);

-kel

0 TER over 5 years ago in reply to Markel Robregado

TI__Guru**** 317180 points

For questions regarding CCS specific items, use the CCS forum.

0 Markel Robregado over 5 years ago in reply to TER

Guru 85695 points

The inconsistent code indentation and spacing are related to SDK. So my first question "Is there any settings at CCS that when I import the CCS project all code indentation is proper?" has no answer. There is no CCS setting solution.

I will leave this post as unanswered. I just like to inform the TI Engineers responsible for the code files to at least do some indentation and spacing correction at future SDK releases.

As I have mentioned at work we follow coding guidelines After correcting the all code files indentation and spacing, we do regression testing. So it just add to work load.

-kel

0 TER over 5 years ago in reply to Markel Robregado

TI__Guru**** 317180 points

I sent a mail to the SDK team on this, I haven't gotten a feedback so far.

Bluetooth®︎

Bluetooth forum

CCS/SIMPLELINK-CC2640R2-SDK: Code indentation really not okay?