java-gnome-hackers — Hacking on the bindings themselves: design issues and co-ordination

[java-gnome-hackers] strict

[Andrew C. <an...@op...>]

Since 4.0.8 the build has automatically enabled C defines like
GTK_DISABLE_DEPRECATED for anyone who was working from a Bazaar
checkout. These macros were not defined for people working from
releases, but it seemed a good idea for people hacking on the bindings
themselves.

I've just discovered that Ubuntu is shipping GTK 2.15 (and god knows
what else) in their J version, and so this was screwing up anyone trying
to causally build java-gnome on Ubuntu Jaunty from VCS.

I'm not super impressed about this, but to get around this I've changed
the previously elegant automatic solution to now be explicit.

If you run

$ ./configure strict

you'll get the -DGTK_DISABLE_DEPRECATED etc declarations passed to the
compiler.

Once GTK 2.16 is out sometime in the next 6 months then we'll do a pass
of getting everything ported to it and getting rid of anything that uses
such deprecated code. If someone with 2.15 wants to start working on
this now they're welcome to.

(ie, there's nothing wrong with starting to implement GtkOrientable;
it'd just have to be a bunch of no-ops for the bulk of cases until we
raise our minimum requirement to 2.16 once it's out)

AfC
Sydney

-- 
Andrew Frederick Cowie

Operational Dynamics is an operations and engineering consultancy
focusing on IT strategy, organizational architecture, systems
review, and effective procedures for change management: enabling
successful deployment of mission critical information technology in
enterprises, worldwide.

http://www.operationaldynamics.com/

Sydney   New York   Toronto   London

Re: [java-gnome-hackers] Bug in loadNativeCode with non-ascii characters in path

[Andrew C. <an...@op...>]

On Mon, 2009-01-26 at 19:41 +0200, Serkan Kaba wrote:

> The yield depends on whether File.exists() runs quicker than
> URLDecoder.decode() or not.

Indeed.

And none of us can say unless they do some profiling.

>  Even if that saves a bit of time, I don't
> think it's worth to complicate the code that runs only once on
> initialization.

True; and I certainly normally willing to pay a premium on code clarity
& subsequent project maintainability.

This, however, is on the critical path of code that the user has to wait
to complete before their application starts. So it certainly is the sort
of thing that I'd like to keep an eye on. But without hard data I'm not
going to screw you around.

We can merge it as is now.

AfC
Sydney

Re: [java-gnome-hackers] Bug in loadNativeCode with non-ascii characters in path

[Serkan K. <se...@ge...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andrew Cowie yazmış:
> I might suggest that you do the test as an alternative if the
> first .exists() check fails, thereby saving the decode calls for the
> common case where most people don't have the escaping problem?

The yield depends on whether File.exists() runs quicker than
URLDecoder.decode() or not. Even if that saves a bit of time, I don't
think it's worth to complicate the code that runs only once on
initialization. If you think it's worth it, I can implement that before
merge.

Sincerely,
Serkan KABA
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkl99dMACgkQRh6X64ivZaILZACdHUg2OIYPebNNnpf2LeA1ZJd6
0HAAn3hY6nBjViHBt4MEpSRfCeFdCTj8
=jfZ3
-----END PGP SIGNATURE-----

Re: [java-gnome-hackers] Bug in loadNativeCode with non-ascii characters in path

[Andrew C. <an...@op...>]

On Sun, 2009-01-25 at 14:18 +0200, Serkan Kaba wrote:
> After I moved my local java-gnome Bazaar repository under a directory
> with Turkish characters, I was unable to run it in place because of
> escaped url usage for file operations

The patch is fine.

I might suggest that you do the test as an alternative if the
first .exists() check fails, thereby saving the decode calls for the
common case where most people don't have the escaping problem?

Just a thought.

Otherwise I will merge it.

AfC
Sydney

Re: [java-gnome-hackers] Upgraded branches

[Andrew C. <an...@op...>]

On Sun, 2009-01-25 at 14:10 +0200, Serkan Kaba wrote:
> I guess not everybody is using bleeding edge software

Hm. Someone told me at the time that >= 1.6 was in Ubuntu's I version.
Oh well.

> Newer bzr is installable to 8.04, right?

Yes. Anyone with Ubuntu can use the Bazaar PPA to get the current
release, which is 1.11 with this apt line:

deb http://ppa.launchpad.net/bzr/ubuntu/ intrepid main

I do not understand why Canonical does not publish the current Bazaar in
the current Ubuntu.

There are dramatic improvements available in 1.6 and again in 1.9-1.11,
so you're far better off using the latest release of something like
this. People running Ubuntu should use Bazaar from the PPA.

Indeed, with that, Ubuntu people have 1.11, and Gentoo people of course
have 1.11 available. Pretty soon we'll be able to upgrade the public
branch to at format=1.9

++

FYI, one can always create a branch in an older format if you have a
newer client available. Serkan or anyone else could have done this if
they'd known.

++

Please don't top post.

AfC
Sydney

Re: [java-gnome-hackers] Upgraded branches

[Serkan K. <se...@ge...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

I guess not everybody is using bleeding edge software[1] :( Although bzr
1.6 was released after Ubuntu 8.04 I would expect it to be upgraded
online by now.Newer bzr is installable to 8.04, right?

1: http://article.gmane.org/gmane.comp.gnome.bindings.java/1648

Andrew Cowie yazmış:
> The 'mainline' branch was upgraded to a newer repository format today,
> so you'll need bzr >= 1.6 to use it. Given that Bazaar is at 1.9 now,
> that shouldn't be a problem for anyone.
> 
> AfC
> Sydney
> 
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> java-gnome-hackers mailing list
> jav...@li...
> https://lists.sourceforge.net/lists/listinfo/java-gnome-hackers

- --
Sincerely,
Serkan KABA
Gentoo/Java
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkl8VtEACgkQRh6X64ivZaKEoACePVaZFXibuo1UnFz9iu/8127B
wqoAn2DzdWOth5ZPJ2AOY0UwAa5jLv43
=ei0E
-----END PGP SIGNATURE-----

[java-gnome-hackers] Bug in loadNativeCode with non-ascii characters in path

[Serkan K. <se...@ge...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

After I moved my local java-gnome Bazaar repository under a directory
with Turkish characters, I was unable to run it in place because of
escaped url usage for file operations. I was able to fix the issue by
unescaping the string and pushed the branch to
bzr://research.operationaldynamics.com/bzr/java-gnome/hackers/serkan/bug-load-native/

Please test and consider its merge to mainline before 4.0.10 we soon
plan to release.

Thanks in advance,

Sincerely,
Serkan KABA
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkl8WJAACgkQRh6X64ivZaJSJACfYjFV3GJlTCvu0GvDYH5r9OUa
okUAnipjJTLRT19TKaoPcgtc4Ypp7pH0
=QRHw
-----END PGP SIGNATURE-----

Re: [java-gnome-hackers] setSourceRGB patch for SnapshotMatrix

[Andrew C. <an...@op...>]

On Thu, 2009-01-22 at 21:35 -0500, Kenneth Prugh wrote:
>

Merged to 'mainline'. Thanks.

AfC
Hobart

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Andrew C. <an...@op...>]

On Mon, 2009-01-19 at 16:40 +0100, Goundy wrote:
> +    /*
> +     * Perform progress from left to right
> +     */

So attention to detail is important. In this case, if you had previewed
your work in Eclipse's JavaDoc window, or run `make doc` and then
previewed the result in a web browser, you might have noticed that this
is not JavaDoc. Fixed.

Patch merged.

AfC
Hobart

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Andrew C. <an...@op...>]

On Mon, 2009-01-19 at 16:40 +0100, Goundy wrote:
> > You forgot to attach the bundle, please do so that we may review it.
> Sorry

And while we're at it, can you maybe try and quote a little less text
when replying to emails on the list? We all got the original message. No
need to quote it all back.

AfC
Hobart

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Goundy <go...@gm...>]

Kenneth Prugh wrote:
> On Sat, 17 Jan 2009 16:15:27 +0100
> Goundy <go...@gm...> wrote:
>
>   
>> Hi.
>>
>> Thank you Afc for your hints I corrected all what you pointed to.
>> And sorry for the time I put to do it but recently I got really hard 
>> exams but fortunately I succeeded.
>> (Two more very easy exams and I'm done :-))
>>
>> Well I hope this patch is better than the first one.
>> If there's more changes I should do, just reply back :-)
>>
>> Thanks and have a good day
>>
>> <snip>
>>     
>
> You forgot to attach the bundle, please do so that we may review it.
>   
> ------------------------------------------------------------------------
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by:
> SourcForge Community
> SourceForge wants to tell your story.
> http://p.sf.net/sfu/sf-spreadtheword
> ------------------------------------------------------------------------
>
> _______________________________________________
> java-gnome-hackers mailing list
> jav...@li...
> https://lists.sourceforge.net/lists/listinfo/java-gnome-hackers
>   
Ew Sorry -_-"

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Kenneth P. <ken...@gm...>]

On Sat, 17 Jan 2009 16:15:27 +0100
Goundy <go...@gm...> wrote:

> Hi.
> 
> Thank you Afc for your hints I corrected all what you pointed to.
> And sorry for the time I put to do it but recently I got really hard 
> exams but fortunately I succeeded.
> (Two more very easy exams and I'm done :-))
> 
> Well I hope this patch is better than the first one.
> If there's more changes I should do, just reply back :-)
> 
> Thanks and have a good day
>
> <snip>

You forgot to attach the bundle, please do so that we may review it.

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Goundy <go...@gm...>]

Hi.

Thank you Afc for your hints I corrected all what you pointed to.
And sorry for the time I put to do it but recently I got really hard 
exams but fortunately I succeeded.
(Two more very easy exams and I'm done :-))

Well I hope this patch is better than the first one.
If there's more changes I should do, just reply back :-)

Thanks and have a good day

Andrew Cowie wrote:
> In the main, your patch is fine. Some QA-ish points:
>
> On Sun, 2009-01-11 at 20:37 +0100, Miloud Bel wrote:
>   
>> - * Copyright (c) 2007-2008 Operational Dynamics Consulting Pty Ltd,
>> and Others
>> + * Copyright (c) 2009 Operational Dynamics Consulting Pty Ltd, and
>> Others
>>     
>
> The correct change is to make it "2007-2009".
>
>   
>> +    /**
>> +     * Set the orientation of the progress bar
>> +     * Four possible values:
>> +     * 
>> +     * LEFT_TO_RIGHT
>> +     * RIGHT_TO_LEFT
>> +     * TOP_TO_BOTTOM
>> +     * BOTTOM_TO_TOP
>>     
>
> If you run javadoc over this (ie, do `make doc`) you will discover that
> whitespace is not preserved (it is, to all intents, HTML) and so you'll
> need to make that an <ul> list or use <br> if you want to have four
> items.
>
> I'm not really sure you need to list them, but whatever.
> http://java-gnome.sourceforge.net/4.0/doc/api/org/gnome/pango/Style.html
> is an example of where we were verbose about it.
>
> You really ought to preview in a web browser. Like I said,
>
> $ make doc
>
>   
>> +     * @param orientation
>> +     *            a ProgressBarOrientation value
>>     
>
> You don't need that.
>
>   
>> +    /**
>> +     * Get the current progress bar orientation
>>     
>
> if you're going to talk about the proper name of a class, then use the
> class name:
>
>         * Get orientation currently in effect in this ProgressBar.
>
> or whatever. Perhaps Kenneth will volunteer to help you with the prose.
>
> ++
>
> You also really need to run the code formatter before committing. Serkan
> can help you with that, but if you're not in Eclipse:
>
> $ make format
>
> and if you are, Source -> Format.
>
> AfC
> Sydney
>
>   
> ------------------------------------------------------------------------
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by:
> SourcForge Community
> SourceForge wants to tell your story.
> http://p.sf.net/sfu/sf-spreadtheword
> ------------------------------------------------------------------------
>
> _______________________________________________
> java-gnome-hackers mailing list
> jav...@li...
> https://lists.sourceforge.net/lists/listinfo/java-gnome-hackers
>   

[java-gnome-hackers] Next up

[Andrew C. <an...@op...>]

We'll probably do 4.0.10 before the end of January. If anyone has a
branch they want merged they would do well to polish it up soon so it
has a chance of being merged in time.

AfC
Sydney

Re: [java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Andrew C. <an...@op...>]

In the main, your patch is fine. Some QA-ish points:

On Sun, 2009-01-11 at 20:37 +0100, Miloud Bel wrote:
> - * Copyright (c) 2007-2008 Operational Dynamics Consulting Pty Ltd,
> and Others
> + * Copyright (c) 2009 Operational Dynamics Consulting Pty Ltd, and
> Others

The correct change is to make it "2007-2009".

> +    /**
> +     * Set the orientation of the progress bar
> +     * Four possible values:
> +     * 
> +     * LEFT_TO_RIGHT
> +     * RIGHT_TO_LEFT
> +     * TOP_TO_BOTTOM
> +     * BOTTOM_TO_TOP

If you run javadoc over this (ie, do `make doc`) you will discover that
whitespace is not preserved (it is, to all intents, HTML) and so you'll
need to make that an <ul> list or use <br> if you want to have four
items.

I'm not really sure you need to list them, but whatever.
http://java-gnome.sourceforge.net/4.0/doc/api/org/gnome/pango/Style.html
is an example of where we were verbose about it.

You really ought to preview in a web browser. Like I said,

$ make doc

> +     * @param orientation
> +     *            a ProgressBarOrientation value

You don't need that.

> +    /**
> +     * Get the current progress bar orientation

if you're going to talk about the proper name of a class, then use the
class name:

        * Get orientation currently in effect in this ProgressBar.

or whatever. Perhaps Kenneth will volunteer to help you with the prose.

++

You also really need to run the code formatter before committing. Serkan
can help you with that, but if you're not in Eclipse:

$ make format

and if you are, Source -> Format.

AfC
Sydney

-- 
Andrew Frederick Cowie

Operational Dynamics is an operations and engineering consultancy
focusing on IT strategy, organizational architecture, systems
review, and effective procedures for change management: enabling
successful deployment of mission critical information technology in
enterprises, worldwide.

http://www.operationaldynamics.com/

Sydney   New York   Toronto   London

[java-gnome-hackers] ProgressBar Orientation patch (Need revision)

[Goundy <go...@gm...>]

Hi dear hackers,

Well, I just added the ProgressBarOrientation support and tried to 
document it but my english skills are too limited.
So I wish, if someone has a few time and would like to take look at my 
patch, that'd be nice :)
Attached the patch + a java class as example of use.

Cheers

Re: [java-gnome-hackers] Where is the code for loading libgtkjni.so

[Andrew C. <an...@op...>]

On Thu, 2009-01-08 at 16:16 +0800, Ke Wang wrote:
> I can't find [what loads the native library]

(if you're worrying about setting java.library.path from the command
line in order to run something, people don't have to do that anymore.
Quite a nice little achievement; mentioned in the 4.0.9 release notes,
http://java-gnome.sourceforge.net/4.0/NEWS.html#4.0.9
under "Making it easier to run java-gnome programs")

> Can anyone tell me where the code [that does that] is?

Sure. Grab a current branch of the code and look in:
src/bindings/org/freedesktop/bindings/Plumbing.java
starting at line 88.

AfC
Sydney

-- 
Andrew Frederick Cowie

Operational Dynamics is an operations and engineering consultancy
focusing on IT strategy, organizational architecture, systems
review, and effective procedures for change management: enabling
successful deployment of mission critical information technology in
enterprises, worldwide.

http://www.operationaldynamics.com/

Sydney   New York   Toronto   London

[java-gnome-hackers] Where is the code for loading libgtkjni.so

[Ke W. <Ke...@Su...>]

Hi,
I'm reading source code of Java-gnome now, but I can't find where the
code is to load libgtkjni.so.
Can anyone tell me where the related source code is?
Thank you!

Ke Wang

Re: [java-gnome-hackers] setSource patch for java-gnome

[Andrew C. <an...@op...>]

On Fri, 2009-01-02 at 19:27 -0500, Kenneth Prugh wrote:
>

Merged.

AfC

Re: [java-gnome-hackers] modifyFont patch

[Andrew C. <an...@op...>]

On Sat, 2009-01-03 at 00:43 +0100, Goundy wrote:
> Here's a new revision of my patch.

Merged.

AfC

[java-gnome-hackers] Pango work merged to 'mainline'

[Andrew C. <an...@op...>]

I've been working pretty hard at finishing up our 'pango' branch adding
functionality for drawing text, and tonight merged that work to
'mainline'.

It's looking pretty good at this point. Drawing text to images, screen,
and PDF backends are all working nicely. In particular, we have a lovely
example of drawing text to PDF which was some good fun (and more to the
point, an effective test bed).

Serkan Kaba and Kenneth Prugh have both been really helpful testing, and
of course Vreixo Formoso was the one who started the branch back in
April.

This also merges some work on a related 'cairo-image-caching' branch,
where I did my best to incorporate some of the knowledge and best
practices for image loading and efficient resource management passed on
by Carl Worth, Davyd Madeley, and Mathias Hasselman to whom we owe great
thanks.

Cheers,

AfC
Sydney

-- 
Andrew Frederick Cowie

Operational Dynamics is an operations and engineering consultancy
focusing on IT strategy, organizational architecture, systems
review, and effective procedures for change management: enabling
successful deployment of mission critical information technology in
enterprises, worldwide.

http://www.operationaldynamics.com/

Sydney   New York   Toronto   London

[java-gnome-hackers] Upgraded branches

[Andrew C. <an...@op...>]

The 'mainline' branch was upgraded to a newer repository format today,
so you'll need bzr >= 1.6 to use it. Given that Bazaar is at 1.9 now,
that shouldn't be a problem for anyone.

AfC
Sydney

-- 
Andrew Frederick Cowie

Operational Dynamics is an operations and engineering consultancy
focusing on IT strategy, organizational architecture, systems
review, and effective procedures for change management: enabling
successful deployment of mission critical information technology in
enterprises, worldwide.

http://www.operationaldynamics.com/

Sydney   New York   Toronto   London

[java-gnome-hackers] Building Java-Gnome on windows

[Fabio R. da R. <fab...@gm...>]

Hi all,
What is necessary to build java-gnome for running with windows. 
I noticed that you don't have binaries for it, and there are no
instructions with the source code for building it. 

Thanks in advance
Regards

-- 
Fabio Rafael da Rosa <fab...@gm...>

Re: [java-gnome-hackers] problem with compilation of java-gnome 4.0.9

[Kenneth P. <ken...@gm...>]

On Thu, 06 Nov 2008 19:49:31 +1100
Andrew Cowie <an...@op...> wrote:
> ...
> 
> Hooray.
> 
> Yikes!
> 
> Anyone else run into problems with GCC 4.3? I'm still on GCC 4.2
> myself.
> 
> AfC
> Sydney

I'm running GCC 4.3.2 here on Gentoo and it works fine. 

Re: [java-gnome-hackers] problem with compilation of java-gnome 4.0.9

[Andrew C. <an...@op...>]

On Thu, 2008-11-06 at 19:42 +1100, Federico wrote:
> do you remember of my problem with the compilation of java-gnome 4.0.9?
...

> The problem: ubuntu 8.10 intrepid updated
> gcc from 4.2 to 4.3 and this makes a lot of changes.
> So I tried to edit my .config file of java-gnome changing gcc-4.3 with
> gcc-4.2 and then the error *disappears*.

Hooray.

Yikes!

Anyone else run into problems with GCC 4.3? I'm still on GCC 4.2 myself.

AfC
Sydney