CCS C Software and Maintenance Offers
FAQFAQ   FAQForum Help   FAQOfficial CCS Support   SearchSearch  RegisterRegister 

ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

CCS does not monitor this forum on a regular basis.

Please do not post bug reports on this forum. Send them to support@ccsinfo.com

Const struct

 
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion
View previous topic :: View next topic  
Author Message
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

Const struct
PostPosted: Thu May 26, 2016 11:12 am     Reply with quote

Hi all

I use this kind of code with older CCS compiler (<5.50) without any issue.
But with latest version 5.59 I get no error but the code is not working any more.

I point out the default at this point
Quote:
const struct TypeMenu MenuTest[ALL_MENU]

The const struct is not working just struct is working with the last version of ccs.
Is this a compiler error or did I miss something?
Code:

#include <18f46k20.h>
#use delay(clock=16000000)
#fuses INTRC_IO, NOWDT, PUT, NOPROTECT, NOBROWNOUT
#use fast_io(c)
#use fast_io(d)

#include "HDM64GS12.C"   // driver GLCD
#include "GRAPHICS.C"   // graphics

#define VAL_TRIS_C 0b00000000   // PORTC
#define VAL_TRIS_E 0b00000000

#define UP     PIN_A1
#define DOWN   PIN_A2
#define ENTER  PIN_A3
#define POWER  PIN_A4   //
#define CANCEL PIN_C0
#define BACKLIGHT  PIN_E2   //

/**************************************************************************//**
 * Def menu
 ******************************************************************************/
#define MENU      0      // defaut start
#define ALL_MENU    4
//-----------------------------------------------------------------------------
// Tree menu
//-----------------------------------------------------------------------------
enum ids{   _BEGIN,
            _MAIN,
            _MAIN1,
                _MAIN11,
                _MAIN12
        };
       
struct TypeMenu {
             int8 MenuID;
             int8 SubMenu;
             char MenuName[18];
            };
 
int8 ContainMenu, STATE = _BEGIN, PAGE = 0, LEVEL[7], LevelPtr=0;
Boolean  AccessENTER = TRUE, AccessCANCEL=TRUE;

//-------------------------------------------
//const struct TypeMenu MenuTest[ALL_MENU]
struct TypeMenu MenuTest[ALL_MENU]
   ={
        {_MAIN, _BEGIN,"Menu1"},
        {_MAIN1,_BEGIN,"Menu2"},
            {_MAIN11,_MAIN1,"Menu2 sub1"},
            {_MAIN12,_MAIN1,"Menu2 sub2"}
   };
struct TypeMenu MenuPointer[2];

//---------------------------------------
unsigned char SelectFrom(int8 from) {
    int8 i, NumMenu = 0,k=0, i0;               
    for(i=0; i<ALL_MENU; i++) {
        if(MenuTest[i].SubMenu == STATE) {
            if(NumMenu == 0) i0 = i;
                NumMenu++;
            if(NumMenu>from && k < 2 ) { //  if(NumMenu>from && k < 2)
                MenuPointer[k] =  MenuTest[i];
                k++;
            }
            if(k == 1) MenuPointer[1] = MenuTest[i0];
        }                                   
    }
    return NumMenu;
}
//----------------------------------------
int8 QueryLastMenu() {
    int8 i;                         
    for(i=0; i<ALL_MENU; i++) {     
        if(MenuTest[i].MenuID == MenuPointer[0].SubMenu) 
            return MenuTest[i].SubMenu;
    }
    return 0x00;
}
//----------------------------------------
short HasSubMenu() {
    char i;
    for(i=0; i<ALL_MENU; i++) {           
        if(STATE == MenuTest[i].SubMenu) 
            return TRUE;                     
    }                                     
    return FALSE;                         
}
/**************************************************************************//**
 * Init PIC
*****************************************************************************/
void PIC_init(void){
    set_tris_c(VAL_TRIS_C);        // graphic lcd control lines all output   
    set_tris_d(0x00);         //Set PortD to output
    set_tris_e(VAL_TRIS_E);
    //---- PIC init ------------------------------------------------
    setup_oscillator(OSC_16MHZ);
    enable_interrupts(GLOBAL);
}
/**************************************************************************//**
 * Fonction Menu
 *****************************************************************************/
void ProcessMenu() {
    if(PAGE > ContainMenu-1) PAGE = 0;
    if(PAGE == 0xFF) PAGE = ContainMenu - 2;
        ContainMenu = SelectFrom(PAGE);
    glcd_fillScreen(OFF);            //Display clear
    glcd_text57(3, 40, MenuPointer[0].MenuName, 1, ON);   
 }
/**************************************************************************//**
 * Fonction quit Menu
 *****************************************************************************/
