Report Control - CXTPReportPaintManager::SetGridStyle

Print Page | Close Window

CXTPReportPaintManager::SetGridStyle

Printed From: Codejock Forums
Category: Codejock Products
Forum Name: Report Control
Forum Description: Topics Related to Codejock Report Control
URL: http://forum.codejock.com/forum_posts.asp?TID=17013
Printed Date: 21 November 2024 at 12:12pm
Software Version: Web Wiz Forums 12.04 - http://www.webwizforums.com

Topic: CXTPReportPaintManager::SetGridStyle

Posted By: ABuenger
Subject: CXTPReportPaintManager::SetGridStyle
Date 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.

-------------
Codejock support