Re: [Dpp-devel] Review Request: [REFACTOR] Renamed SharedProject to SarosSession. Renamed ISharedProject to ISarosSession.

[Karl B. <kar...@fu...>]

I agree that we should be all or nothing here, otherwise we will create
confusion in future. Sucks, but that's the nature of it.

If you break the task into remaining parts (e.g. method names,
documentation, lcoal variables etc.) I'll volunteer to take some of
them.

If anyone else can be persuaded to take some (e.g. your buddy) that
would be great too!

Karl


On Thu, 2010-08-05 at 10:50 +0200, Andreas Haferburg wrote:
> Yea well what can I say. It's 119 files. :)
> 
> I tried the "rename in comments" plus "also find suffixes" options, but 
> Eclipse would have also renamed *every* occurrence of "project" to 
> "session" in the comments. So I scratched the suffix option, but now 
> there are multiple places were local variables were called "project", 
> which I didn't rename to "session". Or "newSharedProject" or 
> "oldSharedProject". I did rename the ISessionManager#getSharedProject() 
> method though.
> 
> Best regards
> Andreas
> 
> 
> On 8/4/2010 10:13 PM, Christopher Oezbek wrote:
> >> Just Refactor->Rename's.
> >> I'm not sure if anyone would want to review this. I'll probably just
> >> commit it like this.
> >>      
> > Did you check if the documentation still makes sense?
> >
> > "Saros SarosSession"
> >
> > isInSharedProject ->  isInSession
> >
> > Christopher
> >
> > ------------------------------------------------------------------------------
> > The Palm PDK Hot Apps Program offers developers who use the
> > Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> > of $1 Million in cash or HP Products. Visit us here for more details:
> > http://p.sf.net/sfu/dev2dev-palm
> > _______________________________________________
> > Dpp-devel mailing list
> > Dpp...@li...
> > https://lists.sourceforge.net/lists/listinfo/dpp-devel
> >    
> 
> 
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://p.sf.net/sfu/dev2dev-palm
> _______________________________________________
> Dpp-devel mailing list
> Dpp...@li...
> https://lists.sourceforge.net/lists/listinfo/dpp-devel