>
> void CallGraphPtr::saveCG( ostream& out ){
>     saveCallGraph( *callGraph, out );
> }
>
> Should be define or inline function

> int CallGraphPtr::whatIsIt( string name )
>
> it isn't clear what this function do from its name

> int CallGraphPtr::isCallExist( string who, string whom ){
>     for ( size_t i=0; i<calls.size(); ++i )
>         if ( calls[i].who == who && calls[i].whom == whom )
>             return i;
>     return -1;
> }
>
> function with such name must return logical value(0 or 1)



2010/6/15 Fedor Korshunov <[email protected]>

> Does Thu 19:30 works for everyone?
> I can't attend today, sry! :-(
>
>
> On Tue, Jun 15, 2010 at 11:48 AM, Андрей Лапин 
> <[email protected]>wrote:
>
>> Hello every one!
>> I want to suggest CodeReview of SA C pointer parser.
>> You can see it here:
>> http://bitbucket.org/lunatik/sacppointer/src/tip/src/common/callgraphptr.cpp
>> callgraphptr.cpp 360 number of lines
>> Requirement to attend: Andrew B, Andrew M, Feder K, Alex G, Yaroslav G.
>> And somebody else, if you want.
>> I suggest to thursday at 19-30, or some else day at the same time.
>> What you think about?
>>
>> --
>> Andrew Lapin
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "SourceAnalyzer Team" group.
>> To post to this group, send email to [email protected].
>> To unsubscribe from this group, send email to
>> [email protected]<sa_team%[email protected]>
>> .
>> For more options, visit this group at
>> http://groups.google.com/group/sa_team?hl=en.
>>
>
>
>
> --
> Best Regards,
> Fedor Korshunov
>
>  --
> You received this message because you are subscribed to the Google Groups
> "SourceAnalyzer Team" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected]<sa_team%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/sa_team?hl=en.
>



-- 
Starov, Evgeniy

-- 
You received this message because you are subscribed to the Google Groups 
"SourceAnalyzer Team" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sa_team?hl=en.

Reply via email to