On Fri, Oct 25, 2013 at 11:25 AM, Cyd Haselton <chaselton@xxxxxxxxx> wrote:
> Stupid, stupid gmail interface. Once again, reposting to entire group:
>
> *snip*
>>>
>>> *snip*
>>> struct tm *tb = NULL;
>>> struct stat *st = _cpp_get_file_stat (file);
>>> if(st)
>>> time_t t = (time_t) st->st_mtime;
>>> tb = localtime (&t);
>>> /* tb= localtime (&st->st_mtime); */
>>
>> I'm guessing that you are used to Python? In C you need braces:
>>
>> if(st)
>> {
>> time_t t = (time_t) st->st_mtime;
>> tb = localtime (&t);
>> /* tb= localtime (&st->st_mtime); */
>> }
>
> Not really, but I understand the concept. But...and I went back and
> checked the original source file to make absolutely sure...there's no
> braces around that if statement. Here's a copy and paste of the
> original code:
>
> if (file)
> {
> /* super long comment. */
> struct tm *tb = NULL;
> struct stat *st = _cpp_get_file_stat (file);
> if (st)
> tb = localtime (&st->st_mtime);
> if (tb)
> {
>
> I can add them...I didn't before because, well, they weren't there.
Yes. In C, the syntax of an if statement is
"if" "(" <condition> ")" <statement>
A <statement> may be a single statement or it may be a brace-enclosed
block.
If you expect to proceed along these lines I really have to urge you
to learn C. C is a simple language, easy to learn if you already know
how to program. (C++, on the other hand, is quite complex.)
Ian
>>
>> Ian
>
> On Fri, Oct 25, 2013 at 12:54 PM, Ian Lance Taylor <iant@xxxxxxxxxx> wrote:
>> On Fri, Oct 25, 2013 at 8:59 AM, Cyd Haselton <chaselton@xxxxxxxxx> wrote:
>>>
>>> Just to clarify...adding the snippet you provided (thank you very
>>> much...by the way) to the code as follows should **not** throw an 'not
>>> declared in scope' error?
>>>
>>> *snip*
>>> struct tm *tb = NULL;
>>> struct stat *st = _cpp_get_file_stat (file);
>>> if(st)
>>> time_t t = (time_t) st->st_mtime;
>>> tb = localtime (&t);
>>> /* tb= localtime (&st->st_mtime); */
>>
>> I'm guessing that you are used to Python? In C you need braces:
>>
>> if(st)
>> {
>> time_t t = (time_t) st->st_mtime;
>> tb = localtime (&t);
>> /* tb= localtime (&st->st_mtime); */
>> }
>>
>> Ian