void Quit(){
    if(!AccessCANCEL) break;
    if(LevelPtr) LevelPtr--;   
    PAGE = LEVEL[LevelPtr];   
    if(!HasSubMenu()) {       
        AccessENTER = TRUE; 
        STATE = MenuPointer[0].SubMenu;
    } else {                   
        STATE = QueryLastMenu();   
    }
    ContainMenu = SelectFrom(0);
    ProcessMenu();   
}
/**************************************************************************//**
 * Fonction Action Menu
 *****************************************************************************/
void ProcessAction(long Key) {
    int1 faux;
    faux = FALSE;
    if(STATE == _BEGIN) AccessCANCEL = FALSE; 
    else  AccessCANCEL = TRUE;               
    switch(Key) {
        case ENTER:
            if(!AccessENTER) break;       
            LEVEL[LevelPtr] = PAGE;   
            LevelPtr++;               
            STATE = MenuPointer[0].MenuID;
            PAGE = 0;                 
            ContainMenu = SelectFrom(0);
            if(ContainMenu) {
                 ProcessMenu();         
            }else {
                AccessENTER = FALSE;   
                switch(STATE){
                    case _MAIN:                   //Bat
                        glcd_fillScreen(OFF);
                        Delay_ms(1000);
                        Quit();
                    break;

                    case _MAIN1:       //Backlight ON
                        output_toggle(BACKLIGHT);   
                        Delay_ms(100);
                        Quit();
                    break;             
                }
            }
            break;
                   
            case UP:                   
                 if(!AccessENTER) break;
                 if(PAGE == 0) PAGE = ContainMenu;
                 PAGE --;               
                 ProcessMenu();
                 break;

            case DOWN:                 
                 if(!AccessENTER) break;
                 PAGE ++;               
                 ProcessMenu();
                 break;

            case CANCEL:                 
                if(!AccessCANCEL) break;
                if(LevelPtr) LevelPtr--;   
                PAGE = LEVEL[LevelPtr];   
                if(!HasSubMenu()) {       
                    AccessENTER = TRUE; 
                    STATE = MenuPointer[0].SubMenu;
                } else {                   
                    STATE = QueryLastMenu();   
                }
                    ContainMenu = SelectFrom(0);
                    ProcessMenu();         
                break;

            case MENU:
                ContainMenu = SelectFrom(0);
                ProcessMenu();
                Break; 
    }
}
//Main
void main()
{
    //Setup Pic
    PIC_init();
    //Setup LCD
    glcd_init(ON);
    glcd_fillScreen(OFF);            //   Display clear
    STATE = _BEGIN;
    processAction(MENU); 
   
   while(TRUE)
   {
        if(input(DOWN) == 1)                                     
            {
            while(input(DOWN) == 1) {}; // wait for releasing Button1
            processAction(DOWN);
            }
        if(input(UP) == 1)                                     
            {
            while(input(UP) == 1) {}; // wait for releasing Button1
            processAction(UP);
            }
        if(input(ENTER) == 0)                                     
            {
            while(input(ENTER) == 0) {}; // wait for releasing Button1
            processAction(ENTER);
            }
        if(input(CANCEL) == 1)                                     
            {
            while(input(CANCEL) == 1) {}; // wait for releasing Button1
            processAction(CANCEL);
            }
    }

}
asmboy



Joined: 20 Nov 2007
Posts: 2128
Location: albany ny

View user's profile Send private message AIM Address

PostPosted: Thu May 26, 2016 12:49 pm     Reply with quote

stuff like this is silly oddly-coded.

Code:

case DOWN: 
case CANCEL:

///..

that the argument driving the decisions is a pin number designator
is pretty unusual.


Last edited by asmboy on Thu May 26, 2016 3:32 pm; edited 1 time in total
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

PostPosted: Thu May 26, 2016 1:52 pm     Reply with quote

I use this https://www.ccsinfo.com/forum/viewtopic.php?p=123206 since a long time and it has always worked
PCM programmer



Joined: 06 Sep 2003
Posts: 21708

View user's profile Send private message

PostPosted: Thu May 26, 2016 6:58 pm     Reply with quote

Quote:
but the code is not working any more.

What did your code do when it was working correctly, and what is it doing now ?

Example:
"My program used to scroll through the menu when the up/down keys
were pressed. But now it only shows the first menu. The up/down keys
do not cause it to scroll".

or

"My program used to display the menu text. Now it displays garbage".
(Then post the text that it should display, and post the garbage text that
it now displays).

Give us a detailed explanation of how it's failing. Tell us what you see.
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

PostPosted: Fri May 27, 2016 5:52 am     Reply with quote

on a simple menu like on the example

My program used to scroll through the menu when the up/down keys
were pressed. But now it only shows the first menu. The up/down keys
do not cause it to scroll. It stays stuck at the first menu position. The first position menu is working when I press enter (clear screen and go back to menu after delay)

In more complicated menu key do not scroll and text is garbage.
working : Menu
default : ꟻꟻꟻꟻꟻꟻꟻꟻꟻꟻꟻꟻꟻꟻ

