Handle NULL attribute matchers safely, as we return this for empty

2008-01-03  Alexander Larsson  <alexl@redhat.com>

        * gfileinfo.c:
Handle NULL attribute matchers safely, as we return this
for empty attribute matcher strings.

svn path=/trunk/; revision=6238

Add /usr/local to list of internal mountpoints

2008-01-03  Alexander Larsson  <alexl@redhat.com>

* gunixmounts.c (g_unix_is_mount_path_system_internal):
Add /usr/local to list of internal mountpoints

svn path=/trunk/; revision=6237

Check for HAVE_LCHOWN (#505887)

2008-01-03  Alexander Larsson  <alexl@redhat.com>

        * glocalfileinfo.c:
Check for HAVE_LCHOWN (#505887)

svn path=/trunk/; revision=6236

Add define for selinux context attribute. Fix missing : -> :: namespace

2008-01-03  Alexander Larsson  <alexl@redhat.com>

        * gfileinfo.h:
        * glocalfileinfo.c:
Add define for selinux context attribute.
Fix missing : -> :: namespace separator change
Fix missing _ -> - name change for xattr-sys.
(#505058)

svn path=/trunk/; revision=6235

Link to libglib and libgobject directly (#504879) Patch from Sebastien

2008-01-03  Alexander Larsson  <alexl@redhat.com>

        * fam/Makefile.am:
Link to libglib and libgobject directly (#504879)
Patch from Sebastien Bacher

svn path=/trunk/; revision=6234

These void functions were trying to return a value. It was causing the

2008-01-02  Alvaro Lopez Ortega  <alvaro@sun.com>

        * gio/gunixmount.c (g_unix_mount_unmount, g_unix_mount_eject):
        These void functions were trying to return a value. It was causing
        the compilation to fail.

svn path=/trunk/; revision=6232

sv.po: Swedish translation updated

svn path=/trunk/; revision=6231

These functions prototype defines its output as void, and therefore they

2008-01-02  Alvaro Lopez Ortega  <alvaro@sun.com>

       * glib/ghash.c (g_hash_table_replace, g_hash_table_insert): These
       functions prototype defines its output as void, and therefore they
       should not return any value. This patch fixes a compilation error:
       the "return" clauses were incompatible with the functions prototype.

svn path=/trunk/; revision=6230

Expanded the g_file_new_for_commandline_arg description a bit, based on

2008-01-01  Wouter Bolsterlee  <wbolster@svn.gnome.org>

* gfile.c: Expanded the g_file_new_for_commandline_arg
description a bit, based on the code and the docs of the
other g_file_new_for_* functions.

svn path=/trunk/; revision=6229

Updated Occitan translation

svn path=/trunk/; revision=6228

Remove C99 comments

svn path=/trunk/; revision=6227

Updated Occitan translation

svn path=/trunk/; revision=6226

Fixed typo in docs.

2007-12-31  Wouter Bolsterlee  <wbolster@svn.gnome.org>

* gfilemonitor.h: Fixed typo in docs.

svn path=/trunk/; revision=6225

Fixed invalid XML entity

2007-12-31  Wouter Bolsterlee  <wbolster@svn.gnome.org>

* gio/migrating.xml: Fixed invalid XML entity

svn path=/trunk/; revision=6224

Fixed typo

2007-12-31  Wouter Bolsterlee  <wbolster@svn.gnome.org>

* glib/tmpl/macros_misc.sgml: Fixed typo

svn path=/trunk/; revision=6223

Updates to GIO documentation. (#506395, Mikael Hermansson)

* gcontenttype.c: Describe memory management for return value of
g_content_types_get_registered(). Missing piece from #505815.
* gdrive.c, gmount.c, gvolumemonitor.c: Add more description to
GVolume, GDrive, GMounts, which hopefully gives the user less
confusions when using this API. Following explainations from
Alexander Larsson on gtk-devel-list.

svn path=/trunk/; revision=6222

Mark assertion functions as G_GNUC_NORETURN. (#506461, Sebastian Dröge)

2007-12-24  Matthias Clasen  <mclasen@redhat.com>

        * glib/gtestutils.h:
        * glib/glib.symbols: Mark assertion functions as G_GNUC_NORETURN.
        (#506461, Sebastian Dröge)

svn path=/trunk/; revision=6221

Doc additions

svn path=/trunk/; revision=6220

Fix build with --disable-visibility

svn path=/trunk/; revision=6219

Don't include NULLs in the list of returned app infos.

2007-12-30  Matthias Clasen  <mclasen@redhat.com>

        * gdesktopfileinfo.c (g_app_info_get_all): Don't include NULLs
        in the list of returned app infos.

svn path=/trunk/; revision=6218

Fix a cross-reference

svn path=/trunk/; revision=6217

More docs

svn path=/trunk/; revision=6216

Doc updates

svn path=/trunk/; revision=6215

Updates

svn path=/trunk/; revision=6214

More doc updates

svn path=/trunk/; revision=6213

More doc updates

svn path=/trunk/; revision=6212

Documentation updates

svn path=/trunk/; revision=6211

Documentation updates

svn path=/trunk/; revision=6210

Documentation updates

svn path=/trunk/; revision=6209

Add a note on restrictions of child setup functions

svn path=/trunk/; revision=6208

Updated Basque translation.

2007-12-27  Inaki Larranaga Murgoitio  <dooteo@euskalgnu.org>

        * eu.po: Updated Basque translation.

svn path=/trunk/; revision=6207

Include crt_externs.h. (#505730, Tommi Komulainen)

2007-12-26  Matthias Clasen  <mclasen@redhat.com>

        * gdesktopappinfo.c: Include crt_externs.h.  (#505730,
        Tommi Komulainen)

svn path=/trunk/; revision=6206

Don't return freed memory (#505815, Mikael Hermansson)

2007-12-26  Matthias Clasen  <mclasen@redhat.com>

        * gcontenttype.c (g_content_types_get_registered): Don't return
        freed memory (#505815, Mikael Hermansson)

svn path=/trunk/; revision=6205

Fix macro argument. (#505674, Areg Beketovski)

2007-12-26  Behdad Esfahbod  <behdad@gnome.org>

        * gclosure.h (G_CCLOSURE_SWAP_DATA): Fix macro argument.
        (#505674, Areg Beketovski)

svn path=/trunk/; revision=6204

fix typo in the ifdef used to detect statbuf->st_blocks. (#505042)

2007-12-25  Paolo Borelli  <pborelli@katamail.com>

* glocalfileinfo.c (set_info_from_stat): fix typo in the ifdef
used to detect statbuf->st_blocks. (#505042)

svn path=/trunk/; revision=6202

Include sys/time.h. (#505258)

2007-12-24  Matthias Clasen  <mclasen@redhat.com>

        * glib/gtestutils.c: Include sys/time.h.  (#505258)

svn path=/trunk/; revision=6201

Fix the environment handling. (#504829, Cosimo Cecchi)

2007-12-24  Matthias Clasen  <mclasen@redhat.com>

        * gdesktopappinfo.c (g_desktop_app_info_launch): Fix the
        environment handling.  (#504829, Cosimo Cecchi)

svn path=/trunk/; revision=6200

Use correct date of in ChangeLog entry (once again)

svn path=/trunk/; revision=6199

Do not show empty groups in --help output. Initial patch from Yevgen
Muntyan. (#504142)

* glib/goption.c: Do not show empty groups in --help output.
* glib/tests/Makefile.am: Add option-context.c
* glib/tests/option-context.c: Test skipping of empty groups.

svn path=/trunk/; revision=6198

Add a warning about g_thread_init invalidating timers. (#491218)

* glib/tmpl/timers.sgml: Add a warning about g_thread_init
invalidating timers. (#491218)

svn path=/trunk/; revision=6197

Fix typo in g_try_new0 docs (#505195, Felix Riemann).

* docs/reference/glib/tmpl/memory.sgml: Reference
n_structs, instead of non-existant n_counts argument.

svn path=/trunk/; revision=6196

Updated Spanish translation

svn path=/trunk/; revision=6195

Updated Arabic Translation by Djihed Afifi.

svn path=/trunk/; revision=6193

Don't fail to load modules with suffix .la. (#480122, Andrey Tsyvarev)

2007-12-22  Matthias Clasen  <mclasen@redhat.com>

        * gmodule.c (g_module_open): Don't fail to load modules
        with suffix .la.  (#480122, Andrey Tsyvarev)

svn path=/trunk/; revision=6191

Remove wrong documentation about start group handling. (#476856, Areg

2007-12-22  Matthias Clasen  <mclasen@redhat.com>

        * glib/gkeyfile.c: Remove wrong documentation about start group
        handling.  (#476856, Areg Beketovski)

svn path=/trunk/; revision=6190

Fix a typo

svn path=/trunk/; revision=6189

Doc improvements

2007-12-22  Matthias Clasen  <mclasen@redhat.com>

        * gio/gappinfo.c: Doc improvements

        * gio/gdesktopappinfo.c (g_app_info_get_all): Return app infos,
        not ids.

svn path=/trunk/; revision=6188

use "sed -i.bak <CMD> <FILE> && rm -f <FILE>.bak" syntax for

2007-12-21 18:02:30  Tim Janik  <timj@imendio.com>

        * glib/Makefile.am: use "sed -i.bak <CMD> <FILE> && rm -f <FILE>.bak"
        syntax for install-exec-hook, which seems to be the only "sed -i"
        variant that is portable across linux and MacOS.

svn path=/trunk/; revision=6187

Updated Slovenian translation

svn path=/trunk/; revision=6186

Bring up to GLib coding standards: remove C99 comments, trailing commas in

2007-12-21  Matthias Clasen  <mclasne@redhat.com>

        * glib/gtestutils.h: Bring up to GLib coding standards: remove
        C99 comments, trailing commas in enumerations and extra ; after
        G_BEGIN/END_DECLS. Among other things, this makes xulrunner build
        against GLib 2.15.

        * glib/gtester.c: More of the same

svn path=/trunk/; revision=6185

adapt to recent api changes variable declaration at the beginning of a

2007-12-09  Hans Breuer  <hans@breuer.org>

* tests/gio-ls.c : adapt to recent api changes
* tests/testglib.c : variable declaration at the beginning of a block

(Lieven van der Heide, #503602)
* win32-fixup.pl : process *.rc.in as well; substitute
LT_CURRENT_MINUS_AGE
* glib/makefile.msc.in : alphabetic sorting of OBJECTS

svn path=/trunk/; revision=6184

Updated Irish translation.

2007-12-20  Seán de Búrca  <sdeburca@svn.gnome.org>

    * ga.po: Updated Irish translation.

svn path=/trunk/; revision=6183

Bump version

svn path=/trunk/; revision=6182

2.15.0

svn path=/trunk/; revision=6180

Updates

svn path=/trunk/; revision=6179

don't build gdesktopappinfo.obj, it collides with symbols

2007-12-20  Hans Breuer  <hans@breuer.org>

* makefile.msc : don't build gdesktopappinfo.obj, it collides
with symbols gwin32appinfo.obj, added gmount.obj
* gio.symbols : mark g_desktop_app_* as G_OS_UNIX

svn path=/trunk/; revision=6178

Fix make dist

svn path=/trunk/; revision=6177

commented class definitions. moved HTML character escaping out of

2007-12-20 16:34:04  Tim Janik  <timj@imendio.com>

        * glib/gtester-report: commented class definitions. moved HTML character
        escaping out of javascript. fixed string->bool conversions. added performance
        results to test case "Details" window.

svn path=/trunk/; revision=6176

Return NULL if the checksum_type is unknown

svn path=/trunk/; revision=6175

Use g_slice_new0, to fix "conditional jump or move depends on

2007-12-20  Christian Persch  <chpe@gnome.org>

* glib/gchecksum.c: (g_checksum_new): Use g_slice_new0, to fix
"conditional jump or move depends on uninitialised value(s)" error
from valgrind. Bug #504527.

svn path=/trunk/; revision=6174

Updated Slovenian translation

svn path=/trunk/; revision=6173

generate HTML reports for test-report perf-report full-report.

2007-12-20 15:17:04  Tim Janik  <timj@imendio.com>

        * Makefile.decl: generate HTML reports for test-report perf-report full-report.

svn path=/trunk/; revision=6172

new python script that generates an HTML unit test report from the XML

2007-12-20 15:03:51  Tim Janik  <timj@imendio.com>

        * glib/gtester-report: new python script that generates an HTML
        unit test report from the XML files generated by gtester.

        * glib/Makefile.am: install gtester-report in $bindir and configure
        it upon installation (version number and python shebang).

svn path=/trunk/; revision=6171

File attribute renames: std:: -> standard:: fs:: -> filesystem:: id::fs ->

2007-12-20  Alexander Larsson  <alexl@redhat.com>

        * gfile.c:
        * gfileattribute.c:
        * gfileinfo.c:
        * gfileinfo.h:
        * gfilenamecompleter.c:
        * glocalfile.c:
        * glocalfileinfo.c:
        * gpollfilemonitor.c:
File attribute renames:
std:: -> standard::
fs:: -> filesystem::
id::fs -> id::filesystem

svn path=/trunk/; revision=6170

Add GMountUnmountFlags to all unmount and eject calls. Add

2007-12-20  Alexander Larsson  <alexl@redhat.com>

        * gfile.[ch]:
        * gdrive.[ch]:
        * gmount.[ch]:
        * gvolume.[ch]:
        * gunixmount.c:
        * gunixvolume.c:
        * gio.symbols:
Add GMountUnmountFlags to all unmount and
eject calls.
Add g_mount_remount() call.

svn path=/trunk/; revision=6169

Fix unused variable warning

2007-12-20  Alexander Larsson  <alexl@redhat.com>

* gvfs.c (get_default_vfs):
Fix unused variable warning

svn path=/trunk/; revision=6168

Updates

svn path=/trunk/; revision=6167

Remove C99 comments

svn path=/trunk/; revision=6166

Additions

svn path=/trunk/; revision=6165

Additions

svn path=/trunk/; revision=6164

capture g_log() messages and send to gtester. also, send assertion

2007-12-19 20:30:18  Tim Janik  <timj@imendio.com>

        * glib/gtestutils.c: capture g_log() messages and send to gtester.
        also, send assertion messages to gtester.

        * glib/gtester.c: add error messages to output log file.
        force child poll loop to abort if waitpid() signaled child exit,
        eventhough the child's report file descriptor wasn't closed.

svn path=/trunk/; revision=6163

Add g_checksum_type_get_length, and change g_checksum_get_digest to use a

2007-12-19  Christian Persch  <chpe@gnome.org>

* glib/gchecksum.c: (g_checksum_type_get_length),
(g_checksum_get_digest):
* glib/gchecksum.h:
* glib/glib.symbols:
* tests/checksum-test.c: (test_checksum): Add
g_checksum_type_get_length, and change g_checksum_get_digest to use a
provided buffer instead of returning allocated memory. Bug #501853.

svn path=/trunk/; revision=6162

Fix the date validation check. (#503029)

2007-12-19  Emmanuele Bassi  <ebassi@gnome.org>

* glib/gtimer.c (g_time_val_from_iso8601): Fix the date validation
check. (#503029)

* tests/testglib.c (various_string_tests): Add an invalid date
for testing the above fix.

svn path=/trunk/; revision=6160

Make g_io_modules_load_all_in_directory not unuse loaded modules so that

2007-12-19  Alexander Larsson  <alexl@redhat.com>

        * giomodule.c:
Make g_io_modules_load_all_in_directory not unuse
loaded modules so that users of it can do stuff
before unloading.
Init internal "module" types.
Initialize static prio and name for types so that
we don't have to load modules to get it.

        * gnativevolumemonitor.h:
* gvolumemonitor.h:
Move is_supported to parent class so that
non-native monitors can avoid being initialized
too. (For instance GDaemonVolumeMonitor if we're
not using GDaemonVfs.)

        * glocaldirectorymonitor.[ch]:
        * glocalfilemonitor.[ch]:
* gunionvolumemonitor.c:
        * gunixvolumemonitor.c:
        * gvfs.c:
Find plugins using the static prio+name to
avoid unnecessarily loading the modules.

svn path=/trunk/; revision=6159

Rename g_format_file_size_for_display to g_format_size_for_display.

2007-12-19  Alexander Larsson  <alexl@redhat.com>

        * glib/gfileutils.[ch]:
        * glib/glib.symbols:
        Rename g_format_file_size_for_display to g_format_size_for_display.

svn path=/trunk/; revision=6158

Remove warnings

2007-12-19  Alexander Larsson  <alexl@redhat.com>

        * giomodule.c:
Remove warnings

svn path=/trunk/; revision=6157

Store the native type as GType, not class so that we can unload it. But

2007-12-19  Alexander Larsson  <alexl@redhat.com>

        * gunionvolumemonitor.c:
Store the native type as GType, not class so that
we can unload it. But still avoid unnecessarily
unload modules.

svn path=/trunk/; revision=6156

Updated Norwegian bokmål translation.

2007-12-19  Kjartan Maraas  <kmaraas@gnome.org>

* nb.po: Updated Norwegian bokmål translation.

svn path=/trunk/; revision=6155

Some minor fixes and additions.

2007-12-19  Alexander Larsson  <alexl@redhat.com>

        * gio/migrating.xml:
        * gio/overview.xml:
Some minor fixes and additions.

svn path=/trunk/; revision=6154

Introduce g_volume_monitor_adopt_orphan_mount() function. Also add signals

2007-12-19  David Zeuthen  <davidz@redhat.com>

Introduce g_volume_monitor_adopt_orphan_mount() function. Also
add signals 'disconnected' and 'eject-button' on GDrive. Add
signal 'removed' on GVolume and 'unmounted' on GMount.

* gdrive.c: (g_drive_base_init):
* gdrive.h:
* gfile.c: (g_file_mount_mountable),
(g_file_mount_enclosing_volume):
* gio.symbols:
* gioerror.h:
* gmount.c: (g_mount_base_init):
* gmount.h:
* gunionvolumemonitor.c: (g_volume_monitor_adopt_orphan_mount):
* gunixvolumemonitor.c: (update_volumes), (update_mounts):
* gvolume.c: (g_volume_base_init), (g_volume_mount):
* gvolume.h:
* gvolumemonitor.h:

svn path=/trunk/; revision=6153

add g_async_queue_new_full() which takes a GDestroyNotify function to free

* docs/reference/glib/glib-sections.txt:
* glib/gasyncqueue.c: (g_async_queue_new), (g_async_queue_new_full),
  (g_async_queue_unref):
* glib/gasyncqueue.h: add g_async_queue_new_full() which takes a
  GDestroyNotify function to free any remaining queue items when the
  queue is destroyed after the final atomic unref (#367550).

svn path=/trunk/; revision=6152

added g_test_trap_assert_stdout_unmatched() and

2007-12-18 13:45:23  Tim Janik  <timj@imendio.com>

        * glib/gtestutils.[hc]: added g_test_trap_assert_stdout_unmatched() and
        g_test_trap_assert_stderr_unmatched(), based on a suggestion by Mathias
        Hasselmann. reworked g_test_trap_assertions() to use flags to encode
        assertion semantics, fixes #504227.

svn path=/trunk/; revision=6151

Flesh out the docs

svn path=/trunk/; revision=6150

Fix up includes in section docs

svn path=/trunk/; revision=6149

Additions

svn path=/trunk/; revision=6148

Updated Spanish translation

svn path=/trunk/; revision=6147

Add some very modest beginnings

svn path=/trunk/; revision=6146

Stub of a migration chapter Stub of an overview Overview diagram taken

2007-12-17  Matthias Clasen <mclasen@redhat.com>

        * gio/migration.xml: Stub of a migration chapter
        * gio/overview.xml: Stub of an overview
        * gio/gvfs-overview.{odg,png}: Overview diagram taken
        from Alex Guadec slides.

        * gio/Makefile.am:
        * gio/gio-docs.xml: Include these

svn path=/trunk/; revision=6145

Add is_supported() to GNativeVolumeMonitorClass so that we can avoid

2007-12-17  Alexander Larsson  <alexl@redhat.com>

        * gnativevolumemonitor.h:
        * gunionvolumemonitor.c:
        * gunixvolumemonitor.c:
Add is_supported() to GNativeVolumeMonitorClass so
that we can avoid having to create an object to see
if the backend is supported at runtime.
Also add name member and an env var to pick a specific
volume monitor backend.

        * gmountprivate.h:
        * glocalfile.c:
Add cancellable to _g_mount_get_for_mount_path()

        * glocaldirectorymonitor.c:
        * glocalfilemonitor.c:
Avoid loading and unloading modules while sorting.

svn path=/trunk/; revision=6144

Namespace waste reduction, move some g_get_unix_mount functions to the

2007-12-17  Matthias Clasen  <mclasen@redhat.com>

        * gio.symbols:
        * gunixmounts.[hc]: Namespace waste reduction, move some
        g_get_unix_mount functions to the g_unix_mount namespace.

        * gunixmounts.c:
        * gunixvolumemonitor.c:
        * glocalfile.c:
        * glocaldirectorymonitor.c: Update all callers.

        * gunixmounts.h: Remove leftover g_unix_get_canonical_device_path

svn path=/trunk/; revision=6143

Add doc comments about what GFile operations are guaranteed to not block.

2007-12-17  Alexander Larsson  <alexl@redhat.com>

        * gfile.c:
Add doc comments about what GFile operations are
guaranteed to not block.

svn path=/trunk/; revision=6142

Add missing #ifdef fixing OSX build. (#503334, patch from Richard Hult)

2007-12-17  Alexander Larsson  <alexl@redhat.com>

        * gunixmounts.c:
Add missing #ifdef fixing OSX build.
(#503334, patch from Richard Hult)

svn path=/trunk/; revision=6141

Provide eject() on both GMount and GVolume and utility functions to guess

2007-12-14  David Zeuthen  <davidz@redhat.com>

* Makefile.am:
* gio.symbols:
* gmount.c: (g_mount_get_uuid), (g_mount_can_eject),
(g_mount_eject), (g_mount_eject_finish):
* gmount.h:
* gunionvolumemonitor.c: (g_union_volume_monitor_finalize),
(get_volume_for_uuid), (get_mount_for_uuid),
(g_union_volume_monitor_class_init),
(get_default_native_type_with_exclude), (get_default_native_type),
(get_native_type), (update_native_type),
(g_union_volume_monitor_init), (_g_mount_get_for_mount_path):
* gunixmount.c: (_g_unix_mount_new), (g_unix_mount_get_uuid),
(g_unix_mount_can_eject), (eject_unmount_cb),
(eject_unmount_read_error), (eject_unmount_do),
(g_unix_mount_unmount), (g_unix_mount_eject),
(g_unix_mount_eject_finish), (g_unix_mount_mount_iface_init):
* gunixmounts.c: (g_unix_mount_guess_can_eject),
(g_unix_mount_point_guess_can_eject):
* gunixmounts.h:
* gunixvolume.c: (_g_unix_volume_new), (g_unix_volume_get_uuid),
(g_unix_volume_can_eject), (g_unix_volume_get_drive),
(eject_mount_cb), (eject_mount_read_error), (eject_mount_do),
(g_unix_volume_mount), (g_unix_volume_eject),
(g_unix_volume_eject_finish), (g_unix_volume_volume_iface_init):
* gunixvolumemonitor.c: (get_volume_for_uuid),
(get_mount_for_uuid), (g_unix_volume_monitor_class_init),
(update_mounts):
* gvolume.c: (g_volume_get_uuid), (g_volume_can_eject),
(g_volume_eject), (g_volume_eject_finish):
* gvolume.h:
* gvolumemonitor.c: (g_volume_monitor_get_volume_for_uuid),
(g_volume_monitor_get_mount_for_uuid):
* gvolumemonitor.h:

Provide eject() on both GMount and GVolume and utility functions
to guess whether a GUnixMountPoint or GUnixMountEntry should be
ejected. Introduce the concept of UUID's and wire it into GVolume
and GMount and provide API on GVolumeMonitor to find such
instances. Also handle the case where an external
GNativeVolumeMonitor fails to initialize. Lock around the
_g_get_mount_for_mount_path() function such that volume monitor
implementations won't have to do locking themselves.

svn path=/trunk/; revision=6140

Updates

svn path=/trunk/; revision=6139

Documentation updates

svn path=/trunk/; revision=6138

Allow NULL strings in g_parse_debug_string. (#503862, Matthew Barnes)

* glib/gutils.c:
Allow NULL strings in g_parse_debug_string. (#503862, Matthew Barnes)

svn path=/trunk/; revision=6137

Updated Spanish translation

svn path=/trunk/; revision=6136

Add gio-ls.c Updated Norwegian bokmål translation.

2007-12-15  Kjartan Maraas  <kmaraas@gnome.org>

* POTFILES.in: Add gio-ls.c
* nb.po: Updated Norwegian bokmål translation.

svn path=/trunk/; revision=6135

More doc fixups

svn path=/trunk/; revision=6134

Fix up docs

svn path=/trunk/; revision=6133