BugsBiteBugs / sqlitepersistentobjects

Automatically exported from code.google.com/p/sqlitepersistentobjects
0 stars 0 forks source link

Does not compile + patch of quick fixes to get it working #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Greets.

Tried to compile in Xcode for iPh*ne development, got some errors.  Made
some (somewhat dirty) fixes to get it working and am attaching the diff
file here...

Things that need to be fixed fer'real include my quick hacks to get it
working using UIKit

-#if (TARGET_OS_MAC && ! (TARGET_OS_EMBEDDED || TARGET_OS_ASPEN))
+#if (0 && TARGET_OS_MAC && ! (TARGET_OS_EMBEDDED || TARGET_OS_ASPEN))

as I wasn't sure how to detect this reliably. Also, the error message
container from sqlite3_exec was never declared, so I guessed at an
appropriate type... uhm, it probably needs to be freed too in the section:

+               char * errorMsg;
                if (sqlite3_exec(database, "PRAGMA encoding = \"UTF-8\"",
NULL, NULL, &errorMsg) !=
 SQLITE_OK) {

So that's it.  Compiles now, have yet to actually try it out... can't wait.
 Thanks for sharing, this is neat work :)

Regards,
Pat Deegan            

Original issue reported on code.google.com by goo...@psychogenic.com on 21 Oct 2008 at 2:30

Attachments:

GoogleCodeExporter commented 9 years ago
I don't think this patch is necessary - Apple changed the macro names, the code 
has since been updated to use 
the new ones.

Original comment by jeff.lam...@gmail.com on 21 Oct 2008 at 4:22

GoogleCodeExporter commented 9 years ago

Original comment by jeff.lam...@gmail.com on 21 Oct 2008 at 4:24

GoogleCodeExporter commented 9 years ago
Greets,

My comments above only underscored those parts of the patch I feel are, lets 
say, of
dubious value.

There are, however, a few fixes in there for typos/omissions that are required 
to get
it to compile on any system, eg:

-       [dateFormatter setDateFormat:@"yyyy-MM-dd HH:mm:ss.SSSS"
+       [dateFormatter setDateFormat:@"yyyy-MM-dd HH:mm:ss.SSSS"];

Regards,
Pat Deegan

Original comment by goo...@psychogenic.com on 22 Oct 2008 at 10:58