When I only declare
Quote:
struct TypeMenu MenuTest[ALL_MENU]
instead of
Quote:
const struct TypeMenu MenuTest[ALL_MENU]
it work but only with small menu. with big menu I get not enough ram size.
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Fri May 27, 2016 6:14 am     Reply with quote

this..
Quote:
with big menu I get not enough ram size.

.. should be a clue.
Look at the end of the listing and see what resources are used
ROM%

RAM%

STACK.

'not enough RAM' means there isn't enough RAM to run your program !
The fact you're using a GLCD means you probably ave a HUGE mount of RAM given to the driver's db of 'fonts'. This is not shown in the program you've given us.
Since your 'menu' system/functions worked before, simply go back xx versions, and see what you've done since then. THAT is probably why 'it doesn't work'.

Jay
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

PostPosted: Fri May 27, 2016 6:19 am     Reply with quote

Hi Jay

I made no change, if a build a version with 5.50 it works if a build a version 5.59 it doesn't.
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Fri May 27, 2016 8:53 am     Reply with quote

OK, I copied your program, remarked out the GLCD code and it compiles fine, NO errors. Uses 2% ROM, 3-4% RAM.
One glaring issue I just saw is that you have the interrupts enabled with NO ISRs ! That is very, very bad and will lead to unpredictable problems like crashes, 'funny' results, etc.

Jay
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

PostPosted: Mon May 30, 2016 6:24 am     Reply with quote

Hi Jay

Your right living the interrupt enable is not a good idea for my sample prog but it's not the cause of the problem I get.
As I told you I have no build error so I suppose it's a buggy version of CCS
PCM programmer



Joined: 06 Sep 2003
Posts: 21708

View user's profile Send private message

PostPosted: Fri Jun 17, 2016 7:20 am     Reply with quote

It is a bug. I finally had time to investigate this thoroughly. There is
a problem with copying an array element from one array to another
when the source is a structure declared as 'const'. There are a few lines
of generated ASM code that are incorrect, starting with compiler vs. 5.057.
I did a complete write-up of the bug and sent it to CCS just now.
PCM programmer



Joined: 06 Sep 2003
Posts: 21708

View user's profile Send private message

PostPosted: Fri Jun 17, 2016 10:54 am     Reply with quote

CCS quickly emailed me a new PCH.DLL file. I used it to replace the one
in the DLL\5.059\ subdirectory in the CCS folder. It fixed the problem.
When I ran the test program shown below in MPLAB vs. 8.92 simulator
it displayed the following results in the Output Window. This is the correct output:
Quote:

> Menu1
> Menu2
> Menu2 sub1
> Menu2 sub2

Test program:
Code:
#include <18F46K20.h>
#fuses INTRC_IO, NOWDT, PUT, NOPROTECT, NOBROWNOUT
#use delay(clock=16M)
#use rs232(baud=9600, UART1, ERRORS)

#define ALL_MENU  4

enum ids{   _BEGIN,
            _MAIN,
            _MAIN1,
            _MAIN11,
            _MAIN12
        };
       
struct TypeMenu {
             int8 MenuID;
             int8 SubMenu;
             char MenuName[18];
            };
 
//-------------------------------------------
const struct TypeMenu MenuTest[ALL_MENU]
   ={
        {_MAIN, _BEGIN,"Menu1"},
        {_MAIN1,_BEGIN,"Menu2"},
        {_MAIN11,_MAIN1,"Menu2 sub1"},
        {_MAIN12,_MAIN1,"Menu2 sub2"}
   };

struct TypeMenu MenuPointer[2];


//==================================
void main()
{
int8 i;
int8 k;

k = 0;

for(i = 0; i < ALL_MENU; i++)
   {
    MenuPointer[k] =  MenuTest[i];
    printf("> %s \r", MenuPointer[0].MenuName); 
   }

while(TRUE);
}

After installing the new PCH.DLL file, the compiler reports its version as
5.060. If you need this DLL file, contact CCS (if you own the compiler)
or wait for vs. 5.060 to be released.
in_nursery



Joined: 25 Oct 2012
Posts: 51

View user's profile Send private message

PostPosted: Mon Jun 20, 2016 5:45 am     Reply with quote

thanks PCM programmer for your time and support.
I will contact ccs.
younder



Joined: 24 Jan 2013
Posts: 53
Location: Brazil

View user's profile Send private message

PostPosted: Sun Oct 09, 2016 8:56 pm     Reply with quote

Hi in_nursery

I'm still facing the same problem as yours. With latest CCS Compiler version this menu code doesn't work! Were you able to found a solution for this other than compile the code with 5.50?

Thanks
Hugo
_________________
Hugo Silva


Last edited by younder on Wed Oct 12, 2016 7:46 pm; edited 2 times in total
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Mon Oct 10, 2016 3:22 am     Reply with quote

I don't think you have the same problem.

The fix, is there in 5.061, 5.062, and 5.064 (just tested all of them, and they run fine).
Display posts from previous:   
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion All times are GMT - 6 Hours
Page 1 of 1

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001, 2005 phpBB Group