After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 720029 - docs/etc: Fix typos
docs/etc: Fix typos
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: Normal enhancement
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-12-07 16:09 UTC by Sebastian Rasmussen
Modified: 2013-12-07 17:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch fixing typos in documentation (77.09 KB, patch)
2013-12-07 16:10 UTC, Sebastian Rasmussen
committed Details | Review
Proposed patch fixing typos in "random" documentation (11.25 KB, patch)
2013-12-07 16:11 UTC, Sebastian Rasmussen
rejected Details | Review
Proposed patch fixing typos in debug and error messages (2.87 KB, patch)
2013-12-07 16:11 UTC, Sebastian Rasmussen
committed Details | Review
Proposed patch fixing typos in non-documentation comments (5.38 KB, patch)
2013-12-07 16:12 UTC, Sebastian Rasmussen
committed Details | Review

Description Sebastian Rasmussen 2013-12-07 16:09:59 UTC
I noticed several typos when fixing the GStreamer documentaiton, the attached patches fixes the ones I have noticed.

I divided the typo fixes into patches for the documentation, for docs/random, for debug/error messages and finally a patch only for patches in comments (that are not part of the documentation). I hope this makes it easier for you to choose what parts you care about and what parts you want to ignore (may random and comments..?).
Comment 1 Sebastian Rasmussen 2013-12-07 16:10:56 UTC
Created attachment 263713 [details] [review]
Proposed patch fixing typos in documentation
Comment 2 Sebastian Rasmussen 2013-12-07 16:11:23 UTC
Created attachment 263714 [details] [review]
Proposed patch fixing typos in "random" documentation
Comment 3 Sebastian Rasmussen 2013-12-07 16:11:47 UTC
Created attachment 263715 [details] [review]
Proposed patch fixing typos in debug and error messages
Comment 4 Sebastian Rasmussen 2013-12-07 16:12:11 UTC
Created attachment 263716 [details] [review]
Proposed patch fixing typos in non-documentation comments
Comment 5 Tim-Philipp Müller 2013-12-07 17:03:32 UTC
Comment on attachment 263714 [details] [review]
Proposed patch fixing typos in "random" documentation

I don't think we should touch these. Maybe we should go through them and remove what's not current or interesting any longer.
Comment 6 Sebastian Rasmussen 2013-12-07 17:09:09 UTC
> I don't think we should touch these. Maybe we should go through them and remove
> what's not current or interesting any longer.

I anticipated that, this is is why they are in a separate patch. It seems kind of awkward for GStreamer to have a scratch pad in the git repo, but ok... ;)

I'm not sure I'm qualified to go through other peoples scratch pad documents and throw stuff away though.
Comment 7 Tim-Philipp Müller 2013-12-07 17:26:50 UTC
Thanks pushed

 commit 53ae1b2c9c160a98a338adbb8a7ded0cad8eeebe
 Author: Sebastian Rasmussen <sebras@hotmail.com>
 Date:   Sat Dec 7 15:38:19 2013 +0100

    docs: Fix typos in function/object descriptions
    
    Fixes https://bugzilla.gnome.org/show_bug.cgi?id=720029

 commit e8ecf3c407e6733232013fafc52af8c98e931bc6
 Author: Sebastian Rasmussen <sebras@hotmail.com>
 Date:   Sat Dec 7 15:40:32 2013 +0100

    Fix some typos in code comments and debug messages
    
    https://bugzilla.gnome.org/show_bug.cgi?id=720029


> It seems kind of awkward for GStreamer to
> have a scratch pad in the git repo, but ok... ;)

I don't think it's awkward. It's somewhere where it can easily be found. It provides a history of people's ideas. And it seems to provide a better historical record than more transient things such as certain wikis.
Comment 8 Sebastian Rasmussen 2013-12-07 17:31:00 UTC
> Thanks pushed

Thanks for reviewing them.

> > It seems kind of awkward for GStreamer to
> > have a scratch pad in the git repo, but ok... ;)
> 
> I don't think it's awkward. It's somewhere where it can easily be found. It
> provides a history of people's ideas. And it seems to provide a better
> historical record than more transient things such as certain wikis.

Mmm, I have found somewhere that there are some links in the documentation to
a GStreamer wiki-page on how to submit patches. Of course the wiki no longer
has the page...
Comment 9 Tim-Philipp Müller 2013-12-07 17:39:41 UTC
It may still be recovered/revived, haven't had a chance to look into that again lately.