AutoHotkey Community

[bmcclure]

Version 0.6.0 posted with all the changes we discussed. Most of them are included in the IniFile_test.ahk example now as well.

Let me know what you think, or if you see any errors I missed

New attributes:
headerComments under IniFile

New functions:
-IniFile_getComment
-IniFile_setComment
-IniFile_getHeaderComment
-IniFile_setHeaderComment
-IniFile_addHeaderComment
-IniFile_removeHeaderComment
-IniFile_loadFromVar
-IniFile_reload
-IniFile_reloadFromVar

Some functions have been overloaded with additional functionality. They should work exactly like before if you don't do anything differently, however.

Note that you can avoid calling the SortOptions parameter in every function by specifying you always want the file sorted:

[animeaime]

Looks good. Nice additions, can't wait to try them out.

I always use null ("") to mean an error (like removing an item from an empty Vector). I use 0 when zero (like an index) is returned. Also, I use 0 to mean the function was not successful (but not due to an error). For example, when destroying an object, it returns false (0) to indicate the object still exists (i.e. not ready to delete); whereas it returns null if the object is NULL (0 or blank), to indicate the error.
_________________
As always, if you have any further questions, don't hesitate to ask.

Add OOP to your scripts via the Class Library. Check out my scripts.

[animeaime]

Observation: I see a value for duplicate keys, but it doesn't look like it is ever used.

Edit:
Not sure if it would cause problems, but for IniFile_loadFromVar (and IniFile_reLoadFromVar), you might want <data> to be ByRef. Since loadFromVar should be passing a variable, it means that large amounts of data don't need to be copied first, because the value is ByRef. However, this only is possible if data will ALWAYS be a variable.

Edit2:
Bug report. In IniFile_clear, you should be clearing the sections - you currently set it to zero (destroying the sections Vector)

This fixes it:

[bmcclure]

Still working on duplicate keys, I sort of forgot to include that in the release and have been busy with the DOM library.

I'll try to get that in place tomorrow.

In the meantime, I've changed data (and its reference in the reloadFromVar function) to ByRef, removed the extra call you messaged me about, and changed IniFile_clear() to clear the vector to fix the bug you found.

Thanks as usual!

Edit New version uploaded with the fixes.
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[animeaime]

No rush, thanks for your great work.
_________________
As always, if you have any further questions, don't hesitate to ask.

Add OOP to your scripts via the Class Library. Check out my scripts.

[bmcclure]

New version (0.6.5) posted. Updating first topic.

1. Implemented changes in OOP Library V2, meaning:
- IniFiles, IniSections, and IniFields are all cloneable
- IniFiles, IniSections, and IniFields are all comparable

2. Fixed HeaderComment code to account for a change to the Vector class.

3. When a function has a problem, it now returns "" (null) instead of 0 (false)
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[bmcclure]

re-uploaded a new, very minor release, 0.6.6.

I had previously only used 4-bits per byte when setting bit flags, meaning I was using twice the amount of space I needed to for storing boolean values. (thanks, animeaime!)

I have now shortened IniFile objects from 23 bytes to 22
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[bmcclure]

I also fixed the Clone function to utilize a feature I was unaware of in the Class library to clean up the Settings-only IniFile clone operation.

It's such a minor change that I didn't update the version number, but feel free to grab the new release from the first post if you plan to utilize IniFile cloning.
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[bmcclure]

I'm starting to create an IniFileI interface to define what an IniFile class should be. Before I get too far, I want to check with anyone who has used this library, or any of the available Ini libraries:

Do you like the current function names in this library? Do you use the provided aliases to match Titan's functions, or do you access my functions directly?

if I'm going to make a change to the methods, I'd like to do it before releasing the interface if possible.
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[Mystiq]

Am i correct that in the current release none of the duplicate keys removal options work? If so, any news on when it might get added?

Also i'd like to say thanks

[bmcclure]

There are options (duplicateSections and duplicateKeys) you can set on the IniFile object to choose whether or not you want to allow duplicates in your IniFile.

After you create your IniFile object, use the following:

[ghee22]

When a value contains a ';' (semicolon), IniFile_read stops reading the value after the semicolon. Even suffixing a break ("`") before the semicolon fails to allow IniFile_read to read the complete line.

Here's my data:
task_notes=Execute following via Tivoli on all servers in environment:`ncd /var/spool/cron/crontabs`;for file in *`; do if [ -f $file ]`; then lines=`grep -v '^#'`n$file | grep -v '^$'|wc -l | sed 's/ //g'``;if [ "$lines" = "0" ]`; then crontab -r $file`;fi`;f`ni`;done`n

Here's my code:

[bmcclure]

This was built surrounding the recognized INI file "standards" (basically non-existent) stating that a semicolon indicates the start of a comment and everything after it should be ignored (see http://en.wikipedia.org/wiki/INI_file).

However you're right, some sort of escape sequence should be supported so that semicolons can be used within actual content.

I've been heavily focused on my C# apps and libraries lately, but will make it a point to touch up this INI library and allow escape sequences for characters with meanings.
_________________
Ben

My Trac projects
My Wiki
[Broken] - My music

[ghee22]

[kenn]

download link is dead