Bug 158159 - 2 * missing return statements
2 * missing return statements
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: yelp (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ray Strode [halfline]
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2005-05-19 05:20 EDT by David Binderman
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2005-08-31 14:47:21 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description David Binderman 2005-05-19 05:20:46 EDT
Description of problem:

I just tried to compile package yelp-2.9.3-6 from Redhat
Fedora Core development tree with the most excellent Intel C
compiler.

The compiler said

1.

yelp-gecko-utils.cpp(211): warning #1011: missing return statement at end of
non-void function "yelp_gecko_copy_selection"

The source code is

extern "C" gboolean
yelp_gecko_copy_selection (GtkMozEmbed *embed)
{
        nsCOMPtr<nsIWebBrowser> webBrowser;
        gtk_moz_embed_get_nsIWebBrowser (embed, getter_AddRefs(webBrowser));

        nsCOMPtr<nsIClipboardCommands> clip (do_GetInterface(webBrowser));
        NS_ENSURE_TRUE (clip, NS_ERROR_FAILURE);

        clip->CopySelection();
}

Suggest add return statement to the end of the routine.

2.

yelp-gecko-utils.cpp(224): warning #1011: missing return statement at end of
non-void function "yelp_gecko_select_all"

Same thing again.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Ray Strode [halfline] 2005-08-31 14:47:21 EDT
These problems don't seem to be an issue anymore. Closing...
Comment 2 David Binderman 2005-08-31 15:19:20 EDT
>These problems don't seem to be an issue anymore. 

Seek clarification.

Does this mean that you have checked the source code manually,
and can verify that these two problems no longer exist ?
 
Comment 3 Ray Strode [halfline] 2005-08-31 15:51:51 EDT
Yes.  Those functions don't exist anymore as far as I can see.

Note You need to log in before you can comment on or make changes to this bug.