public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
@ 2023-01-31  9:17 marxin at gcc dot gnu.org
  2023-01-31  9:19 ` [Bug modula2/108612] " marxin at gcc dot gnu.org
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2023-01-31  9:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

            Bug ID: 108612
           Summary: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control
                    reaches end of non-void function [-Werror=return-type]
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: modula2
          Assignee: gaius at gcc dot gnu.org
          Reporter: marxin at gcc dot gnu.org
                CC: rguenth at gcc dot gnu.org
  Target Milestone: ---

It's very similar to PR108551.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
@ 2023-01-31  9:19 ` marxin at gcc dot gnu.org
  2023-01-31 13:14 ` gaius at gcc dot gnu.org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2023-01-31  9:19 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
I think the warning is correct:

   196  PROCEDURE dorbytes (g: GenDevIF; d: DeviceTablePtr;
   197                      to: ADDRESS;
   198                      max: CARDINAL;
   199                      VAR actual: CARDINAL) : BOOLEAN ;
   200  VAR
   201     fd: INTEGER ;
   202     c : ClientInfo ;
   203     p : PtrToLoc ;
   204     i : INTEGER ;
   205  BEGIN
   206     c := GetData(d, mid) ;
   207     WITH d^ DO
   208        IF max>0
   209        THEN <--- HERE we miss a return value for ELSE branch
   210           p := to ;
   211           IF getPushBackChar(c, p^)
   212           THEN
   213              actual := 1 ;
   214              RETURN( TRUE )
   215           END ;
   216           fd := getClientSocketFd(c) ;
   217           i := read(fd, p, max) ;
   218           IF i>=0
   219           THEN
   220              actual := i ;
   221              RETURN( TRUE )
   222           ELSE
   223              errNum := geterrno() ;
   224              actual := 0 ;
   225              RETURN( FALSE )
   226           END
   227        END
   228     END
   229  END dorbytes ;

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
  2023-01-31  9:19 ` [Bug modula2/108612] " marxin at gcc dot gnu.org
@ 2023-01-31 13:14 ` gaius at gcc dot gnu.org
  2023-02-01  0:57 ` gaius at gcc dot gnu.org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-01-31 13:14 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

Gaius Mulley <gaius at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2023-01-31

--- Comment #2 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Yes indeed the warning is correct and should return FALSE in the ELSE branch.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
  2023-01-31  9:19 ` [Bug modula2/108612] " marxin at gcc dot gnu.org
  2023-01-31 13:14 ` gaius at gcc dot gnu.org
@ 2023-02-01  0:57 ` gaius at gcc dot gnu.org
  2023-02-01 12:33 ` gaius at gcc dot gnu.org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-02-01  0:57 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

Gaius Mulley <gaius at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gaius at gcc dot gnu.org

--- Comment #3 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Created attachment 54380
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=54380&action=edit
Proposed fix

PR108612 - m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control
reaches end of non-void function [-Werror=return-type]
PR108551 - gcc/m2/gm2-libs-pim/Termbase.mod:128:1: error: control
reaches end of non-void function [-Werror=return-type]

The immediate fixes would be to ensure that
the offending procedure functions return a value.

However the front end does not correctly declared the throw function to be
noreturn.  Also HALT, Halt, Raise procedures in the libraries need
the <* noreturn *> attribute to be applied.  Finally when these changes
have been made the bootstrap tools mc and pge need rebuilding.

The patch also changes the build flags for m2 libraries to -Wreturn-type
-fcase.
(It it worth checking whether -fcase needs a 2nd implementation should
-fno-exceptions be used - and call Halt).

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2023-02-01  0:57 ` gaius at gcc dot gnu.org
@ 2023-02-01 12:33 ` gaius at gcc dot gnu.org
  2023-02-01 15:03 ` marxin at gcc dot gnu.org
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-02-01 12:33 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

--- Comment #4 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Created attachment 54383
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=54383&action=edit
Proposed fix v2

Here is version 2 of the proposed fix which should also help resolve
PR modula2/108551.

Note a new file in the patch: gcc/m2/mc-boot/GRTentity.h.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2023-02-01 12:33 ` gaius at gcc dot gnu.org
@ 2023-02-01 15:03 ` marxin at gcc dot gnu.org
  2023-02-01 15:53 ` gaius at gcc dot gnu.org
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2023-02-01 15:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

