Merge lp:~gotopad/do-plugins/Opera into lp:do-plugins/community

[gotopad]

[gotopad (gotopad)]

Plugin to index Opera 6 Bookmarks (based on the Epiphany-Bookmark Plugin).

[Alex Launi (alexlauni)]

You forgot to add the addin.xml file to the bzr tree. I can't test/merge without this. Other than that the code looks pretty good, some minor Mono guidelines fixes could be made; there should be a space between method calls, and parens:

myInstance.Method (arg1, arg2);

brackets should be on same line as property name:

public ICollection<IItem> Items {
    get { return items; }
}

If you're feeling ambitious, you could update this to the 0.7 API using IEnumerable<IItem> rather than Arrays, don't feel obligated to do this though, it would just be nice for me :P

review: Needs Fixing (community does-it-explode-test)

[lp:~gotopad/do-plugins/Opera]

2. By gotopad on 2008-12-20

Update Coding Guidelines

3. By gotopad on 2008-12-20

Added src-Directory

4. By gotopad on 2008-12-20

stupid bracket at try -.-

[gotopad (gotopad)]

The addin.xml file was just on the wrong place.
Now you can find the addin.xml in the src Directory.

I adapt the code to the mono coding guidelines : http://mono-project.com/Coding_Guidelines

I'm not familar with the 0.7 API, sorry for that.

[Alex Launi (alexlauni)]

Looks good, I'll merge this in tonight

review: Approve (community does-it-explode-test)