vweb: remove `return vweb.Result{}` everywhere

CHANGELOG.md

@@ -5,7 +5,8 @@
 - Overloading of  `>`, `<`, `!=`, and `==` operators.
 - New struct updating syntax: `User{ ...u, name: 'new' }` to replace `{ u | name: 'new' }`.
 - `byte.str()` has been fixed and works like with all other numbers. `byte.ascii_str()` has been added.
-- Smart cast in for-loops: `for mut x is string {}`
+- Smart cast in for loops: `for mut x is string {}`.
+- `[noinit]` struct attribute to disallow direct struct initialization with `Foo{}`.
 
 ## V 0.2.1
 *30 Dec 2020*

tutorials/building-a-simple-web-blog-with-vweb.md

@@ -70,8 +70,7 @@ fn main() {
 }
 
 pub fn (mut app App) index() vweb.Result {
-	app.text('Hello, world from vweb!')
+	return app.text('Hello world from vweb!')
-	return vweb.Result{}
 }
 
 pub fn (app &App) init() {
@@ -110,8 +109,7 @@ import vweb
 import time
 
 fn (mut app App) time() vweb.Result {
-	app.text(time.now().format())
+	return app.text(time.now().format())
-	return vweb.Result{}
 }
 ```
 
@@ -352,8 +350,7 @@ pub fn (mut app App) new_article() vweb.Result {
 	title := app.form['title']
 	text := app.form['text']
 	if title == '' || text == '' {
-		app.text('Empty text/title')
+		return app.text('Empty text/title')
-		return vweb.Result{}
 	}
 	article := Article{
 		title: title
@@ -363,8 +360,7 @@ pub fn (mut app App) new_article() vweb.Result {
 	sql app.db {
 		insert article into Article
 	}
-	app.redirect('/')
+	return app.redirect('/')
-	return vweb.Result{}
 }
 ```
 
@@ -391,8 +387,7 @@ import json
 
 pub fn (mut app App) articles() vweb.Result {
 	articles := app.find_all_articles()
-	app.json(json.encode(articles))
+	return app.json(json.encode(articles))
-	return vweb.Result{}
 }
 ```
 

vlib/v/checker/tests/vweb_routing_checks.vv

@@ -5,29 +5,26 @@ struct App {
 }
 
 pub fn (mut app App) no_attributes(a string) vweb.Result {
-	return vweb.Result{}
+	return app.text('ok')
 }
 
 // works fine, as long as fcn gets 1 arg and route takes 1 var
 ['/foo/:bar']
 pub fn (mut app App) foo(a string) vweb.Result {
 	eprintln('foo')
-	app.html('works')
+	return app.html('works')
-	return vweb.Result{}
 }
 
 // segfault because path taks 0 vars and fcn takes 1 arg
 ['/bar']
 pub fn (mut app App) bar(a string) vweb.Result {
-	app.html('works')
+	return app.html('works')
-	return vweb.Result{}
 }
 
 // no segfault, but it shouldnt compile
 ['/cow/:low']
 pub fn (mut app App) cow() vweb.Result {
-	app.html('works')
+	return app.html('works')
-	return vweb.Result{}
 }
 
 pub fn (app App) init_once() {

vlib/vweb/tests/vweb_test_server.v

@@ -53,8 +53,7 @@ pub fn (mut app App) simple() vweb.Result {
 }
 
 pub fn (mut app App) html_page() vweb.Result {
-	app.html('<h1>ok</h1>')
+	return app.html('<h1>ok</h1>')
-	return vweb.Result{}
 }
 
 // the following serve custom routes
@@ -63,8 +62,7 @@ pub fn (mut app App) settings(username string) vweb.Result {
 	if username !in known_users {
 		return app.not_found()
 	}
-	app.html('username: $username')
+	return app.html('username: $username')
-	return vweb.Result{}
 }
 
 ['/:user/:repo/settings']
@@ -72,8 +70,7 @@ pub fn (mut app App) user_repo_settings(username string, repository string) vweb
 	if username !in known_users {
 		return app.not_found()
 	}
-	app.html('username: $username | repository: $repository')
+	return app.html('username: $username | repository: $repository')
-	return vweb.Result{}
 }
 
 [post]