Present a warning to the user when attempting to print the current table's content view which exceeds a set number of rows. This number is currently set to 1000 which can be changed via the hidden preference key 'PrintWarningRowLimit'. The approach to cancelling the print HTML generation thread may also need to be re-implemented to ensure that it is killed straight away, something that is currently not possible using NSThread. Rowan suggested using pthreads instead, but this should be discussed in terms of potential memory leaks and stability.

1 files changed, 10 insertions, 8 deletions

diff --git a/Source/TableContent.m b/Source/TableContent.m
index b0e5e8fe..c5090573 100644
--- a/Source/TableContent.m
+++ b/Source/TableContent.m
@@ -1421,11 +1421,13 @@
 	[alert beginSheetModalForWindow:tableWindow modalDelegate:self didEndSelector:@selector(sheetDidEnd:returnCode:contextInfo:) contextInfo:contextInfo];
 }
 
-//getter methods
-- (NSArray *)currentDataResult
-/*
- returns the current result (as shown in table content view) as array, the first object containing the field names as array, the following objects containing the rows as array
+// Accessors
+
+/**
+ * Returns the current result (as shown in table content view) as array, the first object containing the field 
+ * names as array, the following objects containing the rows as array.
  */
+- (NSArray *)currentDataResult
 {
 	NSArray *tableColumns;
 	NSEnumerator *enumerator;
@@ -1480,11 +1482,11 @@
 	return currentResult;
 }
 
-//getter methods
-- (NSArray *)currentResult
-/*
- returns the current result (as shown in table content view) as array, the first object containing the field names as array, the following objects containing the rows as array
+/**
+ * Returns the current result (as shown in table content view) as array, the first object containing the field 
+ * names as array, the following objects containing the rows as array.
  */
+- (NSArray *)currentResult
 {
 	NSArray *tableColumns;
 	NSEnumerator *enumerator;