--- Comment #5 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Gaius Mulley from comment #4)
> Created attachment 54383 [details]
> Proposed fix v2
> 
> Here is version 2 of the proposed fix which should also help resolve
> PR modula2/108551.
> 
> Note a new file in the patch: gcc/m2/mc-boot/GRTentity.h.

The patch works for me, please install it soonish.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2023-02-01 15:03 ` marxin at gcc dot gnu.org
@ 2023-02-01 15:53 ` gaius at gcc dot gnu.org
  2023-02-01 17:26 ` cvs-commit at gcc dot gnu.org
  2023-02-03  9:02 ` gaius at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-02-01 15:53 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

--- Comment #6 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Will do - I'm currently seeing problems on a bootstrap build and regression
tests
(after a rebase).  Once this clears I'll commit / push the changes.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2023-02-01 15:53 ` gaius at gcc dot gnu.org
@ 2023-02-01 17:26 ` cvs-commit at gcc dot gnu.org
  2023-02-03  9:02 ` gaius at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-02-01 17:26 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Gaius Mulley <gaius@gcc.gnu.org>:

https://gcc.gnu.org/g:9fadd8dec79876d3c393daccc62959f6f4853cc5

commit r13-5634-g9fadd8dec79876d3c393daccc62959f6f4853cc5
Author: Gaius Mulley <gaiusmod2@gmail.com>
Date:   Wed Feb 1 17:26:00 2023 +0000

    Fixup noreturn attributes in modula-2 [PR108551] and [PR108612]

    PR108612 - m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control
    reaches end of non-void function [-Werror=return-type]
    PR108551 - gcc/m2/gm2-libs-pim/Termbase.mod:128:1: error: control
    reaches end of non-void function [-Werror=return-type]
    This patch adds missing return values to the procedure functions
    mentioned in PR108612 and PR108551.  It corrects the noreturn
    attribute to throw and rethrow in the modula2 frontend.  The patch
    also changes HALT, Halt, Raise procedures in the libraries to use
    the <* noreturn *> attribute.  Finally the patch includes
    rebuilt bootstrap tools mc and pge.

    gcc/m2/ChangeLog:

            * Make-lang.in (GM2_FLAGS): Add -fno-return -Wreturn-type.
            (GM2_ISO_FLAGS): Add -fno-return -Wreturn-type.
            * Make-maintainer.in (GM2PATH): Split into separate -I components.
            (MC-LIB-DEFS): Add RTentity.def.
            (m2/boot-bin/mc-devel$(exeext)): Changed -I$(GM2PATH) to
            $(GM2PATH).
            (m2/boot-bin/mc-opt$(exeext)): Separate -I paths.
            (m2/mc/decl.o): Separate -I paths.
            (gm2-bootstrap): Separate -I paths.
            (m2/mc-boot-gen/$(SRC_PREFIX)%.h): Separate -I paths.
            (m2/mc-boot-gen/$(SRC_PREFIX)decl.c): Separate -I paths.
            (m2/mc-boot-gen/$(SRC_PREFIX)%.c): Separate -I paths.
            (gm2.verifyparanoid): Separate -I paths.
            (gm2.verifystage12): Separate -I paths.
            * gm2-compiler/M2ALU.mod (GetConstructorElement): Add default
            Return NulSym.  Remove return from the error case.
            * gm2-compiler/M2Base.mod (ComplexToScalar): Return RType
            from the error case.
            (MixMetaTypes):  Return MakeError as a default.
            * gm2-compiler/M2GCCDeclare.mod (GetTypeMin): Return NulSym
            from the error case.
            (GetTypeMax): Return NulSym from the error case.
            * gm2-compiler/M2GenGCC.mod (IsExportedGcc): Replace Assert
            by InternalError.
            * gm2-compiler/M2Quads.mod (GetItemPointedTo): Add InternalError.
            (GetTypeMin): Add InternalError.
            (GetTypeMax): Add InternalError.
            * gm2-compiler/M2System.mod (InitSystem): Call
            PutProcedureNoReturn on Throw.
            * gm2-gcc/m2except.cc (m2except_InitExceptions): fn_throw_tree
            declare as noreturn.  fn_rethrow_tree declare as noreturn.
            * gm2-libs-coroutines/Debug.def (Halt): Add noreturn attribute.
            * gm2-libs-coroutines/SYSTEM.def (THROW): Add noreturn attribute.
            * gm2-libs-iso/ClientSocket.mod (dorbytes): Add return FALSE.
            * gm2-libs-iso/EXCEPTIONS.def (RAISE): Add noreturn attribute.
            * gm2-libs-iso/IOLink.def (RAISEdevException): Add noreturn
