Re: Re: CvsAlert changes

"Jerzy Kaczorowski" <[email protected]>
Newsgroups gmane.comp.version-control.cvs.gui.devel
Message-ID <[email protected]>
I just commited my changes to CvsAlert API. The function is replaced with
the class. The contructor looks similiar to the function but it attempts to
guess default button depending on the icon type and there is no problem how
to layout the parameters. It will also try to eliminate the Cancel button
title if the message is a "note" type.

All that can be adjusted using the Get/Set methods if necessary.

Additionally there is a preliminary support for "Don't show this again"
checkbox which I will be putting to use soon. I still need to make some
small resizing work in WinCvs alert dialog code (CCVSAlert_MAIN) but common
CvsAlert class is unlikely to change any more so you can go ahead and fix
the build for gCvs and MacCvs.

I copied the function code into the CvsAlert::ShowAlert method. It should be
easy to fix the build by replacing the parameters with the CvsAlert class
members. The messages are stored as std::string values so you can test the
buttons and messages using the .empty() instead of checking the pointer
against the NULL.

I'm going to take a look at Oliver's path for Shell API.

BTW: Oliver, you should be able to get the "latest" from the repo and build.
I am not moving the next release tag just yet because I want to review all
latest changes more carefully and see if I have to branch off for the next
release or can I move ahead.

Best Regards,
Jerzy

------------------------ Yahoo! Groups Sponsor ---------------------~-->
Buy Ink Cartridges or Refill Kits for your HP, Epson, Canon or Lexmark
Printer at MyInks.com. Free s/h on orders $50 or more to the US & Canada.
http://www.c1tracking.com/l.asp?cid=5511
http://us.click.yahoo.com/mOAaAA/3exGAA/qnsNAA/NhFolB/TM
---------------------------------------------------------------------~->

To unsubscribe from this group, send an email to:
[email protected]

 

Your use of Yahoo! Groups is subject to http://docs.yahoo.com/info/terms/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.