http://forum.codejock.com/ Copyright (c) 2006-2013 Web Wiz Forums - All Rights Reserved. Sun, 05 Apr 2026 11:47:03 +0000 Tue, 27 Jul 2010 17:43:59 +0000 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 12.04 360 forum.codejock.com/RSS_post_feed.asp?TID=17013 http://forum.codejock.com/forum_images/codejock-logo.gif http://forum.codejock.com/ http://forum.codejock.com/forum_posts.asp?TID=17013&PID=59483&title=cxtpreportpaintmanagersetgridstyle#59483 Author: ABuenger
Subject: 17013
Posted: 27 July 2010 at 5:43pm

Hello,

the function is defined as:

CXTPReportPaintManager::SetGridStyle(BOOL bVertical, XTPReportGridStyle gridStyle)

Using a BOOL for the orientation is poor in my opinion and makes the code harder to read.

Please overload it as:

enum XTPReportGridOrientation
{
    xtpReportGridOrientationVertical,
    xtpReportGridOrientationHorizontal,
    xtpReportGridOrientationBoth
};

CXTPReportPaintManager::SetGridStyle(XTPReportGridOrientation orientation, XTPReportGridStyle gridStyle)

Leave the old method for backwards compatibility, mark it as deprecated and get rid of it in the next version.

]]> Tue, 27 Jul 2010 17:43:59 +0000 http://forum.codejock.com/forum_posts.asp?TID=17013&PID=59483&title=cxtpreportpaintmanagersetgridstyle#59483