attribute.
            * gm2-libs-iso/M2RTS.def (HALT): Add noreturn attribute.
            (Halt): Ditto.
            (HaltC): Ditto.
            (ErrorMessage): Ditto.
            (AssignmentException): Ditto.
            (ReturnException): Ditto.
            (IncException): Ditto.
            (DecException): Ditto.
            (InclException): Ditto.
            (ExclException): Ditto.
            (ShiftException): Ditto.
            (RotateException): Ditto.
            (StaticArraySubscriptException): Ditto.
            (DynamicArraySubscriptException): Ditto.
            (ForLoopBeginException): Ditto.
            (ForLoopToException): Ditto.
            (ForLoopEndException): Ditto.
            (PointerNilException): Ditto.
            (NoReturnException): Ditto.
            (CaseException): Ditto.
            (WholeNonPosDivException): Ditto.
            (WholeNonPosModException): Ditto.
            (WholeZeroDivException): Ditto.
            (WholeZeroRemException): Ditto.
            (WholeValueException): Ditto.
            (RealValueException): Ditto.
            (ParameterException): Ditto.
            (NoException): Ditto.
            * gm2-libs-iso/SYSTEM.def (THROW): Ditto.
            * gm2-libs-iso/TermFile.mod (dorbytes): Add default return FALSE.
            * gm2-libs-min/M2RTS.def: Add noreturn attribute.
            * gm2-libs/FIO.mod (BufferedRead): Return -1.
            (getFileName): Return NIL.
            (getFileNameLength): Return 0.
            * gm2-libs/M2RTS.def (HaltC): Add noreturn attribute.
            (AssignmentException): Ditto.
            (ReturnException): Ditto.
            (IncException): Ditto.
            (DecException): Ditto.
            (InclException): Ditto.
            (ExclException): Ditto.
            (ShiftException): Ditto.
            (RotateException): Ditto.
            (StaticArraySubscriptException): Ditto.
            (DynamicArraySubscriptException): Ditto.
            (ForLoopBeginException): Ditto.
            (ForLoopToException): Ditto.
            (ForLoopEndException): Ditto.
            (PointerNilException): Ditto.
            (NoReturnException): Ditto.
            (CaseException): Ditto.
            (WholeNonPosDivException): Ditto.
            (WholeNonPosModException): Ditto.
            (WholeZeroDivException): Ditto.
            (WholeZeroRemException): Ditto.
            (WholeValueException): Ditto.
            (RealValueException): Ditto.
            (ParameterException): Ditto.
            (NoException): Ditto.
            * gm2-libs/RTExceptions.def (Raise): Ditto.
            * gm2-libs/RTExceptions.mod (InvokeHandler): Ditto.
            * gm2-libs/SYSTEM.def (THROW): Ditto.
            * m2.flex (_M2_m2flex_fini): Remamed to...
            (_M2_m2flex_finish): ...here.
            * mc-boot-ch/GBuiltins.c (_M2_Builtins_finish): Remamed to...
            (_M2_Builtins_fini): ...this.
            * mc-boot-ch/GRTco.c (_M2_RTco_finish): Remamed to...
            (_M2_RTco_fini): ...this.
            * mc-boot-ch/GSYSTEM.c (_M2_SYSTEM_finish): Remamed to...
            (_M2_SYSTEM_fini): ...this.
            * mc-boot-ch/GSelective.c (_M2_Selective_finish): Remamed to...
            (_M2_Selective_fini): ...this.
            * mc-boot-ch/GSysExceptions.c (_M2_SysExceptions_init): Add
            parameters.
            (_M2_SysExceptions_finish): Remamed to...
            (_M2_SysExceptions_fini): ...this.
            * mc-boot-ch/GUnixArgs.cc (_M2_UnixArgs_finish): Remamed to...
            (_M2_UnixArgs_fini): ...this.
            (_M2_UnixArgs_ctor::_M2_UnixArgs_ctor): Change parameter
            to _M2_UnixArgs_fini.
            * mc-boot-ch/Gdtoa.c (_M2_dtoa_finish): Remamed to...
            (_M2_dtoa_fini): ...this.
            * mc-boot-ch/Gerrno.c (_M2_errno_finish): Remamed to...
            (_M2_errno_fini): ...this.
            * mc-boot-ch/Gldtoa.c (_M2_ldtoa_finish): Remamed to...
            (_M2_ldtoa_fini): ...this.
            * mc-boot-ch/Gtermios.cc (_M2_termios_init): Add parameters.
            (_M2_termios_finish): Remamed to...
            (_M2_termios_fini): ...this.
            * mc-boot-ch/Gwrapc.c (_M2_wrapc_init): Add parameters.
            (_M2_wrapc_finish): Remamed to...
            (_M2_wrapc_fini): ...this.
            * mc-boot/GASCII.c: Rebuild.
            * mc-boot/GArgs.c: Rebuild.
            * mc-boot/GAssertion.c: Rebuild.
            * mc-boot/GBreak.c: Rebuild.
            * mc-boot/GCmdArgs.c: Rebuild.
            * mc-boot/GDebug.c: Rebuild.
            * mc-boot/GDynamicStrings.c: Rebuild.
            * mc-boot/GEnvironment.c: Rebuild.
            * mc-boot/GFIO.c: Rebuild.
            * mc-boot/GFormatStrings.c: Rebuild.
            * mc-boot/GFpuIO.c: Rebuild.
            * mc-boot/GIO.c: Rebuild.
            * mc-boot/GIndexing.c: Rebuild.
            * mc-boot/GM2Dependent.c: Rebuild.
            * mc-boot/GM2EXCEPTION.c: Rebuild.
            * mc-boot/GM2RTS.c: Rebuild.
            * mc-boot/GM2RTS.h: Rebuild.
            * mc-boot/GMemUtils.c: Rebuild.
            * mc-boot/GNumberIO.c: Rebuild.
            * mc-boot/GPushBackInput.c: Rebuild.
            * mc-boot/GRTExceptions.c: Rebuild.
            * mc-boot/GRTExceptions.h: Rebuild.
            * mc-boot/GRTco.h: Rebuild.
            * mc-boot/GRTint.c: Rebuild.
            * mc-boot/GSArgs.c: Rebuild.
            * mc-boot/GSFIO.c: Rebuild.
            * mc-boot/GStdIO.c: Rebuild.
            * mc-boot/GStorage.c: Rebuild.
            * mc-boot/GStrCase.c: Rebuild.
            * mc-boot/GStrIO.c: Rebuild.
            * mc-boot/GStrLib.c: Rebuild.
            * mc-boot/GStringConvert.c: Rebuild.
            * mc-boot/GSysStorage.c: Rebuild.
            * mc-boot/GTimeString.c: Rebuild.
            * mc-boot/Galists.c: Rebuild.
            * mc-boot/Gdecl.c: Rebuild.
            * mc-boot/Gkeyc.c: Rebuild.
            * mc-boot/Glists.c: Rebuild.
            * mc-boot/GmcComment.c: Rebuild.
            * mc-boot/GmcComp.c: Rebuild.
            * mc-boot/GmcDebug.c: Rebuild.
            * mc-boot/GmcError.c: Rebuild.
            * mc-boot/GmcFileName.c: Rebuild.
            * mc-boot/GmcLexBuf.c: Rebuild.
            * mc-boot/GmcMetaError.c: Rebuild.
            * mc-boot/GmcOptions.c: Rebuild.
            * mc-boot/GmcPreprocess.c: Rebuild.
            * mc-boot/GmcPretty.c: Rebuild.
            * mc-boot/GmcPrintf.c: Rebuild.
            * mc-boot/GmcQuiet.c: Rebuild.
            * mc-boot/GmcReserved.c: Rebuild.
            * mc-boot/GmcSearch.c: Rebuild.
            * mc-boot/GmcStack.c: Rebuild.
            * mc-boot/GmcStream.c: Rebuild.
            * mc-boot/Gmcp1.c: Rebuild.
            * mc-boot/Gmcp2.c: Rebuild.
            * mc-boot/Gmcp3.c: Rebuild.
            * mc-boot/Gmcp4.c: Rebuild.
            * mc-boot/Gmcp5.c: Rebuild.
            * mc-boot/GnameKey.c: Rebuild.
            * mc-boot/GsymbolKey.c: Rebuild.
            * mc-boot/Gtop.c: Rebuild.
            * mc-boot/Gvarargs.c: Rebuild.
            * mc-boot/Gwlists.c: Rebuild.
            * mc-boot/GRTentity.h: New file.
            * mc/decl.mod (scaffoldStatic): Change _finish to _fini.
            * mc/mc.flex (_M2_mcflex_fini): New function.
            (_M2_mcflex_finish): Remove function.
            * tools-src/mklink.c (GenerateFinishCalls): Change
            _finish to _fini.
            (GeneratePrototypes): Change _finish to _fini.

    libgm2/ChangeLog:

            * libm2cor/Makefile.am (libm2cor_la_M2FLAGS): Add -Wreturn-type
            -fcase.
            * libm2cor/Makefile.in: Rebuild.
            * libm2iso/Makefile.am (libm2iso_la_M2FLAGS): Add -Wreturn-type
            -fcase.
            * libm2iso/Makefile.in: Rebuild.
            * libm2log/Makefile.am (libm2log_la_M2FLAGS): Add -Wreturn-type
            -fcase.
            * libm2log/Makefile.in: Rebuild.
            * libm2pim/Makefile.am (libm2pim_la_M2FLAGS): Add -Wreturn-type
            -fcase.
            * libm2pim/Makefile.in: Rebuild.

            PR modula2/108612
            PR modula2/108551

    Signed-off-by: Gaius Mulley <gaiusmod2@gmail.com>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug modula2/108612] m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type]
  2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
                   ` (6 preceding siblings ...)
  2023-02-01 17:26 ` cvs-commit at gcc dot gnu.org
@ 2023-02-03  9:02 ` gaius at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-02-03  9:02 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108612

Gaius Mulley <gaius at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Closing now the patch has been applied.

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2023-02-03  9:02 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-31  9:17 [Bug modula2/108612] New: m2/gm2-libs-iso/ClientSocket.mod:229:1: error: control reaches end of non-void function [-Werror=return-type] marxin at gcc dot gnu.org
2023-01-31  9:19 ` [Bug modula2/108612] " marxin at gcc dot gnu.org
2023-01-31 13:14 ` gaius at gcc dot gnu.org
2023-02-01  0:57 ` gaius at gcc dot gnu.org
2023-02-01 12:33 ` gaius at gcc dot gnu.org
2023-02-01 15:03 ` marxin at gcc dot gnu.org
2023-02-01 15:53 ` gaius at gcc dot gnu.org
2023-02-01 17:26 ` cvs-commit at gcc dot gnu.org
2023-02-03  9:02 ` gaius at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).