glib bugs

[Owen Taylor <otaylor redhat com>]

Finally going through Matthias's GLib bug list. Below,  unless I
actually say that I changed the resolution, I haven't done so,
so [2.4.1] or whatever is a suggestion for how it should be
changed.

Regards,
						Owen

> GLib 2.2.4
> ==========
> +p 119261 gspawn can hang when limits are set too high
>    patch needs to be redone to incorporate Owens feedback

[2.4.1] if a redone patch shows up, we can apply.

> +  119490 system snprintf used even if broken
>    should be easy, just tighten up some configure checks

[2.4.0]

> ?p 11059 Linux poll issue
>    has a two years old patch which is said to need testing/debugging

[2.4.1] Still a potential issue (though no reports of problems), but 
I don't think patch is close to ready.

> ?p 116617 still failed assertions for EINTR
>    not clear if the patches are complete

[2.2.4] Added an OK-to-commit.

> ?p 107626 g_module_symbol(NULL, ...) fails with loaded modules
>    has a patch, but seems to require further thought and testing

[2.4.1] Don't want to make the change without the clarifications
I asked for.

> ?  82093 strsignal assumed to be either const or not - should be test
>    Owen proposes a simple solution for 2.4 

[2.4.1] Hmm, my solution I think is general, but requires writing a new
configure test.

> ?  98737 g_spawn* Win32 implementation needs work
>    partially done, no idea how hard the remaining bits are

[2.4.1] though Tor would know better.

> -  66199 Potential memory leak in gtk_init
>    not touched for more than a year, seems to be blocked on disagreement
>    between Owen & Tim

[WONTFIX] not a big deal either way.

> -  73861 iconv missing conversions on IRIX
>    not touched for more than a year, open issue is to remove a warning
>    if people start complaining about it

[WONTFIX] actually open issue was to add some missing charset.alias
entries for IRIX.

> -  101248 MSVC makefiles need updates
>    no idea what needs doing here

[NOTABUG] I think Hans has picked the msvc makefiles back up, so nothing
needed for now.

> -  101792 g_dir_read_name: filename encoding on Win32
>    not clear what to do here

[2.6 API Freeze] the path is clear, but too late for the API additions
to fix it.

> -  118952 G_THREAD_LIBS used instead of G_THREAD_LIBS_FOR_GTHREAD in p
>    toolchain/configuration problem on s390

[2.4.1/Low] Basically comes down to debugging someone's system...

> GLib 2.4 API freeze
> ===================
> *p 50296 UNIX signal main loop source
>    has been through multiple iterations

I'd really like to get Tor's comments on this, and on implementability
on Windows.

Other than that, we should go ahead and add this... if I remembered
what was going on at all, I'd look through the changes since the
last patch, but I don't at this point. Note jrb's comment about 
extraneous chunks in the patch

Do we need GPid or something like that that is 'int' on UNIX and
a process handle on Windows? GSpawn is already problematical, but
we shouldn't propagate the breakage. (And I assume passing handles
through ints will break for Win64...)

This bug evolved into something different, so we may want to leave it
open after the patch is applied and move it to [future]

> *p 78414 g_queue_remove() and g_queue_remove_all()
>    has been though multiple iterations

Wouldn't hurt for someone other than Soeren to check through the 
latest revision, but the version I reviewed was close to ready.
The "random" test case needs to seed the RNG with a fixed seed,
test cases that fail, then we can't reproduce the failure do
no good.

> *p 88329 Add g_strtokenize()
>    even has tests     

[2.4 API freeze] Added some queries about the name and UTF-8
completeness. Maybe a bit late for 2. if we are making changes?

> *p 98536 Add g_timer_continue()
>    this one too

[2.4 API freeze] Looks basically fine, but added various patch comments.

> +p 63621 Add GAtomicInt for atomic integer operations
>    needs to be expanded to other architectures besides i486
> ?p 121027 avoid emission of class closure only signals
>    Tim wasn't convinced of the approach 

Up to Tim, but I can't say I like it much.

> ?p 51639 Equivalent of __FUNCTION__ for Sun's compiler
>    patch needs to be redone to incorporate Owens feedback

Already RESOLVED/FIXED

> ?  81040 G_TYPE_GTYPE is missing from fundamental types
>    should be easy

[2.6 API Freeze] since no patch seems to be forthcoming.

> ?  99226 Add G_HIDDEN
>    should be easy 
>    (Owens comment from Sep 12: Might be worth looking at the 'slim' 
>     stuff that Cairo is using for alternate symbol-hiding magic.)

[2.6 API Freeze]. Probably should add something about slim
to the bug report.

> -p 80498 Added function g_io_channel_new_pipe()
>    needs a win32 implemenation to be useful

Just marked it RESOLVED/WONTFIX.

> -p 111848 function to canonicalize file names
>    needs to be modified to not follow symlinks, win32 implementation
>    (Owens comment from Sep 12: I don't think we really have a good sense
>     of exactly what we want here yet.)

[2.6 API Freeze]

> -  83729 Patches to add iterators to GLib
>    de facto WONTFIXed already, judging by the comments
>    (Owens comment from Sep 12: I consider this a wontfix, but *still* 
>     haven't written up the detailed rational for that)

Marked RESOLVED/WONTFIX. See the mail I just sent.

> -  113075 support "nonnull" attribute
>    not very useful
> -  113697 g_ptr_array_sort_[with_data] is twisted
>    the twistedness has at least been documented
>    (Owens comment from Sep 12: I really don't know what to do with this 
>     one. We need to come up with replacement names and deprecate the 
>     current functions)

[2.6 API Freeze]

> -  122057 Add g_warning_location() 

[2.6 API Freeze]

> -  120299 Win32 iochannel/g_poll/etc code needs rewrite
>    no patch, sounds like a lot of work

[2.6 API freeze]

> GLib 2.4
> ========
> *p 118536 Make g_signal_connect_object'ed handlers disconnect when the
>    patch has had some iterations

Badly needs Tim review.

> *p 118973 Pan does not understand "&" in username field of /etc/passwd
>    implements a relatively obscure traditional passwd feature  

[2.4.0]

> +p 113116 Consider caching of type check information
>    seems to provide a "not insignificant" speedup
> +p 117067 Speed up g_param_spec_pool_lookup    
> ?p 101426 GScanner is very slow with big files
>    sitting there since December last year, waiting for a comment 
>    from Tim...

Needs review from Tim.

> -  106925 g_new0 performance
>    should be simple

[2.6.0] since it is a non-stable-branch change, though trivial.

> -  116308 Support ./configure XGETTEXT=blah
>    sounds like a simple configure tweak   

[2.4.1] Moderately simple, yes.

> -  71704 Header file include order
>    purely janitorial

[2.4.1]

> -  100697 Problems with -pthread and shared libraries
>    just left open for removing some configure hacks when a fixed
>    libtool appears one day

[2.4.1]

> -  108038 Screwed-up headers guards
>    purely janitorial

[2.4.1]

> -  119615 glib-gettext.m4 obsolete macro
>    not clear what to do

[2.4.1] Eventually upstream gettext will have this fixed, and then
we can copy what they did :-)

> -  57691 check error conditions in backend read/write/seek
>    no action on this for a long time; win32 specific

[2.4.0] Just needs Tor to look at it briefly, and change the messages
if appropriate.