Re: oct-parse.yy:3727:29: error: invalid suffix "pp" on integer constant

[Ben Abbott]

On Oct 22, 2010, at 5:38 PM, Ben Abbott wrote:

> On Oct 22, 2010, at 5:33 PM, John W. Eaton wrote:
> 
>> On 22-Oct-2010, Ben Abbott wrote:
>> 
>> | Regarding the changeset below ...
>> | 
>> |    http://hg.savannah.gnu.org/hgweb/octave/rev/a51ac4198e1c
>> | 
>> | I'm getting an error for line 3727
>> | 
>> | oct-parse.yy:3727:29: error: invalid suffix "pp" on integer constant
>> | oct-parse.cc: In function 'int octave_parse()':
>> | oct-parse.cc:2362: warning: use of old-style cast
>> | oct-parse.cc:2379: warning: use of old-style cast
>> | oct-parse.cc:2419: warning: use of old-style cast
>> | make[3]: *** [liboctinterp_la-oct-parse.lo] Error 1
>> | make[2]: *** [all] Error 2
>> | make[1]: *** [all-recursive] Error 1
>> | make: *** [all] Error 2
>> | 
>> | 3721     {
>> | 3722       nm = octave_env::base_pathname (file);
>> | 3723       nm = nm.substr (0, nm.find_last_of ('.'));
>> | 3724 
>> | 3725       size_t pos = nm.find_last_of (file_ops::dir_sep_str ());
>> | 3726       if (pos != std::string::npos)
>> | 3727         nm = nm.substr (pos+1pp);
>> | 3728     }
>> | 
>> | I don't understand what is being done here, but should that be ...
>> | 
>> | 3727         nm = nm.substr (pos+1);
>> 
>> Yes, I checked in that fix.  Maybe I need a new keyboard.  One that
>> records what I mean, not what I type.
>> 
>> jwe
> 
> Now I see
> 
> ./run-octave -g --norc
> 
> ...
> run
> Program received signal EXC_BAD_ACCESS, Could not access memory.
> Reason: KERN_INVALID_ADDRESS at address: 0xfffffff4
> 0x903a40c1 in std::string::find ()
> (gdb) bt
> #0  0x903a40c1 in std::string::find ()
> #1  0x0132d499 in file_ops::tilde_expand (address@hidden) at file-ops.cc:280
> #2  0x0132e900 in file_stat::update_internal (this=0xbfff9974, force=false) 
> at file-stat.cc:189
> #3  0x00064720 in file_stat [inlined] () at file-stat.h:209
> #4  0x00064720 in file_stat [inlined] () at 
> /Users/bpabbott/Development/mercurial/local_clone/liboctave/file-stat.h:210
> #5  0x00064720 in frob_function (fname=<value temporarily unavailable, due to 
> optimizations>, fcn=0x4840064) at file-stat.h:2898
> #6  0x0006a264 in octave_parse () at oct-parse.yy:1404
> #7  0x0006b61c in parse_fcn_file (address@hidden, address@hidden, 
> force_script=false, require_file=true, address@hidden) at oct-parse.yy:3579
> #8  0x0006c2a7 in load_fcn_from_file (address@hidden, address@hidden, 
> address@hidden, address@hidden, autoload=false) at oct-parse.yy:3779
> #9  0x0007183a in symbol_table::fcn_info::fcn_info_rep::find_user_function 
> (this=0x4177ec0) at symtab.cc:962
> #10 0x0007405f in symbol_table::fcn_info::fcn_info_rep::xfind 
> (this=0x4177ec0, address@hidden, local_funcs=true) at symtab.cc:735
> #11 0x000741b5 in symbol_table::fcn_info::fcn_info_rep::find (this=0x4177ec0, 
> address@hidden, local_funcs=true) at symtab.cc:561
> #12 0x0007287f in symbol_table::fcn_info::find () at 
> /Users/bpabbott/Development/mercurial/local_clone/src/symtab.h:1311
> #13 0x0007287f in symbol_table::do_find (this=0x4136f00, address@hidden, 
> address@hidden, skip_variables=true, local_funcs=true) at symtab.cc:1311
> #14 0x00072a76 in symbol_table::find (address@hidden, address@hidden, 
> skip_variables=<value temporarily unavailable, due to optimizations>, 
> local_funcs=<value temporarily unavailable, due to optimizations>) at 
> symtab.cc:1110
> #15 0x00072f79 in symbol_table::find_function (address@hidden, 
> address@hidden, local_funcs=<value temporarily unavailable, due to 
> optimizations>) at symtab.cc:1145
> #16 0x00403467 in symbol_table::symbol_record::find (this=0x41775b8, 
> address@hidden) at symtab.cc:116
> #17 0x0069a58a in tree_identifier::do_lookup () at 
> /Users/bpabbott/Development/mercurial/local_clone/src/pt-id.h:91
> #18 0x0069a58a in tree_index_expression::rvalue (this=0x4177640, nargout=0, 
> lvalue_list=0x0) at pt-id.h:313
> #19 0x0069b22a in tree_index_expression::rvalue (this=0x4177640, nargout=0) 
> at pt-idx.cc:278
> #20 0x00699a7c in tree_index_expression::rvalue1 (this=0x4177640, nargout=0) 
> at pt-idx.cc:419
> #21 0x0068e2cc in tree_evaluator::visit_statement (this=0xe595e0, 
> address@hidden) at pt-eval.cc:729
> #22 0x006c0eff in tree_statement::accept (this=0x4161930, address@hidden) at 
> pt-stmt.cc:152
> #23 0x0068b0cb in tree_evaluator::visit_statement_list (this=0xe595e0, 
> address@hidden) at pt-eval.cc:765
> #24 0x0008333b in tree_statement_list::accept (this=0x4161960, 
> address@hidden) at pt-stmt.cc:216
> #25 0x005a7d31 in octave_user_script::do_multi_index_op (this=0x48df600, 
> nargout=0, address@hidden) at ov-usr-fcn.cc:135
> #26 0x003247dc in source_file (address@hidden, address@hidden, verbose=false, 
> require_file=true, address@hidden) at oct-parse.yy:3997
> #27 0x002a990c in execute_pkg_add_or_del (address@hidden, script_file=<value 
> temporarily unavailable, due to optimizations>) at load-path.cc:1917
> #28 0x002a9ac0 in execute_pkg_add (address@hidden) at load-path.cc:1923
> #29 0x002af514 in std::_List_iterator<load_path::dir_info>::operator++ () at 
> /usr/include/c++/4.2.1/bits/stl_list.h:588
> #30 0x002af514 in load_path::do_set (this=0x41370a0, address@hidden, 
> warn=false) at load-path.cc:583
> #31 0x002af847 in load_path::do_initialize (this=0x41370a0, 
> set_initial_path=false) at load-path.cc:516
> #32 0x003b59ce in octave_main (argc=7, argv=0xbfffafc4, embedded=0) at 
> load-path.h:53
> #33 0x00001f80 in main (argc=7, argv=0xbfffafc4) at main.c:35
> (gdb) 
> 
> Ben

After another fresh build, I don't see this anymore. However, something strange 
happens when the tests for chol.cc are run.

octave:2> test chol.cc
error: `__msg' undefined near line 502 column 16
error: evaluating argument list element number 1
error: called from:
error:   ?unknown? at line 502, column 1

The "?unknown?" function is test.m

Ben